00:12:44 will there be a release in a month or so? 01:21:10 Experimental (bcrawl) branch on underhound.eu updated to: 0.23-a0-1916-gb93c957c58 02:48:28 Monster database of master branch on crawl.develz.org updated to: 0.24-a0-443-g80245de 03:52:40 Fork (stoatsoup) on crawl.kelbi.org updated to: 0.21.2-1006-g1118e5e0f9 05:07:24 -!- amalloy is now known as amalloy_ 06:41:46 brasw1bg (L27 GrBe) Crash caused by signal #15: Terminated (Depths:4) 09:21:32 Stable (0.22) branch on underhound.eu updated to: 0.22.1-59-ge965b3aba3 09:23:18 !crashlog brasw1bg 09:23:31 1. brasw1bg, XL27 GrBe, T:90312 (milestone): http://crawl.berotato.org/crawl/morgue/brasw1bg/crash-brasw1bg-20190708-104147.txt 09:30:20 New branch created: pull/1099 (1 commit) 13https://github.com/crawl/crawl/pull/1099 09:30:20 03alexjurkiewicz02 {GitHub} 07https://github.com/crawl/crawl/pull/1099 * 0.24-a0-480-ga25fcf6: align braces 10(4 minutes ago, 1 file, 6+ 6-) 13https://github.com/crawl/crawl/commit/a25fcf68767b 09:44:44 03alexjurkiewicz02 {advil} 07* 0.24-a0-480-g7da6c14: align braces 10(18 minutes ago, 1 file, 6+ 6-) 13https://github.com/crawl/crawl/commit/7da6c1453bbf 09:47:24 Unstable branch on crawl.kelbi.org updated to: 0.24-a0-480-g7da6c1453b (34) 10:13:26 !tell amalloy concerning the leakage of world coordinates to players, would it be better if set_exclude was a no-op when (x, y) is unseen as well as when it's OOB? I can code that up 10:13:27 gmarks: OK, I'll let amalloy know. 10:19:26 amalloy_: ^ 10:30:21 that might work 10:32:03 there are some use cases for setting exclusions on unseen tiles, not sure how widely that's done 10:33:03 but sometimes if I see the edge of a pack that I want to exclude, death yaks or something, I'll do a bunch of spaces so that autoexplore is under control 10:33:35 not sure if you saw my comment, but there's a a more general issue with exclusions leaking than just lua 10:34:05 if, via the ui, you set an exclusion on an oob location it has different behavior than on an unseen space 10:34:13 Any thoughts on bounding-box-plus? 10:35:35 there's some technical issues with doing anything much with coordinates that are out of bounds, so just bounding box would be easier 10:35:54 that actually seemed consistent with differentiating edge-of-map rock/stone tiles though, right? 10:36:02 Hence the proposal to extend the play area with a MAX_LOS belt in all directions. 10:36:39 Now anywhere you're allowed to move the cursor is in bounds. 10:37:22 not sure if you've looked at the code but I think messing with the play area is trickier than it might seem 10:37:38 it's fairly old / brittle code 10:37:48 I have but also I'm aware messing with anything is trickier than it might seem. :-/ 10:38:07 heh 10:39:17 maps do run up against the level bounds but they're often mostly not too close to it, so it might already be closer to de facto what you have in mind for most situations 10:40:33 Trouble is, if it _ever_ hits the map edge, either (complete prohibition on putting exclusions in unexplored territory) or (it leaks information) :-/ 10:41:25 Maybe I should go away and have a bit of an experiment before I say more about it. 10:41:39 I guess I've never been so convinced that leaking map edges is too horrible, and there's actually a lot of weird little corner cases to worry about if one really wants to prevent that from happening 10:42:19 for example you can use digging to check them, and the dig targeter (kind of unavoidably) leaks; I decided it was better to have a leaky targeter than not one at all 10:52:26 advil: about the death yak exclusion use case, the way that map borders are handled seems a bit jank. If the death yak horde is the rightmost thing I've ever seen on the map, I can't exclude the right side of the pack because I can't move my cursor there. 10:53:14 yes, there's a weird UI issue there, sometimes it will let you and sometimes it won't, I've never figured out why 10:54:28 Isn't it based on the map? It will let you move your cursor in the bounding box of explored tiles 10:54:52 So if you've ever explored further right than the pack, you can exclude the unseen part but otherwise you can't 10:55:33 I guess that's the part that doesn't apply to the set_exclude function 10:56:43 it deosn't always seem to do even that, though I've only ever paid attention in wizmode (so who knows what I was doing) 10:58:02 In my brief testing (I.e. The last 3 minutes) that's what it always looks like, who knows though 11:20:58 Stable (0.23) branch on underhound.eu updated to: 0.23.1-82-g5129a83184 12:09:50 Kraken can move his tentacles while petrified. This doesn't seem right. 12:38:21 Yermak yeah, long-standing issue, same with paralyze 12:40:25 -!- amalloy_ is now known as amalloy 12:41:46 -!- amalloy is now known as amalloy_ 12:44:33 -!- amalloy_ is now known as amalloy 12:44:48 Not sure of the etiquette here, but could anyone take a look at my PR? It'd mean a lot. Thanks. https://github.com/crawl/crawl/pull/1098 12:47:30 That's good etiquette 12:48:12 Just to give you a heads up I've been the most active when it comes to reviewing clua PRs and working on merging them and at present I don't have much crawl time 12:49:23 OK, no rush or anything 12:49:28 Thanks for letting me know 12:57:13 I did a quick review 13:05:45 Unstable branch on crawl.akrasiac.org updated to: 0.24-a0-480-g7da6c14 (34) 14:11:43 man do I have a good bugfix coming up today 14:48:10 03advil02 07* 0.24-a0-481-ga9b13e7: Handle delay 0 values correctly in lm_trig.lua 10(32 minutes ago, 2 files, 4+ 3-) 13https://github.com/crawl/crawl/commit/a9b13e7d511b 14:48:10 03advil02 07* 0.24-a0-482-ge98fbd9: Adjust desolation fog machines 10(5 minutes ago, 1 file, 11+ 6-) 13https://github.com/crawl/crawl/commit/e98fbd949879 14:48:50 advil: Glee! Thanks ever so much 14:48:54 like years of angst about desolation fog, all because 0 is not false in lua 14:49:20 I had that on my list to look at - another successful victory for Captain Procrastination! 14:49:26 heh 14:51:14 Seriously, that's brilliant 14:51:43 :-) 14:53:43 I do think it would be useful if someone who knows desolation well (I have never played it) could have a look at the new fog machine params and see if the feel isn't too different 14:54:06 even after all that work I still have a fairly tenuous grasp on how fog machine params relate to effects, wouldn't be too shocked if there's more weirdness in there 14:55:54 probably a lot could be done to simplify our fog machines 14:56:04 switch's constructor parameter nightmare 14:56:53 yeah...though that's partly a result of an extremely literal translation from lua (where there's a long parameter list with lots of defaults) into c++ 14:57:10 I'm sort of inclined to close that PR 14:57:22 these commits were the result of trying to understand it 14:58:34 as much as that's going to annoy them...but that PR is really far from being mergeable and I think unnecessary 15:02:13 Unstable branch on crawl.kelbi.org updated to: 0.24-a0-482-ge98fbd9498 (34) 15:02:56 well I'll test your new stuff when I have time 15:03:17 but desolation isn't the only place where clouds/fog cause lag 15:03:42 I think if we're going to use more than the occasional single vault placed fog machine with a few clouds we should probably have c++ clouds as a goal 15:04:22 where else does it happen? 15:04:39 in this case I don't think the slowdown was from it being lua, just from it being called many times 15:12:08 actually, to be clearer, having done the profiling while working on those commits, I'm pretty sure there aren't speedups to be found by moving anything in lm_fog.lua into c++ 15:12:48 that code is already a fairly light wrapper on c++ calls 15:13:48 if it's called excessively there might be overhead to be saved (which if that PR does get a speedup, is probably what it's getting), but the solution is not to call it excessively 15:15:28 hm. ok, it's been a while since I've looked at it 15:15:34 on the c++ side there probably are a bunch of ways that cloud code isn't great, but I would want to do the profiling first 15:21:29 ah that PR also does not use delay 0 like the lua code it's intended to replace 15:26:13 03alexjurkiewicz02 {advil} 07* 0.24-a0-483-g98ea13e: Simplify logic 10(3 days ago, 1 file, 1+ 5-) 13https://github.com/crawl/crawl/commit/98ea13e50119 15:27:31 03Nikolai Lavsky02 {advil} 07* 0.24-a0-484-g962a3ee: Remove dump_book_spells option 10(10 days ago, 5 files, 3+ 15-) 13https://github.com/crawl/crawl/commit/962a3eeff77e 15:52:19 Unstable branch on crawl.kelbi.org updated to: 0.24-a0-484-g962a3eeff7 (34) 16:35:19 If I develop an AHK script to play crawl webtiles with a PS4 controller is there a place I can submit it? 16:37:13 Wait that's sick, how are there even enough keys for that 16:37:19 I would post it on the sub 16:37:31 And the tavern 16:38:28 I'm using dynamically updated keybinds so that when you open windows it remaps the keybinds. I.E. inventory etc. I dunno I wanna play crawl with a controller so I'm going to figure it out. 16:40:02 Worst case scenario I can set up a key that I just dump all the not commonly used keys into so its a 2 button combo to pray at an altar I.E. 16:40:32 No offense, but why would you want to play this game with a controller? O 16:40:57 I could see joystick + keyboard as being cool but pure controller sounds like a bit of a nightmare 16:42:50 How so? With a proper keybind of spells, and a hot key for shift movement, it seems much more comfortable. Mainly its to get away from auto explore, which I use entirely too much and its holding me back, but arrow keys aren't comfortable for me, and I don't like the num pad. 16:46:28 have you tried using the hjklyubn keys for movement? 16:47:05 it's not immediately intuitive, but it lets you keep your hand on the home keys, unlike arrow keys or numpad 16:47:18 03Gabriel Marks02 07https://github.com/crawl/crawl/pull/1098 * 0.24-a0-483-gc9f6d9a: Cause error on failed range check for waypoint number 10(3 hours ago, 1 file, 4+ 4-) 13https://github.com/crawl/crawl/commit/c9f6d9a71bef 16:47:18 03Gabriel Marks02 07https://github.com/crawl/crawl/pull/1098 * 0.24-a0-484-g6a8a0ff: Error on failed bounds checks 10(3 hours ago, 1 file, 3+ 3-) 13https://github.com/crawl/crawl/commit/6a8a0ff0c1c6 16:47:18 03Gabriel Marks02 07https://github.com/crawl/crawl/pull/1098 * 0.24-a0-485-g41c9e02: Replace comment with an ASSERT_RANGE 10(3 hours ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/41c9e0246146 16:47:18 03Gabriel Marks02 07https://github.com/crawl/crawl/pull/1098 * 0.24-a0-486-g8da3e8a: Update re-assignment message to reflect recent waypoint changes 10(3 hours ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/8da3e8a85b43 16:47:18 03Gabriel Marks02 07https://github.com/crawl/crawl/pull/1098 * 0.24-a0-487-g378e4ba: Return error 10(8 minutes ago, 1 file, 5+ 5-) 13https://github.com/crawl/crawl/commit/378e4ba003fe 16:47:35 Ive used nethack keys, I dunno Im on a controller kick recently. Mainly it lets me recline while playing 16:50:49 If you're interested, I have a set of WASDQEZC binds ;) 16:51:35 I would probably use them.:D 16:52:17 I should probably make a post huh 17:13:10 I updated my PR btw ^ 17:25:40 is there a way to use the clua interpreter outside of wizmode? i'm not sure what's an accurate way to test how a player would interact with lua functions 17:25:40 amalloy: You have 1 message. Use !messages to read it. 17:27:05 gmarks: like advil i'm not sure. i have definitely set exclusions in unexplored tiles before; it would be weird to do the same with waypoints, but who knows? 17:28:12 currently you can't use the in-game UI to set an exclusion or waypoint that's out of bounds, because the cursor only moves as far in each direction as you've explored 17:29:10 perhaps the lua equivalents could do the same: allow you to target a point (x,y) if you've seen any point with the same x, and any point with the same y 17:30:52 As for testing outside of wizmode, you can bind a key to execute a function that will do_file your main file 17:31:07 relatedly: how much do we care about crashes you can only trigger in wizmode? e.g., pressing xlm enough times in a row will crash because it tries to move you out of bounds 17:32:59 As for the placing of exclusions/waypoints, I had the same thought as you that clua should only be able to place exclusions/waypoints where the player can manually place them. We might be able to use code from eXamine to do that. 17:51:04 03Gabriel Marks02 07https://github.com/crawl/crawl/pull/1098 * 0.24-a0-488-g95bc39c: Fix comments 10(6 minutes ago, 1 file, 3+ 3-) 13https://github.com/crawl/crawl/commit/95bc39c41989 17:51:04 03Gabriel Marks02 07https://github.com/crawl/crawl/pull/1098 * 0.24-a0-489-g83f2f39: Player-centered coordinates in error return to not leak 10(5 minutes ago, 1 file, 3+ 3-) 13https://github.com/crawl/crawl/commit/83f2f3927d54 18:21:51 Unstable branch on underhound.eu updated to: 0.24-a0-484-g962a3eeff7 (34) 18:35:57 03marksg0702 {amalloy} 07* 0.24-a0-485-ge77e6fb: Add clua function to set waypoint (#1098) 10(9 seconds ago, 3 files, 37+ 8-) 13https://github.com/crawl/crawl/commit/e77e6fb25671 18:36:40 <3 18:37:42 gmarks: we typically add people to CREDITS.txt after they've had a patch accepted, unless the author prefers not to be credited. as far as i can see this is your first commit; would you like to be listed as Gabriel Marks, gmarks, something else, or not listed? 18:38:08 Gabriel Marks works 18:38:11 Thanks 18:38:37 gmarks: also, we can configure Cheibriados to show your commits as by "gmarks" instead of "marksg07" if you like 18:38:49 congrats on your first commit :) 18:40:05 amalloy: you can get to the clua interpreter by binding a key to CMD_LUA_CONSOLE 18:40:19 advil: NICE bugfix 18:40:51 I use this for debugging and developing rc lua magic 18:41:33 amalloy: that would be great 18:41:40 ah, thanks ebering. that's nicer. i could only find the command to get to it from wizmode, and i wasn't sure if it was possible for players to get to the same console 18:41:40 floraline: thanks :) 18:43:55 03amalloy02 07* 0.24-a0-486-ged88169: Add gmarks to CREDITS and mailmap after e77e6fb 10(29 seconds ago, 2 files, 2+ 0-) 13https://github.com/crawl/crawl/commit/ed8816983f89 18:45:34 amalloy: it's not bound to any key by default and I'm not sure it's documented outside of the CMD_* enum :P 18:46:47 Speaking of the CMD_* enum, is one of those mapped to the action of Ctrl-E, i.e. Making you travel at the speed of your slowest ally? 18:53:20 Unstable branch on crawl.kelbi.org updated to: 0.24-a0-486-ged8816983f (34) 19:46:30 !source cmd-keys.h 19:46:31 https://github.com/crawl/crawl/blob/master/crawl-ref/source/cmd-keys.h 19:46:35 gmarks: {CONTROL('E'), CMD_TOGGLE_TRAVEL_SPEED} 21:03:38 advil: Good timing, I just hit a Desolation :-) 21:33:40 -!- amalloy is now known as amalloy_ 21:53:12 -!- amalloy_ is now known as amalloy 23:52:22 New branch created: pull/1100 (1 commit) 13https://github.com/crawl/crawl/pull/1100 23:52:22 03alexjurkiewicz02 {GitHub} 07https://github.com/crawl/crawl/pull/1100 * 0.24-a0-487-g46be30f: Add !cancellation to Zig loot table (EugeneJudo) 10(3 minutes ago, 1 file, 2+ 1-) 13https://github.com/crawl/crawl/commit/46be30f54246