00:59:46 -!- indigaz25 is now known as indigaz2 01:35:22 Pinkbeast, everything! 01:35:25 Tiles, for example. 03:32:48 Experimental (bcrawl) branch on underhound.eu updated to: 0.23-a0-5261-gd9800d219b 03:45:36 <03implojin> Lore: Our contribution process documentation now has links to hopefully make "how-to artwork submissions" a bit clearer: https://github.com/crawl/crawl/blob/master/crawl-ref/docs/develop/contribution-process.md 03:46:34 Holy smokes! Thank you very much, dear everyone! 03:46:36 I see! 03:46:40 <03implojin> in short, you can link your tiles here and our tiles devs will probably give feedback eventually, or you can open a github issue if you want something a little more structured, or you can open a github PR if you're comfortable enough with git and C++ to be able to add them to the game yourself 03:58:14 You do such a miracle... Thank you again for the clarification in both the Readme and here, and the highlight in chat, @implojin ! 03:58:59 <03implojin> you're welcome 05:35:12 Unstable branch on crawl.akrasiac.org updated to: 0.34-a0-786-ge5b59a6 (34) 06:05:46 03SentientSupper02 07https://github.com/crawl/crawl/pull/4751 * 0.34-a0-771-g547119537d: Silence a freeze fail message when tabcasting 10(16 seconds ago, 1 file, 2+ 1-) 13https://github.com/crawl/crawl/commit/547119537d1c 08:53:14 03regret-index02 07* 0.34-a0-787-ga5a8217afb: Changelog updates through e5b59a6 10(67 seconds ago, 1 file, 23+ 7-) 13https://github.com/crawl/crawl/commit/a5a8217afb0f 08:53:23 HilariousDeathArtist (L27 DrHs) ASSERT(!cell_is_solid(ctarget)) in 'cloud.cc' at line 800 failed. (Tar:1) 09:53:48 03dolorous02 07* 0.34-a0-788-g53471ace1b: Fix typo and missing word. 10(77 seconds ago, 1 file, 2+ 2-) 13https://github.com/crawl/crawl/commit/53471ace1bfd 09:55:20 Moober (L27 MDFi) ASSERT(!cell_is_solid(ctarget)) in 'cloud.cc' at line 800 failed. (Tar:2) 09:56:37 Moober (L27 MDFi) ASSERT(!cell_is_solid(ctarget)) in 'cloud.cc' at line 800 failed. (Tar:7) 10:02:18 03dolorous02 07* 0.34-a0-789-g1517b53afc: Fix wording and punctuation. 10(51 seconds ago, 1 file, 3+ 3-) 13https://github.com/crawl/crawl/commit/1517b53afcc5 12:01:10 Lak (L19 BaWr) ASSERT(!cell_is_solid(ctarget)) in 'cloud.cc' at line 800 failed. (Depths:2) 12:07:59 03dolorous02 07* 0.34-a0-790-g8624165a17: Use proper pronoun in sticky flame msg (#4790). 10(3 minutes ago, 1 file, 3+ 1-) 13https://github.com/crawl/crawl/commit/8624165a1727 12:10:07 03dolorous02 07[stone_soup-0.33] * 0.33.1-3-g3ed96d4bfa: Use proper pronoun in sticky flame msg (#4790). 10(5 minutes ago, 1 file, 3+ 1-) 13https://github.com/crawl/crawl/commit/3ed96d4bfa38 12:25:02 03dolorous02 07* 0.34-a0-791-g04a43924b8: Fix verb conjugation in sticky flame msg (oops). 10(11 minutes ago, 1 file, 4+ 3-) 13https://github.com/crawl/crawl/commit/04a43924b81f 12:26:07 03dolorous02 07[stone_soup-0.33] * 0.33.1-4-g5a2043f66b: Fix verb conjugation in sticky flame msg (oops). 10(12 minutes ago, 1 file, 4+ 3-) 13https://github.com/crawl/crawl/commit/5a2043f66b70 12:39:09 04Build failed for 08stone_soup-0.33 @ 3ed96d4b 06https://github.com/crawl/crawl/actions/runs/17743871386 12:41:03 04Build failed for 08stone_soup-0.33 @ 5a2043f6 06https://github.com/crawl/crawl/actions/runs/17744262239 12:47:28 03WizardIke02 {dolorous} 07* 0.34-a0-792-g6fb57b8551: Fix tile mismatch on monster lookup menu 10(6 days ago, 1 file, 1+ 0-) 13https://github.com/crawl/crawl/commit/6fb57b8551ec 12:57:08 <04dracoomega> So, that crash is from trying to place smoke when oblivion howl summons an undertaker in a wall. And that's easy enough to fix, but I find myself wondering if this assert is useful compared to just... not placing a cloud if asked to put one in a wall. We already silently don't do anything if we try to place a cloud out of bounds. And instead we currently have dozens of different callers manually checking that a cell isn't solid 12:57:08 before trying to place a cloud. 13:01:30 <04dracoomega> I sort of feel like, in general, if a call can be handled as a semantically valid do-nothing (without putting other things in an invalid state) and is a common enough situation to regularly require the caller check the thing themselves, then probably it should just do nothing instead of assert. 14:34:57 03DracoOmega02 07* 0.34-a0-793-ga93910155b: Refactor place_cloud to do its own validity checking, fix inconsistencies 10(81 seconds ago, 26 files, 68+ 156-) 13https://github.com/crawl/crawl/commit/a93910155b73 14:35:09 -!- MakMorn_ is now known as MakMorn 14:35:53 :thumbs_up: 14:36:20 hm, that should have worked 14:45:43 <09hellmonk> fr: update the description of SPARM_REPULSION to better indicate that rmsl also works on spells 14:47:39 <04dracoomega> I mean, it did produce a thumbs-up on this end, if that's what you mean 14:47:59 <04dracoomega> And yes, that is probably fair 14:48:07 <04dracoomega> 'Missiles' is easy to misinterpret 14:48:55 <09hellmonk> it came up in the main channel yesterday, I've only seen one person confused by it but it does seem misleading 14:49:17 <09hellmonk> I do think we use "ranged attack" to specifically mean ranged weapon/throwing in some other place 14:49:36 <04dracoomega> I mean, it doesn't say 'ranged attack' at the moment. It says 'missiles' 14:49:42 <09hellmonk> oh right 14:49:51 <04dracoomega> However, what about something like "it helps its wearer evade projectiles - both physical and magical." 14:50:06 <09hellmonk> yeah, that seems good 14:50:34 <09hellmonk> I believe it works against all spells that are actually dodgeable, right? 14:50:55 <09hellmonk> so no need for weird nuances about things that are bolts/beams 14:51:09 <04dracoomega> Yes 14:51:27 <04dracoomega> (The old version handled penetrating/non-penetrating differently, but that hasn't been the case in a while) 14:51:43 <09hellmonk> does the game say how much EV it's worth anywhere 14:51:54 <09hellmonk> on the player facing side I mean 14:52:53 <04dracoomega> No, actually. Should it? 14:53:11 <09hellmonk> might be useful to indicate the effect size 14:53:27 <04dracoomega> Probably, I suppose. (The old version was impossible to quantify, of course, and I didn't change the wording at the time) 14:53:30 <09hellmonk> since unlike acrobat it isn't in the interface 14:54:30 <09hellmonk> oh yeah, I guess the rolled to-hit reduction version would not really have made sense to give a number for 14:55:34 <04dracoomega> Was trying to see if I could squeeze the description into one line, but probably impossible if stating the EV out loud. Oh well, being unambiguous is more important, if push comes to shove 14:57:17 <04dracoomega> Okay "it helps its wearer evade all forms of projectile (EV +15)." does fit in one line. Not quite as explicit about magical, but do you think it would still be adequate? 14:57:38 <09hellmonk> hmm, I'm not sure 15:00:03 <09hellmonk> we do use 'projectile' this way in the status description for monsters with reflection but that seems to be the only place. Shields skill description for example specifies that both magical and nonmagical projectiles are affected 15:00:55 <04dracoomega> Oh, huh. So it does. 15:01:22 <09hellmonk> most other places, like the jelly missile mutation, use "projectile" to mean "ranged weapon/throwable" 15:02:12 <04dracoomega> Oh dear. (I was about to say "Is there anything that cares about ranged attacks versus spell projectiles?" but I guess there it is) 15:02:13 <09hellmonk> though we're a bit inconsistent about this, we also call magic dart a projectile in the description of conjurations skill (mdart irrelevant to this discussion ofc) 15:02:30 <04dracoomega> I wonder if that's the only thing? 15:02:37 <09hellmonk> oh, wrong place that's actually in develop spells doc 15:02:42 <09hellmonk> so not a skill desc 15:03:59 <04dracoomega> (Also, the jelly mutation says 'missile', not 'projectile' for whatever that is worth) 15:04:07 <04dracoomega> Though who knows 15:05:03 <09hellmonk> "You have a small jelly attached to you that may absorb projectiles." 15:05:07 <09hellmonk> is the mutation description 15:05:14 <09hellmonk> unless I am reading from the wrong place 15:05:38 <06regret-⸸nde※> A-screen mutation lines are different from examining the individual mutations on said screen. 15:05:53 <06regret-⸸nde※> (Repel missiles is already described as deflects incoming projectiles over in Qazlal powers, for whatever that's worth.) 15:09:45 <04dracoomega> Oh, yeah, I only read the long description of the mutation 15:09:51 <04dracoomega> So apparently it doesn't even use the same wording as itself ^^; 15:10:17 <04dracoomega> But the message for gaining/losing it says 'repel missiles' 15:10:45 <04dracoomega> I mean, this was all easier when there was a 'repel missiles' spell that everyone knew about, of course 😛 15:12:54 <04dracoomega> Anyway, I'll just go for somewhat longer and less ambiguous and not worry about it too much 15:13:16 <04dracoomega> I do wish it were possible to use consistent indenting for ego/artprop descriptions in item popups, though. That would help readability so much T.T 15:13:18 <09hellmonk> I'll defer to what you think is best, it would be nice for all of our text descriptions to be perfectly consistent but that seems difficult 15:14:10 <04dracoomega> I actually started to take a stab at that once, but discovered that the problem is, essentially, that by the time you're at the part of the display code that knows how wide a text area you have, the entire item description is already fused into one semantically undifferentiable string 15:14:53 <04dracoomega> So it would probably take some major refactoring to do a simple column split 15:17:51 03DracoOmega02 07* 0.34-a0-794-gadfededa23: Tweak wording of scarves of repulsion (hellmonk) 10(27 minutes ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/adfededa2328 15:41:43 Unstable branch on underhound.eu updated to: 0.34-a0-793-ga93910155b (34) 15:42:48 <06pleasingfungus> re: cloud placement in walls: seems reasonable to not place clouds in that case, yeah. in application code, i'd prefer to crash, but for game code, one prefers to prioritize player experience 15:43:40 <04dracoomega> Well, it turns out that we did manual checking in lots and lots of places already, and wall monsters violate previous assumptions about 'a space a monster was is always a safe space for a cloud to be' and would have (it turns out) called for many many more checks all over the place 15:44:13 <04dracoomega> So I think it's fine for the call to perform its own validity checking and return whether it actually placed a cloud or not. We do this already for a bunch of things, including monsters, after all. 16:13:00 https://imgur.com/a/Aq7SSCs version 4 16:23:46 <02Monkooky> does anything refer to a newly created cloud? 17:28:56 03dolorous02 07* 0.34-a0-795-g5fa5d96457: Fix typo. 10(2 minutes ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/5fa5d964572e 17:37:45 <04dracoomega> Not as far as I know. (place_cloud didn't even return anything, so you'd have to look up the cloud you just placed anyway. I am reasonably sure this is never done.) 18:17:57 <06mumra> Ah, nice work getting wall monsters in 18:18:15 <06mumra> Hadn't seen yesterday's activity and was pleasantly surprised to encounter a rock fish in the wild 18:19:18 <06mumra> Hmm, i think their pathfinding is bugged 18:20:00 <04dracoomega> Er, howso? 18:20:08 <06mumra> in open space, one refused to pathfind around another 18:22:28 <06mumra> ..... .FF@. ...x. the map looked like above. but i took multiple turns attacking the fish to the left of me, and the fish behind it didn't move 18:24:35 <04dracoomega> Are you sure there couldn't have been something else going on? (At the very least, I spent a lot of time being chased back and forth by rock fish and never noticed that sort of weirdness. I don't think there would be any rock fish-specific code that would affect them on completely open ground either) 18:25:32 <04dracoomega> I made some minor adjustments to your 'wall-following' movement code to make them less likely to take long detours through it, but I don't think that is touched without there being adjacent walls 18:27:27 <06mumra> potentially something else yeah, but it was quite a lot of turns it was sat there. i'm thinking the wall below me was somehow affecting it (but will play around in wizmode when i'm at my other computer to see if i can reproduce) 18:31:07 <06mumra> if anything else was going on i can't identify it. certainly nothing apparent in the messages, so whether it's specific to wall monsters or some general pathfinding glitch there was definitely something not right 18:32:02 <04dracoomega> Can you reproduce this easily from that configuration? 18:33:23 <04dracoomega> (I can't. They move like one would expect) 18:35:28 -!- adept76 is now known as adept7 18:37:26 <06mumra> maybe there's a more complex setup based on the whole map layout 18:42:06 <04dracoomega> At the risk of asking a silly question, there isn't any chance you were worshipping Gozag, is there? >.> 18:49:14 <06mumra> no this was pre-temple (there was definitely no status icon over them anyway, and nothing in the message log to indicate distraction) 18:50:22 <06mumra> oh you made me realise, i forgot to actually take up a religion even when i did get to temple 😂 18:56:34 i can't message any artists on discord because i don't know their username 19:29:10 03Darby02 07* 0.34-a0-796-g05f6c35849: De-spoiler throwable mulch rates (Ge0ff) 10(16 minutes ago, 4 files, 20+ 19-) 13https://github.com/crawl/crawl/commit/05f6c3584925 19:29:10 03Darby02 07* 0.34-a0-797-ge184829fb3: Double disjunction dart mulch rate (DracoOmega) 10(2 minutes ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/e184829fb3dd 21:21:56 03dolorous02 07* 0.34-a0-798-gbc4ca33cb1: Fix wording. 10(71 seconds ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/bc4ca33cb1e8 22:35:48 Unstable branch on crawl.develz.org updated to: 0.34-a0-798-gbc4ca33cb1 (34) 22:58:43 Windows builds of master branch on crawl.develz.org updated to: 0.34-a0-798-gbc4ca33cb1 23:10:07 Stable (0.33) branch on underhound.eu updated to: 0.33.1-4-g5a2043f66b 23:31:38 Unstable branch on cbro.berotato.org updated to: 0.34-a0-798-gbc4ca33cb1 (34) 23:55:46 Monster database of master branch on crawl.develz.org updated to: 0.34-a0-798-gbc4ca33cb1