02:06:10 New branch created: pull/3764 (1 commit) 13https://github.com/crawl/crawl/pull/3764 02:06:11 03Bernardo Sulzbach02 07https://github.com/crawl/crawl/pull/3764 * 0.32-a0-934-gc850aae4ae: Reuse monster_iterator::advance in operator++ 10(16 minutes ago, 1 file, 1+ 3-) 13https://github.com/crawl/crawl/commit/c850aae4aefe 03:02:39 03NormalPerson702 07https://github.com/crawl/crawl/pull/3760 * 0.32-a0-922-g256e25cf8d: Fix wanderer evoker starts from allowing duplicate evokers 10(3 days ago, 3 files, 19+ 9-) 13https://github.com/crawl/crawl/commit/256e25cf8df3 03:31:43 Experimental (bcrawl) branch on underhound.eu updated to: 0.23-a0-5199-gb287095a7e 04:46:23 <05icoson> where does the des cache go when building in repo on linux? 04:46:38 <05icoson> by default at least 04:53:10 <05icoson> maybe this question amounts to where the save dir is supposed to be, which I think should be ~/.crawl 05:07:18 Unstable branch on crawl.akrasiac.org updated to: 0.32-a0-933-ga8fa363 (34) 05:10:11 New branch created: pull/3765 (1 commit) 13https://github.com/crawl/crawl/pull/3765 05:10:12 03LKH02 {GitHub} 07https://github.com/crawl/crawl/pull/3765 * 0.32-a0-36-g3903449e77: Update gods.txt 10(4 minutes ago, 1 file, 8+ 12-) 13https://github.com/crawl/crawl/commit/3903449e7720 05:13:03 <03wheals> the des cache goes to $PWD/saves for me 05:13:17 <03wheals> or rather $PWD/saves/des 05:14:08 <03wheals> but any rate, by default when building in repo ~/.crawl doesn't get touched at all 05:24:01 03LKH02 {GitHub} 07https://github.com/crawl/crawl/pull/3765 * 0.32-a0-37-g70d5bbf1e4: Update backgrounds.txt 10(29 seconds ago, 1 file, 7+ 8-) 13https://github.com/crawl/crawl/commit/70d5bbf1e464 05:34:17 <05icoson> ah thanks, I guess what I was seeing is in the scope of LINUXDEPLOY which is maybe only for appimage builds or the like 05:55:58 03dolorous02 07https://github.com/crawl/crawl/pull/3762 * 0.32-a0-933-g76a05429fb: Allow wanderers with shapeshifting skill. 10(15 hours ago, 1 file, 24+ 3-) 13https://github.com/crawl/crawl/commit/76a05429fbc9 06:51:14 03NormalPerson702 {dolorous} 07* 0.32-a0-934-g5a1759da3d: Fix wanderer evoker starts from allowing duplicate evokers 10(3 days ago, 3 files, 19+ 9-) 13https://github.com/crawl/crawl/commit/5a1759da3d27 06:51:14 03dolorous02 07* 0.32-a0-935-g6cb2a80f7e: Allow wanderers with shapeshifting skill. 10(16 hours ago, 1 file, 24+ 3-) 13https://github.com/crawl/crawl/commit/6cb2a80f7e58 07:36:59 New branch created: pull/3766 (1 commit) 13https://github.com/crawl/crawl/pull/3766 07:37:00 03NormalPerson702 07https://github.com/crawl/crawl/pull/3766 * 0.32-a0-934-g3269000313: Add Permafrost Eruption as a monster spell for servitors/ghosts 10(16 minutes ago, 5 files, 62+ 16-) 13https://github.com/crawl/crawl/commit/326900031396 08:01:30 03dolorous02 07* 0.32-a0-936-gade3313668: Remove remnants of stationary boulders. 10(21 minutes ago, 2 files, 3+ 5-) 13https://github.com/crawl/crawl/commit/ade3313668f3 08:34:47 Gell's Gravitambourine: https://tavern.dcss.io/uploads/default/original/1X/87ba20974aa5f8616cbb17e33900f2411b872ed6.png 08:35:04 (tile art I made) 08:35:54 Gell's Gavotte, cleaned up a bit from the last version I showed that I said I wasn't quite sure about yet, I'm happy with this now: https://tavern.dcss.io/uploads/default/original/1X/7646aa78a28d0239cf084085ec06c93183e05e20.png 09:46:46 03dolorous02 07* 0.32-a0-937-gd7206a2022: Fix wording. 10(2 minutes ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/d7206a202240 11:14:04 <04dracoomega> I dislike a lot of code in directn.cc >.> 11:15:09 <06pleasingfungus> wow, same 11:15:15 <04dracoomega> (I also still don't understand some of it very well. I've put in a rather silly amount of time trying to make an incredibly simple-to-describe thing work) 11:38:32 03dolorous02 07* 0.32-a0-938-g40ac7a0f4d: Clarify foul flame brand description. 10(4 minutes ago, 1 file, 4+ 4-) 13https://github.com/crawl/crawl/commit/40ac7a0f4d24 11:58:58 <04dracoomega> Unpleasant discoveries: _find_square is trying to aim at like ~900 different spaces the moment I try to target a spell. Further investigation shows that it considers the valid aiming radius to be 38. This isn't even my new spell's fault; it still does this for Gavotte, which has a valid targeting range of exactly 1. On closer inspection, the radius it looks at is actually based on your window size >.> 12:00:02 <04dracoomega> Like, I guess some of this stems from how it can be possible to get slightly different beam angles sometimes by deliberately aiming at a point outside LoS? 12:00:24 <04dracoomega> But I do not like this thing at all ^^; 13:22:22 <05icoson> a lot of that code is about as ancient as you can get 13:22:32 <05icoson> travel code, initial revision, greensnark commits, etc 13:22:36 <05icoson> complete with a goto 13:23:10 <05icoson> on the face of it the value for radius doesn't seem to make a lot of sense, but it's from 2007, so who knows what it could break 13:28:33 <04dracoomega> This is the most confusingly written thing 13:29:59 <04dracoomega> Also, I think the (currently unused, but still implemented) prefer_furthest functionality I was trying to use works by taking the closest target and then simulating tabbing backwards through valid monsters once, under the assumption that this will wrap around to the furthest thing? I think?? 13:31:08 <04dracoomega> (Which means it doesn't actually always work, either) 15:39:01 Unstable branch on underhound.eu updated to: 0.32-a0-938-g40ac7a0f4d (34) 18:44:23 <06pleasingfungus> draco: yeah, that sounds like what i remember 18:44:30 <06pleasingfungus> it’s sort of genius…. 18:46:06 <04dracoomega> It works only by coincidence, imo! 😛 18:46:55 <04dracoomega> (Though I think I found a way to actually start from LOS_RADIUS and spiral inward when looking, instead) 18:47:24 <04dracoomega> Technically this doesn't start at the complete nonsense max radius the function already uses, but that max radius was also complete nonsense. 18:49:26 <04dracoomega> (The code also seems to start searching for 'closest target' from the perspective of whatever space it was targeting last, if that target becomes invalid (ie: because it died) which also seems very silly. 18:50:01 <04dracoomega> Which also probably doesn't actually pick the closest target by default in a number of situations either. 18:50:42 <04dracoomega> I am not remotely ambitious enough right now to try giving any of this the rewrite it deserves, so I am just hoping these tweaks don't break some other thing mysteriously (though they seem okay so far?) 18:53:22 put it in trunk and let the players sort it out 😛 18:55:45 <04dracoomega> I am tidying things with that intention, yes 😛 18:56:27 <04dracoomega> Literally more time was spent in directn.cc regarding a UI improvement for this spell than was spent implementing the spell itself, by this point >.> 19:49:23 <04dracoomega> Having trouble figuring out why in the world this targeter is drawing AFF_MAYBE that are on top of creatures in a different color/transparency than AFF_MAYBE that are over empty terrain, when AFF_MAYBE returned by other targeters don't do that. 19:50:05 <04dracoomega> Tracing deeper, directn seems to be applying the same cell colours in both cases, but clearly something else somewhere is doing something different with them 20:05:17 <04dracoomega> Both return the same result, fill the overlay with the same tile, and yet look different. ??? 20:19:23 <06pleasingfungus> oh boy 20:23:16 <04dracoomega> I might put this aside in a bit if I can't figure it out, since it's not exactly harmful to hilight monsters in the beam path moreso than empty tiles. It's just a little annoying, given that marked monsters are extra hilighted already and I worry it could be slightly misleading. But if I managed to seemingly make ranged autofight work properly with magnavolt, I'll call that an actual victory >.>; 20:40:55 <04dracoomega> Yeah, I give up for now. (Though at this rate, I'm not going to have the rest of it push-worthy tonight either. I keep being like "Today's the day" and then find new problems ^^; ) 22:05:45 <04dracoomega> Spellbook wrangling... 22:21:26 -!- Changesite1 is now known as Changesite 22:35:48 Unstable branch on crawl.develz.org updated to: 0.32-a0-938-g40ac7a0f4d (34) 22:37:25 <06regret-⸸nde※> The worst part about spellbook wrangling is knowing there's more spells in the pipeline from other people eventually and everything will have to continue to be juggled whenever they finish up there, and how this has been churning violently and will continue churning indefinitely as we're left with things like Vampiric Draining in the Book of Debilitation and not the Book of Blood, a book which contains Call Imp. 22:59:12 Windows builds of master branch on crawl.develz.org updated to: 0.32-a0-938-g40ac7a0f4d 23:22:45 Unstable branch on cbro.berotato.org updated to: 0.32-a0-938-g40ac7a0f4d (34) 23:38:16 <06mumra> Somehow, what should be a just a really simple monster implementation entirely using existing effects, still involves seventeen file changes 23:55:44 Monster database of master branch on crawl.develz.org updated to: 0.32-a0-938-g40ac7a0f4d