00:20:17 -!- doctrix_aoei is now known as aoei 00:30:56 Fork (bcrawl) on crawl.kelbi.org updated to: 0.23-a0-4616-ga6c816d1e0 00:46:39 Monster database of master branch on crawl.develz.org updated to: 0.29-a0-286-g41199d4a87 04:21:03 Behavioral (L27 KoBr) ASSERT(!monster_at(p) || monster_at(p)->submerged() || fedhas_passthrough(monster_at(p)) || mons_is_player_shadow(*monster_at(p)) || mons_is_wrath_avatar(*monster_at(p))) in 'player.cc' at line 576 failed. (Geh:4) 04:24:28 Experimental (bcrawl) branch on underhound.eu updated to: 0.23-a0-4616-ga6c816d1e0 05:06:50 Unstable branch on crawl.akrasiac.org updated to: 0.29-a0-294-g83ea962 (34) 05:08:03 -!- TAS-2012v is now known as TAS_2012v 05:08:32 <05k​ate> that's an excellent assert 05:10:22 <05k​ate> submerging, fedhas passthrough, wrath avatars, good list of things that nobody would be surprised about causing a crash 05:10:22 <05k​ate> !crashlog 05:10:22 <04C​erebot> 24318. Behavioral, XL27 KoBr, T:92733 (milestone): https://crawl.kelbi.org/crawl/morgue/Behavioral/crash-Behavioral-20220318-112054.txt 05:12:06 <05k​ate> oh wow, is that a creeping inferno getting an attack of opportunity and placing a firestorm fire vortex on the player's new location? 05:18:34 ha, guess I didn't get them all 05:21:03 <05k​ate> I can't test or fix right now but that's my guess anyway, will have a proper look later unless anyone else gets there first 05:23:36 That's very likely what it is considering the other types of crashes I found when I was testing op attacks 05:23:36 <05k​ate> seems reasonable for monsters like those and lurking horrors with self-destruct melee attacks to not get attacks of opportunity either way (since those monsters are usually fast too) 07:07:47 <10P​leasingFungus> yes, agree removing aoops from those monsters is the right 07:07:53 <10P​leasingFungus> er, right fix 07:23:03 <06a​dvil> related: I believe [tested with bats only] that batty monsters can get opp attacks if the stars align, maybe they shouldn't? They sort of already are aoops in a different design package, so it seems a bit much 07:23:03 <06a​dvil> I mean, bats are fine, but shrikes 07:23:03 <10P​leasingFungus> i don’t feel strongly about it either way 07:23:14 <10P​leasingFungus> don’t know there’s anything special about batty monsters vs any other fast monsters 07:23:17 <10P​leasingFungus> wrt aoops 07:23:40 <10P​leasingFungus> neither need em per se, but we don’t want to disincentivize the player from moving faster 07:24:12 <10P​leasingFungus> vaguely inclined against adding more special case rules re batty mons, but could probably be persuaded 07:27:33 <06a​dvil> I agree that special cases are bad, but something with bat speed and ai can already get 3 attacks against you when you step away from it, which goes to 4 with aoops 07:29:48 <06a​dvil> maybe for shrikes this is already so punitive that noone in their right mind would reposition and the extra attack doesn't matter, though 07:45:49 feature request: the LRD targeter should be aware that it can hit things that are behind glass walls 07:53:01 <10P​leasingFungus> advil: hm, i don’t think this is true 07:53:19 <10P​leasingFungus> bat speed and bat ai gives 1 attack per turn, normally 07:53:27 <10P​leasingFungus> not sure where you’re getting 3? 07:53:52 <10P​leasingFungus> probably i’m misunderstanding you 07:54:51 <10P​leasingFungus> i'm confused on a few other points as well: 1) if it's bad for something to get tons of attacks when you step away, surely fast monsters without batty AI, like executioners or zombified shard shrikes, are worse? 07:56:52 <10P​leasingFungus> 2) is it worse for something to go from 3 to 3.333 attacks per turn than to go from 1 to 1.333 attacks per turn? (That is, is it really worse for a fast monster to get aoops than for a normalish one to?) 07:56:52 <06a​dvil> I tested it 07:56:52 <10P​leasingFungus> sounds buggy, unless you're na^chei or something 07:57:04 <10P​leasingFungus> or unless you were in a tight space that didn't let them batty away 07:57:07 Baz (L17 DsGl) Crash caused by signal #6: Aborted (D (Sprint)) 07:57:08 <10P​leasingFungus> batty ai is supposed to screw around for ~2 turns after each attack 07:57:12 <10P​leasingFungus> (2 monster turns) 07:57:13 <06a​dvil> it was in a corridor, yes 07:57:20 <06a​dvil> there are a lot of corridors 07:57:23 <10P​leasingFungus> there are! 07:57:31 <10P​leasingFungus> but that's not really specific to aoops 07:58:44 <06a​dvil> so trying to use a sorites argument as a real argument on someone who studies such things professionally is a good way to annoy them fyi 🙂 07:58:51 <06a​dvil> anyways, I'm stepping out 07:59:09 <10P​leasingFungus> huh? i’m very sorry to annoy you… 07:59:13 <10P​leasingFungus> i wasn’t trying to 07:59:23 <10P​leasingFungus> don’t know sorites 08:04:35 <10P​leasingFungus> oh, googled sorites and think i see the misunderstanding, maybe 08:05:06 <10P​leasingFungus> well, hm, maybe i don’t 08:28:50 03PleasingFungus02 07* 0.29-a0-295-g497ad06: Remove self destructs of opportunity 10(3 minutes ago, 1 file, 2+ 0-) 13https://github.com/crawl/crawl/commit/497ad06e8785 08:32:42 Unstable branch on crawl.kelbi.org updated to: 0.29-a0-295-g497ad06e87 (34) 12:05:09 <06a​dvil> is there a way to override runrest_ignore_message values? setting runrest_stop_message in addition doesn't seem to do it, maybe this is a bug 12:05:30 <06a​dvil> This was asked on reddit re 12:05:33 <06a​dvil> !git 80dc71225541a05e5 12:05:34 <04C​erebot> ebering * 0.28-b1-70-g80dc712: fix: default runrest ignore transformation ending messages (6 weeks ago, 2 files, 18+ 0-) https://github.com/crawl/crawl/commit/80dc71225541 12:14:31 <12e​bering> ^= 12:14:53 <12e​bering> Those two options are a single list option where the first match trips the behavior 12:18:24 <06a​dvil> ah thanks, that does work 12:30:17 03advil02 07* 0.29-a0-296-gda187f2: docs: fix/clarify runrest prepend vs append 10(2 minutes ago, 1 file, 22+ 17-) 13https://github.com/crawl/crawl/commit/da187f2e07d4 12:32:32 Unstable branch on crawl.kelbi.org updated to: 0.29-a0-296-gda187f2e07 (34) 12:47:08 bug report: if a monster follows you when you leave the abyss, it will give XP when you kill it in the main dungeon 12:47:19 providing an extremely slow way to scum the abyss :) 13:05:35 <06a​dvil> lol 13:08:16 trippy catern! i recently left the abyss and an adjacent monster didn't follow me. maybe those screambubbles aren't allowed to exist in other branches 13:09:07 ebering can you remind me, where the logic for spells should be added, if we don't want a monster to use harpoon shot when they can hit with attacks? 13:16:40 <10P​leasingFungus> catern: known, considered not practical to exploit. ty for reporting! 13:18:41 I was thinking of the pudding farming quote… 13:19:12 ha, i was too 13:21:18 what is the pudding farming quote? 13:22:29 "the punishment for pudding farming is pudding farming" 13:38:42 <12e​bering> cebolla: look for a function name spell_to_logic 13:38:42 <12e​bering> and in and around there 13:39:00 <12e​bering> sorry I can't be of more help at the moment, life calls 13:39:34 tidying up a bunch of git history and forcepushin, none of the commits here are new stuff 13:40:26 no worries, not like i hired you to coach me or anything ^_^ 13:44:55 03RojjaCebolla02 {spicyCebolla} 07https://github.com/crawl/crawl/pull/2466 * 0.29-a0-237-gb6eb826: dat/description for starflower 10(2 weeks ago, 1 file, 6+ 0-) 13https://github.com/crawl/crawl/commit/b6eb826e9d39 13:44:55 03RojjaCebolla02 {spicyCebolla} 07https://github.com/crawl/crawl/pull/2466 * 0.29-a0-238-gb8cf055: Try to implement af_reach_disto as a range 3 slap 10(2 weeks ago, 7 files, 23+ 0-) 13https://github.com/crawl/crawl/commit/b8cf0559960e 13:44:55 03RojjaCebolla02 {spicyCebolla} 07https://github.com/crawl/crawl/pull/2466 * 0.29-a0-239-g4d2bc1c: A couple quick starflower tiles 10(2 weeks ago, 4 files, 2+ 1-) 13https://github.com/crawl/crawl/commit/4d2bc1cc7bef 13:44:55 03RojjaCebolla02 {spicyCebolla} 07https://github.com/crawl/crawl/pull/2466 * 0.29-a0-240-ga8d9cf5: buff damage, change attack order 10(13 days ago, 1 file, 3+ 3-) 13https://github.com/crawl/crawl/commit/a8d9cf59cf00 13:44:55 03RojjaCebolla02 {spicyCebolla} 07https://github.com/crawl/crawl/pull/2466 * 0.29-a0-241-gb381af8: Implement many of ebering's improvements 10(11 days ago, 7 files, 16+ 29-) 13https://github.com/crawl/crawl/commit/b381af840d15 13:44:55 03RojjaCebolla02 {spicyCebolla} 07https://github.com/crawl/crawl/pull/2466 * 0.29-a0-242-gae634e8: attempted fix for the reach_three problem 10(11 days ago, 4 files, 9+ 10-) 13https://github.com/crawl/crawl/commit/ae634e86de97 13:44:55 03RojjaCebolla02 {spicyCebolla} 07https://github.com/crawl/crawl/pull/2466 * 0.29-a0-243-g3887a89: a few maps for the new monster 10(11 days ago, 1 file, 138+ 0-) 13https://github.com/crawl/crawl/commit/3887a89f43b3 13:45:29 <12e​bering> I mean, you didn't, but one of the things I've tried to be consistently enthusiastic about is encouraging new pr authors 13:49:30 hmm, could just bum Lodul's thing, _foe_not_nearby 13:52:07 at range 3 there'd be both tentacle slaps and occasional casts but it'd be an improvement 13:52:08 oh, but/and, harpoon shot already has a "not adjacent" thing. so maybe that could check the monster's melee reach instead of being strictly about adjacentness 13:53:00 with a possible side benefit of harold not harpooning if he happens to have a polearm 15:45:30 there's both monster::reach_range and monster_info::reach_range 😲 16:23:55 Unstable branch on underhound.eu updated to: 0.29-a0-296-gda187f2e07 (34) 16:45:33 <10P​leasingFungus> yes 16:45:33 <10P​leasingFungus> monster_info is for player visible info 16:45:33 <10P​leasingFungus> monster is for the underlying game info 16:45:33 <10P​leasingFungus> the difference has mattered less and less over the years 16:45:33 <10P​leasingFungus> in the old days, it could matter for stuff like un-ID'd equipment that a monster had, or which random spellbook they had 16:48:11 ah, that makes sense 16:49:43 <10P​leasingFungus> these days it's mostly stuff like... actual HP/MHP, actual XP value maybe? 16:49:55 <10P​leasingFungus> @??small abomination 16:49:56 <04C​erebot> small abomination (x) | Spd: 7-15 | HD: 7 | HP: 22-31 | AC/EV: 6/9 | Dam: 23 | undead, evil, unbreathing | Res: will(40), poison+++, drown, miasma, neg+++, torm | Vul: holy, silver | XP: 131 | Sz: Large | Int: brainless. 16:50:07 <10P​leasingFungus> maybe weirdness about their speed, i forget if that's hidden 16:50:18 <10P​leasingFungus> (did you know abominations have random speed?! crawl trivia for you) 16:51:41 03spicyCebolla02 07https://github.com/crawl/crawl/pull/2466 * 0.29-a0-244-ge1c239a: Add reach_three to mon-info.cc 10(53 minutes ago, 1 file, 3+ 1-) 13https://github.com/crawl/crawl/commit/e1c239ac31cb 16:58:52 something like this might work maybe? https://gist.github.com/RojjaCebolla/425ab05c85a20d817d7d17f603f18be6 16:58:52 for making harpoon only fire when enemies aren't within reach 16:58:52 i'm still very fuzzy on how you pass monsters into functions, like whether a &foo needs a *foo to be given to it later 16:59:44 monster &foo gets foo, monster *foo gets &foo 17:00:07 (unless the foo you pass in is already a pointer, then it's just foo) 17:03:18 https://en.cppreference.com/w/cpp/language/reference vs. pointers 19:39:07 maplemancer (L24 OgHu) ASSERT(range >= 0) in 'beam.cc' at line 669 failed. (beam '', source 'player', item 'none'; has range -1) (Lair:2) 19:39:07 maplemancer (L24 OgHu) ASSERT(range >= 0) in 'beam.cc' at line 669 failed. (beam '', source 'player', item 'none'; has range -1) (Lair:2)