00:35:29 Autofight incorrectly fails with "No reachable target in view!" 13https://crawl.develz.org/mantis/view.php?id=12576 by Implojin 01:34:57 Unstable branch on crawl.develz.org updated to: 0.27-a0-1084-g78b3836a5f (34) 01:55:51 Windows builds of master branch on crawl.develz.org updated to: 0.27-a0-1084-g78b3836a5f 02:53:51 Monster database of master branch on crawl.develz.org updated to: 0.27-a0-1084-g78b3836a5f 02:53:55 Unstable branch on cbro.berotato.org updated to: 0.27-a0-1084-g78b3836a5f (34) 03:30:51 Fork (bcrawl) on crawl.kelbi.org updated to: 0.23-a0-4135-g7666618ddd 03:32:17 Fork (bcadrencrawl) on crawl.kelbi.org updated to: 0.03-260-g40356c7de1 04:05:57 03kate-02 07* 0.27-a0-1085-gf0060e8: Make Repel Missiles unablatable 10(11 days ago, 6 files, 1+ 21-) 13https://github.com/crawl/crawl/commit/f0060e8222e2 06:14:33 Unstable branch on crawl.kelbi.org updated to: 0.27-a0-1085-gf0060e8222 (34) 10:12:19 <06a​dvil> Implojin: I think the short form answer is that it basically uses the OS conventions which are what you're seeing there (just may not be what you expect) 10:13:10 <06a​dvil> on os x, which has a more modern convention (app settings in ~/Library), that is what is done 10:15:04 <06a​dvil> perhaps it should do something by default like create the blank rc file with all the comments in ~/.crawl though 10:24:19 <06a​dvil> oof I know other people have been enthusiastic about this but i18n looks no less brittle than our previous localization stuff, we will never be able to maintain this 10:27:49 <06a​dvil> not sure I'm the right person to comment on this because I haven't really thought deeply about it or engaged with this contributor (where iirc someone else had), but on first glance I have reservations about post-merge viability 10:28:01 <10P​leasingFungus> i hadn’t gotten a chance to look at the implementation yet 10:28:21 <10P​leasingFungus> i expressed a bunch of concerns when they first expressed interest 10:28:35 <10P​leasingFungus> but i don’t think i managed to communicate them effectively 10:29:54 <06a​dvil> implementation looks ok-ish (I don't know the spectrum of what's possible for localization though), it's mostly wrapping strings in _() which presumably is a way of keeping the substitution out of the code 10:30:02 <06a​dvil> plus separate files with the translations 10:31:13 <06a​dvil> I wonder if we should consider opening up a github discussions board to channel FRs out of the issue list. Although maybe we sorta don't want FRs on github at all 10:33:42 <05k​ate> i don't think we should, they just clogged things up on mantis until we stopped taking FRs there and i don't think they'd be any better on github 10:33:42 <06a​dvil> well, I ask because we're getting them anyways 10:33:44 <06a​dvil> a discussion thing would at least move them somewhere else 10:34:45 <06a​dvil> I guess the issue template doesn't explicitly forbid them, you have to delete it all though 10:34:50 <06a​dvil> has anyone been able to watch the ttyrec for shummie's hep thing in https://github.com/crawl/crawl/issues/1757 ? It's cko so the ts files weren't working for me, I guess it would need to be watched offline 10:37:42 <06a​dvil> by discussion thing I mean something like this https://github.com/Homebrew/discussions/discussions , which is entirely separate from issues/PRs, it's a relatively new github feature 10:38:18 <06a​dvil> anyways, I'm not sure it's really a good idea, just a thought 10:42:26 <05k​ate> i'd probably tend towards just not having FRs on github generally (and i guess closing the existing ones and maybe noting somewhere more explicitly that the issues page is just for bug reports) 10:43:32 <05k​ate> the tavern was a good place to channel them into while it was more active i guess 10:45:26 <12e​bering> In terms of FRs: I think that experience of the project has shown that it is rare for even the best designed and written gameplay FRs to lead to implementation absent an implementer. Brainstorming is fun but it drowns out more brass tacks discussion and attracts less well thought out things. 10:46:15 <12e​bering> So I’d be in favor of keeping non pull request gameplay changes out of our repo and adjacent GitHub parts. 10:47:18 <12e​bering> User interface/usability improvements are a different story and I’m less certain about where they should be 10:47:24 <06a​dvil> most of the open FR issues are basically (or entirely) UI ones 10:47:29 <12e​bering> Since there’s much less subjectivity there 10:48:16 <12e​bering> Comes down to: Yes this does improve things and we’d like the patch; yes that’s be better but the crawlcode cost is too high; and no that doesn’t improve the UI 10:51:18 <06a​dvil> hm, probably the best solution would be to have a new tavern with the corresponding subforums there 10:51:24 <06a​dvil> I suspect most reasonable UI FRs can probably be phrased as a bug of some kind 10:53:38 <06a​dvil> man does my M1 build crawl rapidly 10:54:41 Man, does my 2008-ish MacBook... not 11:17:59 <06a​dvil> hm, I'm not sure using passable for transporter vaults is working as intended 11:20:36 <06a​dvil> maybe this is specifically about gammafunk_its_a_trap, I think this should only be marked passable in vaults 11:20:54 <06a​dvil> because otherwise it is potentially one-way 11:23:10 <05k​ate> oh hm, i think that's the one i got as a disconnected snake vault 11:23:22 <05k​ate> i didn't look into it much further though, i don't know how the tags work really 11:23:23 <06a​dvil> yeah, there's an open issue about it that I have replicated 11:23:50 <06a​dvil> seems that tagging it passable in some cases is preventing the stair check from working correctly, if I remove that tag it vetos 11:30:54 03advil02 07* 0.27-a0-1086-g3a4c58a: fix: adjust connectivity for gammafunk_its_a_trap 10(6 minutes ago, 1 file, 4+ 1-) 13https://github.com/crawl/crawl/commit/3a4c58a182d1 11:42:04 Guys, what will happen after I sacrifice 10% hp for Ru on DS that is bound to get robust mutation? 11:46:43 Unstable branch on crawl.kelbi.org updated to: 0.27-a0-1086-g3a4c58a182 (34) 11:58:17 <08w​ormsofcan> ru shouldn't be offering frail if robust is scheduled 11:58:59 <08w​ormsofcan> see: https://github.com/crawl/crawl/blob/master/crawl-ref/source/god-abil.cc#L3739 11:59:04 Oh, good. Same for love and demonic guardian, I guess? 11:59:18 <08w​ormsofcan> no, in that case your mutation gets suppressed 12:00:55 Yeah, I see. Thanks! 12:03:00 <06a​dvil> I wonder if the behavior should be more uniform 12:08:32 <06a​dvil> !vault onia_ninara_dug_in_and_dangerous 12:08:33 <04C​erebot> Can't find onia_ninara_dug_in_and_dangerous. 12:08:46 <06a​dvil> !vault mini_monsters.des 12:08:46 <04C​erebot> https://github.com/crawl/crawl/blob/master/crawl-ref/source/dat/des/variable/mini_monsters.des 12:10:38 <06a​dvil> someone on reddit reported that this vault can be a tele closet, which seems to be true 12:12:54 <06a​dvil> it can generate digging/tele, but it seems that it mostly doesn't, and on only one side of the vault at that 12:13:35 <06a​dvil> though perhaps it always generates monsters in every square on one side? 12:16:45 <12e​bering> looks like the shuffles are supposed to give it some hatches? 12:17:04 <12e​bering> but only some of the time 12:25:19 <05k​ate> it looks like the non-monster side should always have digging, teleport, or a hatch? 12:25:54 <05k​ate> but yeah, it should just be no_tele_into anyway, since i think shafting into a place where a monster is will displace it? 13:29:49 Unstable branch on crawl.akrasiac.org updated to: 0.27-a0-1086-g3a4c58a (34) 15:31:12 Aura of silence visual effect appears only a turn after manifestation. 13https://crawl.develz.org/mantis/view.php?id=12577 by Yermak 15:43:45 <05k​ate> nice, that's an easy fix but i also discovered that if it were somehow possible to lose the silence aura mut after gaining it, you just leave the silence bubble behind wherever you were 15:44:33 that's a pretty neat bug 15:44:54 now if you could fling that silence donut onto, say, a sleeping unique! 16:01:34 03kate-02 07* 0.27-a0-1087-gf39c287: Redraw agrid on gaining the silence aura mut (#12577) 10(15 minutes ago, 1 file, 9+ 0-) 13https://github.com/crawl/crawl/commit/f39c28767454 16:17:15 Unstable branch on crawl.kelbi.org updated to: 0.27-a0-1087-gf39c287674 (34) 16:42:27 Hep ancestor gets damaged by irradiate 13https://crawl.develz.org/mantis/view.php?id=12578 by Lici_the_Crawler 17:49:06 agrid code is weird 18:02:26 03advil02 07* 0.27-a0-1088-ga8074f5: fix: prevent RegenMP with -cast 10(58 seconds ago, 1 file, 5+ 2-) 13https://github.com/crawl/crawl/commit/a8074f570fc8 18:14:51 <05k​ate> hmm, i guess regenmp is actually the more useful of the properties on the amulet, really? if anything it's the +int that's closest to actual uselessness 18:14:51 <05k​ate> i could see preventing either from combining with -cast though 18:17:16 Unstable branch on crawl.kelbi.org updated to: 0.27-a0-1088-ga8074f570f (34) 18:17:44 yeah, possibly -cast with int makes no sense either 18:18:51 <05k​ate> we do allow eg -cast and +mp, which seems comparable to -cast and regenmp 18:19:05 we do 18:19:23 I think 18:19:35 <05k​ate> oh hmm, or actually that might be prevented elsewhere 18:19:42 oh, do you know where? 18:19:46 not much is disallowed there 18:20:44 <05k​ate> !source _randart_is_conflicting 18:20:44 <04C​erebot> 1/1. https://github.com/crawl/crawl/blob/master/crawl-ref/source/artefact.cc#L1509 18:20:44 ahh 18:20:44 <05k​ate> but that's only checking ring subtype, i think you could still roll +mp and -cast on a different item type 18:21:37 yeah, hm I wonder if my commit above should have been there 18:21:43 they seem similar 18:21:57 <05k​ate> i remember trying to make sense of all this recently, it's a big mess with lots of duplication and i think some checks happening in slightly different places for different reasons 18:22:47 hm maybe _randart_is_conflicting is used when assessing a complete randart, and _artp_can_go_on_item is used when building the item in the first place 18:22:50 <05k​ate> like, _randart_is_conflicting is checked after generation and rerolls the whole artefact if it fails, whereas _artp_can_go_on_item happens during generation 18:22:53 <05k​ate> yeah 18:23:18 Unstable branch on underhound.eu updated to: 0.27-a0-1087-gf39c287674 (34) 18:23:18 oh, probably this is all in service of xom making bad randarts or something 18:24:47 !blame xom 18:24:47 I pronounce xom... Guilty! 18:26:36 well, that was a nice theory but maybe not 19:28:07 03advil02 07* 0.27-a0-1089-g09c2292: fix: prevent -cast and +int (kate) 10(2 minutes ago, 1 file, 6+ 3-) 13https://github.com/crawl/crawl/commit/09c2292e3450 19:31:59 Unstable branch on crawl.kelbi.org updated to: 0.27-a0-1089-g09c2292e34 (34) 20:06:09 <13S​astreii> i'm working on some of the older uniques, like Frederick and Rupert. same color palette for a better recognition but more personality. i wanted Rupert to be more muscular because berserk and hits like a truck 20:06:09 <13S​astreii> https://cdn.discordapp.com/attachments/747522859361894521/836392738756755456/Schermata_2021-04-27_alle_00.53.45.png 20:06:15 <13S​astreii> https://cdn.discordapp.com/attachments/747522859361894521/836392766480318474/frederick.png 20:06:16 <13S​astreii> https://cdn.discordapp.com/attachments/747522859361894521/836392768448233482/Rupert.png 20:26:01 <10P​leasingFungus> wow! 20:47:18 03PleasingFungus02 07* 0.27-a0-1090-gb115f17: Truly unique new tiles (Sastreii) 10(32 seconds ago, 3 files, 4+ 1-) 13https://github.com/crawl/crawl/commit/b115f176c67e 20:53:52 <10P​leasingFungus> hm 20:53:52 <10P​leasingFungus> sort of thinking about making that 'highlight dangerous enemies' setting on by default (for nasties) 20:55:35 <|amethyst> @PleasingFungus needs more advpng 20:55:45 <|amethyst> @PleasingFungus 17711 644 3% rltiles/mon/unique/frederick.png 20:56:11 <10P​leasingFungus> lol 21:02:20 Unstable branch on crawl.kelbi.org updated to: 0.27-a0-1090-gb115f176c6 (34) 23:32:15 -!- krator44-- is now known as krator44