00:01:12 <03w​heals> probably it's just a bad idea though, i'm just thinking out loud that we can fix up the species enum each time the list might change 00:01:41 <03w​heals> ...of course the advantage of autogeneration is that you don't need to touch the c++ code, and having to mess around in tags.cc negates that 00:14:15 <10P​leasingFungus> good news: this code turns meteorans back into palentongas 00:14:24 <10P​leasingFungus> bad news: it also turns everything else into palentongas 00:14:26 <10P​leasingFungus> glad i tested 00:14:36 <09g​ammafunk> lol 00:18:25 <10P​leasingFungus> ok, i pushed maybe a fix 00:18:27 03PleasingFungus02 07* 0.29-a0-529-gfa20dd96a1: Stop turning palentongas into meteorans? 10(13 minutes ago, 7 files, 43+ 0-) 13https://github.com/crawl/crawl/commit/fa20dd96a133 00:18:31 <10P​leasingFungus> this might make things worse 00:18:31 <10P​leasingFungus> we'll see 00:18:33 <10P​leasingFungus> review welcome! 00:20:01 <10P​leasingFungus> maybe should've pushed this to a branch 00:39:22 Unstable branch on crawl.kelbi.org updated to: 0.29-a0-529-gfa20dd96a1 (34) 00:40:33 <10P​leasingFungus> ??rebuild 00:40:34 <04C​erebot> rebuild[1/2]: http://crawl.akrasiac.org/rebuild/ http://underhound.eu:81/rebuild/ https://cbro.berotato.org/rebuild/ http://crawl.xtahua.com/rebuild/ https://crawl.kelbi.org/rebuild/ Bug gammafunk, advil, |amethyst, or Nap.Kin for CDO. Use your powers wisely. 00:41:03 <10P​leasingFungus> ??rebuild[2 00:41:04 <04C​erebot> rebuild[2/2]: Trunk update start times in UTC (second number during summer time): CAO: 1700; CDO: 0600/0500; CUE: 2300/2200; CXC: 0500/0400; CPO: every 15 minutes; CKO: trunk every 15 minutes, other versions 0830/0730. 00:41:14 <10P​leasingFungus> ok, cpo should be done soon 00:46:37 <10P​leasingFungus> ...soon? 00:47:57 cko rebuilt 00:48:05 <10P​leasingFungus> yes, i saw 00:48:07 <10P​leasingFungus> but this player is on cpo 00:48:09 <10P​leasingFungus> &versions 00:48:16 yeah hopefully cpo will just trigger on its own soon, we have no rebuild url for that iirc 00:48:17 <04C​erebot> CAO: 0.29-a0-520-gc852659, CBR2: 0.29-a0-520-gc852659d43, CDO: 0.29-a0-517-gbdf32494f2, CKO: 0.29-a0-528-g05f53baf05, CPO: 0.29-a0-528-g05f53ba, CUE: 0.29-a0-521-g59ce8b2d47, CWZ: 0.29-a0-520-gc852659d43, CXC: 0.29-a0-520-gc852659d43, LLD: 0.28.0-53-g5a82cb258f 00:48:48 good lld trunk 00:51:37 <10P​leasingFungus> rather 01:34:50 Unstable branch on crawl.develz.org updated to: 0.29-a0-529-gfa20dd96a1 (34) 01:56:23 Windows builds of master branch on crawl.develz.org updated to: 0.29-a0-529-gfa20dd96a1 02:54:12 Monster database of master branch on crawl.develz.org updated to: 0.29-a0-529-gfa20dd96a1 03:01:42 Unstable branch on cbro.berotato.org updated to: 0.29-a0-529-gfa20dd96a1 (34) 03:39:24 Fork (bcrawl) on crawl.kelbi.org updated to: 0.23-a0-4719-g50af30713d 05:30:10 <09g​ammafunk> @PleasingFungus lici pointed out a crash for a CXC user upgrading from 0.29-a0-520-gc852659d43 to current trunk (commit 529 at present), character looks like https://dpaste.com/9DLAV3PZF and is crashing with a floating point exception. I was able to recreate by wizmode loading that dump in the commit 520 version and attempting to transfer to commit 529. Crash dump with backtrace: https://dpaste.com/AN6GX3SE9 06:22:16 Experimental (bcrawl) branch on underhound.eu updated to: 0.23-a0-4719-g50af30713d 08:02:52 03kate-02 07* 0.29-a0-530-g1b1fc82263: Allow ice beast zombies again 10(49 minutes ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/1b1fc8226373 08:02:52 03kate-02 07* 0.29-a0-531-g67b01d5277: Mark scarves of invisibility useless when perma-backlit 10(16 minutes ago, 1 file, 2+ 1-) 13https://github.com/crawl/crawl/commit/67b01d527791 08:02:52 03kate-02 07* 0.29-a0-532-gdd5a78fe54: Don't suppress Meteoran glow in forms 10(14 minutes ago, 7 files, 24+ 17-) 13https://github.com/crawl/crawl/commit/dd5a78fe54c1 08:28:46 Unstable branch on crawl.akrasiac.org updated to: 0.29-a0-529-gfa20dd9 (34) 08:32:33 Unstable branch on crawl.kelbi.org updated to: 0.29-a0-532-gdd5a78fe54 (34) 08:52:08 <10P​leasingFungus> @kate oh, i quite liked forms suppressing glow 08:52:21 <10P​leasingFungus> allowed tech with tmuter 08:52:52 <10P​leasingFungus> invisible tree! 08:53:42 <10P​leasingFungus> it's also why they have tmuter recommended 08:53:49 <05k​ate> i think it mostly ends up making the UI worse for all non-tm characters (also encourages weirdness like exploring in forms for stealth bonus) 08:55:49 <05k​ate> i don't think the benefit of giving a niche bonus to tm is really worth that 08:56:01 <10P​leasingFungus> mm, exploring in forms not great, sure 08:56:20 <10P​leasingFungus> i just want emergent interactions... is that so wrong 08:57:26 <05k​ate> heh, some emergent interactions are good! i just don't think this one really is, sadly 09:02:24 <10P​leasingFungus> rip 09:06:04 -!- cot is now known as kiedtl 09:10:20 Re: Meteoran lore: fun fact, I have uhh... two degrees of separation from Neil Gaiman? My parents have a friend whose good friends with him and house sits for him in the UK. 09:11:30 Also I didn't see the LOS flavor gimmick—that's really neat. I dig it. 09:17:45 So do I, if "Pratchett and I had conversations while both of us were urinating" counts 09:23:20 That's pretty good 10:12:26 <06a​dvil> looks like some fun merging happend 10:15:04 <06a​dvil> I really cannot remember why anyone ever thought it was a good idea to reorder the species_type enum even on a tag major version change 10:15:51 <06a​dvil> because that would break save browser compat for incompatible files, which would be unfortunate 10:21:12 <10P​leasingFungus> yeah, sounds awful 10:21:29 <10P​leasingFungus> think that whole enum gen logic should be yanked out 10:21:34 <10P​leasingFungus> just didn’t have the time 10:22:05 <10P​leasingFungus> probably won’t be able to look at that crash until after work today, either 😦 10:29:14 <06a​dvil> divide by 0 crashes don't replicate on mac fwiw 10:29:26 <06a​dvil> they just silently get converted to 0 10:29:55 <06a​dvil> been bit by this many times 10:31:41 <10P​leasingFungus> great. 10:50:36 <10P​leasingFungus> my last commit turned me into my 10:50:39 <10P​leasingFungus> sigh 11:07:25 <06a​dvil> lol oops 11:07:44 <06a​dvil> probably my should just be explicitly fixed up to me anyways 11:07:52 <10P​leasingFungus> sure 11:14:43 <06a​dvil> I guess me, aidanh, and ebering (and of course alex) all worked heavily on the species data patch and didn't notice this problem 11:17:20 <06a​dvil> and then luckily pa was alphabetically after gn! 11:19:16 <06a​dvil> at least it's an my with me muts 11:49:38 03advil02 07* 0.29-a0-533-g200812b14b: fix: pa/me/my fixup fix (real) 10(8 minutes ago, 2 files, 23+ 2-) 13https://github.com/crawl/crawl/commit/200812b14bc1 11:51:15 <10P​leasingFungus> real (serious [final.2]) 11:51:21 <10P​leasingFungus> ty @advil ! 🙂 11:52:01 <06a​dvil> haha yes it seemed too good of an opportunity to add "(real)" to miss 11:53:19 <06a​dvil> that fp error is worrying, it seems to require someone to have max hp 0? 11:53:33 <10P​leasingFungus> if we’re converting all mayflies, we can remove some save compat code for em 11:53:40 <06a​dvil> I didn't actually do that 11:53:43 <10P​leasingFungus> o 11:53:44 <10P​leasingFungus> ok 11:53:53 <10P​leasingFungus> later, then 11:54:01 <06a​dvil> yeah, was just trying to be cautious 11:54:40 <10P​leasingFungus> max hp 0… weird intermediate state during load? 11:54:52 <06a​dvil> maybe 11:55:55 <06a​dvil> the species change code is not exactly unweird 11:56:17 <06a​dvil> well, I can definitely replicate by adding an explicit ASSERT in the obvious place 11:57:20 <06a​dvil> hm, this is not even crashing on the fixup 11:57:50 <10P​leasingFungus> it crashes pre fixup? 11:58:04 <06a​dvil> I think the character I just loaded has undergone the fixup? lost track 11:58:18 <10P​leasingFungus> oop 11:58:34 <06a​dvil> I wonder if this fixup code is getting called before hp is set up?? 12:01:17 <10P​leasingFungus> btw, once we feel like things are stable , we should probably change the fixup code to run earlier & to do less, so all pa bardings don’t pop off 12:05:16 <06a​dvil> that effect may be hard to avoid (but I don't think it should happen to Pa that don't change?) 12:05:44 <10P​leasingFungus> i think it will currently? 12:05:53 <10P​leasingFungus> haven't tested 12:06:02 <10P​leasingFungus> anyway, later 12:06:07 <10P​leasingFungus> crash 1st 12:06:17 <06a​dvil> I've lost track of how the enum goes but it should only happen if change_species_to is called 12:07:21 <06a​dvil> oh, I bet I see why this crash could happen 12:12:09 <10P​leasingFungus> ooh 12:12:29 <10P​leasingFungus> also, change species to will be called for all legacy Pa 12:12:30 <06a​dvil> I think max hp is not truly serialized, it is recalculated on load 12:12:40 <10P​leasingFungus> plausible 12:13:04 <06a​dvil> why? not seeing this in the code 12:13:19 <10P​leasingFungus> hm 12:13:23 <10P​leasingFungus> maybe i’m wrong 12:13:56 <06a​dvil> unless it's from enum reorder? 12:14:01 <10P​leasingFungus> yes 12:14:05 <10P​leasingFungus> that’s what i mean 12:14:24 <10P​leasingFungus> it seems like it’d either be called for all pa before mayflytaurs or all after mayfly and before the fix 12:14:33 <10P​leasingFungus> on mobile atm 12:14:55 <06a​dvil> I guess probably change_species_to doesn't necessarily need to be called at all for these fixups, since everything but the species byte is already correct 12:18:04 <10P​leasingFungus> maybe? i was worried about skills 12:18:13 <10P​leasingFungus> but didn’t read the code in detail 12:19:02 <06a​dvil> well, probably anything that has actually undergone change_species_to will need it 12:19:25 <06a​dvil> gonna need a "(real (real))" commit 12:19:42 <10P​leasingFungus> oh boy 12:19:51 <10P​leasingFungus> sorry about this… 12:19:56 Unstable branch on crawl.kelbi.org updated to: 0.29-a0-533-g200812b14b (34) 12:20:06 <10P​leasingFungus> i should’ve known better than to push that hasty fix 12:20:26 <06a​dvil> but yes, I think you're right, when I loaded up a pre-star Pa it seems to be a My 12:21:50 <10P​leasingFungus> baffling that no one reported it 12:22:39 <10P​leasingFungus> !lm * pa lg:species=mayflytaur 12:22:41 <04C​erebot> 35. [2022-03-28 02:53:09] marklar909 the Warrior (L16 PaFi of Gozag) reached level 4 of the Swamp on turn 40333. (Swamp:4) 12:22:57 <10P​leasingFungus> !lm * pa lg:species=mayflytaur s=gid 12:23:00 <04C​erebot> 35 milestones for * (pa lg:species=mayflytaur): 35x marklar909:cbr2:20220226010838S 12:23:07 <10P​leasingFungus> literally just one person 12:27:18 03advil02 07* 0.29-a0-534-g26aeb8ffec: fix: avoid a potential divide by 0 error 10(89 seconds ago, 2 files, 5+ 1-) 13https://github.com/crawl/crawl/commit/26aeb8ffec9a 12:32:34 Unstable branch on crawl.kelbi.org updated to: 0.29-a0-534-g26aeb8ffec (34) 12:51:27 <06a​dvil> btw, do Me in sprint make any sense? 12:52:44 <06a​dvil> doesn't necessarily seem easy because there's such limited explore possibilities, but it also seems pretty hard to balance 12:53:37 Dj got to roll over MEATSPRINT without it being a problem, I think? 12:53:39 <06a​dvil> maybe lack of explore just doesn't matter without the zot clock though 12:54:00 still matters a bit for regen? 12:54:02 <06a​dvil> I think that's a different case 12:54:08 not sure how much though 12:57:55 <08n​icolae> do meteorans still have damage shaving? i thought that got removed, but their yaml still has a fakemut for "damage resistance" 12:58:01 <06a​dvil> they do 12:58:27 <06a​dvil> see can_shave_damage() in ouch.cc 12:58:44 <08n​icolae> aha 12:59:01 <06a​dvil> oh, hm 12:59:06 <06a​dvil> maybe not? 12:59:19 <06a​dvil> was just looking at the main commit 12:59:28 I thought damage shaving vanished with DD, yeh 12:59:31 <05k​ate> i don't think they're supposed to at least 12:59:39 it was in the original commit and then removed 13:00:03 <06a​dvil> yeah, it got removed in 05f53baf05f6 13:00:47 hm, I really need to teach hexchat to look hashes up in a per-channel commit path :) 13:01:05 <06a​dvil> the meteoran yaml got fixed in that commit too 13:02:34 <06a​dvil> also that commit removes xl15 MUT_REGENERATION which is another thing that was confusing me 😄 13:11:00 <10P​leasingFungus> sorry 13:11:06 <10P​leasingFungus> sloppy squash 13:11:34 <10P​leasingFungus> anyway, yes, no more shaving 13:11:38 <10P​leasingFungus> rip beard stats 13:11:38 <08n​icolae> poor pf, these meteorans are giving him a rough time 13:46:36 IMO species should be allowed to be busted for sprints 13:46:49 Or also very hard 13:47:08 <06a​dvil> yes, I think that's fine, but the central mechanic for Me isn't present in sprints 13:47:22 Well, if it's borderline impossible that's an issue I guess 13:47:35 <06a​dvil> it's not impossible, probably too easy 13:47:51 Oh. 13:47:58 <06a​dvil> there's no zot clock in sprint 13:48:02 Ohhh 13:48:10 I figured there was. I see what you mean. 13:58:17 <10P​leasingFungus> oh, i forgot sprints existed 14:10:57 <08n​icolae> make it so in sprint mayflystars just get drained every XK turns 14:11:13 <06a​dvil> I could imagine it's possible to come up with a clock scheme for sprints? 14:11:31 <06a​dvil> sounds hard to get decently balanced though 14:13:58 <10P​leasingFungus> suspect each sprint would be quite different 14:14:00 <06a​dvil> yeah 14:14:18 <06a​dvil> and for some it may be irrelevant 14:14:29 <10P​leasingFungus> agreed with disabling mays 14:14:40 <06a​dvil> !lg * sprint x=avg(turns) 14:14:42 <04C​erebot> 1029603 games for * (sprint): avg(turn)=763.67 14:15:59 <06a​dvil> !lg * sprint won s=map x=avg(turns) 14:16:00 <04C​erebot> 9096 games for * (sprint won): 3156x sprint_v [4205.58], 1627x linesprint [15214.36], 1293x dungeon_sprint_1 [4622.34], 592x meatsprint [740.36], 543x pitsprint; entrance_room_1 [13784.99], 504x arena_sprint [6656.14], 340x the_violet_keep_of_menkaure; sprint2_stairs_cluster_1 [5605.19], 241x dungeon_sprint_fedhas [8821.53], 191x dungeon_sprint_mu; sprint_mu_entry_cave [14043.16], 180x dungeon_sp... 14:35:48 03advil02 07* 0.29-a0-535-g5014999b08: fix: default `create_enum` to false in species gen 10(8 minutes ago, 43 files, 9+ 130-) 13https://github.com/crawl/crawl/commit/5014999b083b 14:36:21 <10P​leasingFungus> good millikates 14:36:31 <10P​leasingFungus> hadn’t considered modders; good call there 14:36:38 <06a​dvil> except I guess the removed species need their specific tag #ifdefs added back in 14:37:00 <10P​leasingFungus> rip 14:37:38 <09g​ammafunk> rip nicely alphebetized enums 14:38:28 <06a​dvil> at one point aidanh was talking about making some enums serialize in a more robust way 14:39:18 <06a​dvil> or maybe the protobuf save dream is still alive 14:53:14 <10P​leasingFungus> wow, protobuf saves 14:53:20 <10P​leasingFungus> that’d be spicy 14:53:26 <10P​leasingFungus> i’m into it 14:57:24 <10P​leasingFungus> feels like i discussed this with you before, but so many years ago that i didn’t yet know what protobufs were… 14:57:27 04Build failed for 08master @ 5014999b 06https://github.com/crawl/crawl/actions/runs/2423658889 14:58:33 <10P​leasingFungus> hm 15:07:04 Unstable branch on crawl.kelbi.org updated to: 0.29-a0-535-g5014999b08 (34) 15:13:11 <09g​ammafunk> bh was super into protobufs 15:38:40 <06a​dvil> yeah, probably bh 15:38:54 <06a​dvil> I think that build failure is because of missing ifdefs in the species enum 15:41:58 03advil02 07* 0.29-a0-536-ga137e16256: fix: major tag bracketing for removed species 10(55 seconds ago, 1 file, 14+ 0-) 13https://github.com/crawl/crawl/commit/a137e16256f8 16:03:06 04Build failed for 08master @ a137e162 06https://github.com/crawl/crawl/actions/runs/2423999577 16:06:00 Unstable branch on crawl.kelbi.org updated to: 0.29-a0-536-ga137e16256 (34) 16:11:03 <10P​leasingFungus> tag upgrade again 16:11:15 <06a​dvil> I think something never got implemented, but didn't matter for the prev approach 16:12:45 <10P​leasingFungus> interesting 16:15:00 03advil02 07* 0.29-a0-537-g223e3869e7: fix: tag major #ifdefs for aptitudes.h 10(52 seconds ago, 2 files, 4+ 0-) 13https://github.com/crawl/crawl/commit/223e3869e7b6 16:15:07 <06a​dvil> er, #ifs 16:16:06 <10P​leasingFungus> same thing 16:17:57 Unstable branch on crawl.kelbi.org updated to: 0.29-a0-537-g223e3869e7 (34) 16:25:56 <06a​dvil> huh, definitely getting a lot more tag upgrade errors locally than what CI reports 16:29:28 <06a​dvil> I guess it would help to do it correctly 16:37:42 04Build failed for 08master @ 223e3869 06https://github.com/crawl/crawl/actions/runs/2424165200 16:49:57 <06a​dvil> hm, I guess CI doesn't actually try to run the tag upgrade build 17:00:13 03advil02 07* 0.29-a0-538-gcd130a58c2: fix: remove halfling 10(23 minutes ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/cd130a58c270 17:00:13 03advil02 07* 0.29-a0-539-gda7b1fbfba: fix: misc tag upgrade things 10(16 minutes ago, 2 files, 5+ 3-) 13https://github.com/crawl/crawl/commit/da7b1fbfba7d 17:00:13 03advil02 07* 0.29-a0-540-gbfd55560d8: fix: a tag major upgrade crash 10(4 minutes ago, 1 file, 3+ 3-) 13https://github.com/crawl/crawl/commit/bfd55560d858 17:04:32 <10P​leasingFungus> > do think explore_auto_rest = true is possibly not the best default for this species 17:04:47 <10P​leasingFungus> @advil i had the same thought 17:13:29 <06a​dvil> not sure of a good current UI way to sensibly change that though 17:16:27 <06a​dvil> one idea I've had for a while is option setting groups that can be turned on or off as a batch, and layer on top of prev settings 17:16:56 <06a​dvil> it's partly why I've been working off and on to make option parsing more consistent and object-based 17:36:49 Unstable branch on crawl.kelbi.org updated to: 0.29-a0-540-gbfd55560d8 (34) 17:57:00 03advil02 07* 0.29-a0-541-gaf1fc19f1b: fix: tiles umbra/halo interactions 10(69 seconds ago, 1 file, 8+ 3-) 13https://github.com/crawl/crawl/commit/af1fc19f1b75 18:02:43 Unstable branch on crawl.kelbi.org updated to: 0.29-a0-541-gaf1fc19f1b (34) 18:25:41 Unstable branch on underhound.eu updated to: 0.29-a0-541-gaf1fc19f1b (34) 19:19:15 ExiledAnomaly (L13 TrBe) Crash caused by signal #6: Aborted (Lair:5) 19:19:27 CarlosRc (L2 HOMo) Crash caused by signal #6: Aborted (D:1) 19:19:41 Hgcwarrior (L6 MeFi) Crash caused by signal #6: Aborted (D:2) 19:19:52 ExiledAnomaly (L13 TrBe) Crash caused by signal #6: Aborted (Lair:5) 19:19:57 Iggiwashu (L3 MiBe) Crash caused by signal #6: Aborted (D:2) 19:20:07 RenegadeRun (L1 DsNe) Crash caused by signal #6: Aborted (D:1) 19:20:32 cao again overloads during compilation… 19:20:37 Swagmaster123 (L8 DECj) Crash caused by signal #6: Aborted (D:5) 19:20:41 (betting it's linking) 19:20:42 Vhaine (L9 DsBe) Crash caused by signal #6: Aborted (D:7) 19:20:56 CarlosRc (L2 HOFi) Crash caused by signal #6: Aborted (D:1) 19:22:11 New feature of the zot clock 19:23:03 <08n​icolae> in .30 the zot clock formats your hard drive if you hit 0 19:23:50 or worse, deletes your morgue history 19:52:04 Adds metaprogression like modern "roguelikes"; ie, every time you die to Sigmund he gains a hit die. 20:03:01 <10P​leasingFungus> …gains?! 20:06:26 !lg * killer="Sigmund the pandemonium lord" 20:06:27 No games for * (killer='Sigmund the pandemonium lord'). 20:12:42 PF: gains! I didn't say who was metaprogressing :-) 20:18:29 <10P​leasingFungus> good for him. 20:58:41 <03w​heals> heh, that umbra bug was on mantis 3 different times already 21:00:23 <03w​heals> good 8 year old bug 21:01:16 <03w​heals> logging into mantis isn't working for me rn, so if anyone wants to close the tickets they're 8672, 12418, and 12682 21:04:26 it let me log in, not that I have access (nor do I want it, just noting that logins seem to be working) 21:09:39 <03w​heals> i think i just misremember my password 23:14:20 New branch created: fore (1 commit) 13https://github.com/crawl/crawl/tree/fore 23:14:20 03PleasingFungus02 07[fore] * 0.29-a0-530-g2b43de6898: Initial WIP rework tile icons 10(3 months ago, 4 files, 34+ 26-) 13https://github.com/crawl/crawl/commit/2b43de6898c8 23:19:19 04Build failed for 08fore @ 2b43de68 06https://github.com/crawl/crawl/actions/runs/2425799801 23:49:45 -!- Sting1 is now known as Sting