00:11:32 advil: I hope you haven't been secretly working on aidanh's pr-599 branch 00:12:06 I've got a version rebased to new ui that had a gnarly bit of formatted_string bug, so I'm guessing you aren't since you're taking it easy 00:13:36 New branch created: pr-599-rebase (5 commits) 13https://github.com/crawl/crawl/tree/pr-599-rebase 00:13:37 03Sandman25DCSS02 {ebering} 07[pr-599-rebase] * 0.23-a0-713-g49f3dbd: Display range for spells in monster description (aidanh) 10(1 year, 5 months ago, 1 file, 15+ 15-) 13https://github.com/crawl/crawl/commit/49f3dbd1da17 00:13:37 03Aidan Holm02 {ebering} 07[pr-599-rebase] * 0.23-a0-714-g6890817: Add hex-chance and spell-range explanatory text 10(12 months ago, 1 file, 4+ 0-) 13https://github.com/crawl/crawl/commit/68908174db82 00:13:37 03Aidan Holm02 {ebering} 07[pr-599-rebase] * 0.23-a0-715-g8362a7b: Use colour to differentiate in/out-of-range 10(12 months ago, 1 file, 5+ 2-) 13https://github.com/crawl/crawl/commit/8362a7b3469a 00:13:37 03Aidan Holm02 {ebering} 07[pr-599-rebase] * 0.23-a0-716-g1a6e1d9: Right-align hex-chance in monster spell list 10(12 months ago, 1 file, 3+ 6-) 13https://github.com/crawl/crawl/commit/1a6e1d9148f5 00:13:37 03ebering02 07[pr-599-rebase] * 0.23-a0-717-g3d954c0: Improve colour handling for spell ranges 10(5 minutes ago, 2 files, 8+ 4-) 13https://github.com/crawl/crawl/commit/3d954c0798cc 00:17:47 anyhow, it's rebased and working against master now 00:18:07 remaining todo added to the pr; 00:30:05 03ebering02 07* 0.23-a0-713-ge59bbb1: Display (immune) in monster hex chance in webtiles 10(54 seconds ago, 2 files, 6+ 2-) 13https://github.com/crawl/crawl/commit/e59bbb1e9260 00:32:01 Unstable branch on crawl.kelbi.org updated to: 0.23-a0-713-ge59bbb1e92 (34) 01:17:02 03ebering02 07[pr-599-rebase] * 0.23-a0-718-g768dd17: Only display ranges in monster descriptions 10(5 minutes ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/768dd174cc14 01:17:02 03ebering02 07[pr-599-rebase] * 0.23-a0-719-g4d4ae68: Add spell ranges to monster info in webtiles 10(3 minutes ago, 2 files, 12+ 0-) 13https://github.com/crawl/crawl/commit/4d4ae68a5146 01:27:13 The build passed. (pr-599-rebase - 3d954c0 #10849 : Edgar A. Bering IV): https://travis-ci.org/crawl/crawl/builds/479728163 01:48:41 Battlespheres can be blinded from Ru's passive aura 13https://crawl.develz.org/mantis/view.php?id=11827 by CanOfWorms 01:51:20 question: should player invisibility be LOS_RANGE, really? 01:52:41 this is the one special case that needs to be handled either by a gross special case in pr-599-rebase or better, removed 01:53:06 I guess, there are some use cases for invising your allies 01:53:23 hide your dryad from draconians to use summon forest in zot is one I did recently 01:53:48 but because of the default tarageter for invis and monster behavior I thnk most players only target themselves 01:54:51 03ebering02 07[pr-599-rebase] * 0.23-a0-720-gc2677e9: Remove ranges from Haste and Might 10(9 minutes ago, 2 files, 5+ 7-) 13https://github.com/crawl/crawl/commit/c2677e9b463a 02:00:03 Death Channel duration is leaked when cast at max duration 13https://crawl.develz.org/mantis/view.php?id=11828 by CanOfWorms 02:36:23 -!- muffindrake3 is now known as muffindrake 03:36:26 Fork (bcrawl) on crawl.kelbi.org updated to: 0.23-a0-1418-g93b54f1e6f 09:46:05 ebering: no 10:00:11 hi devs - firstly: thanks for all of your hard work on this lovely game. secondly: i just stumbled across a little bug when transferring my save to the latest: "fatal error: lua_marker::read_error: ...mater/crawl-git-e59bbb1e92/data/dat/dlua/fnwrap.lua.30: could not resolve 'callback.spider_trap_stepped' to a function' 10:00:32 if you'd prefer this reported somewhere else, please let me know (and apologies for the noise!) 11:04:41 any way to get windows builds to produce a stack trace when they crash? using `make CROSSHOST=i686-w64-mingw32 package-windows debug-lite` here, although if there's a better way I'm all ears 11:08:18 -!- muffindrake1 is now known as muffindrake 11:17:57 can-of-bees: It would be helpful if you could report it on the bug tracker here: https://crawl.develz.org/mantis/bug_report_page.php. That means your report won't get lost amongst everyone else's comments here. (You might need to make a Mantis account.) :) 11:24:28 hi NP7 - i'll do it. thanks! 11:36:59 Unable to re-access game 13https://crawl.develz.org/mantis/view.php?id=11829 by CanOfBees 11:42:33 &dump CanOfBees 11:42:33 http://crawl.berotato.org/crawl/morgue/CanOfBees/CanOfBees.txt 11:43:03 can-of-bees: here's your dump 11:48:59 NP7: hey, sorry - i was just figuring out which files to try to hand off to Mantis. thanks for that :) 11:49:09 i'll attach to the ticket 11:49:29 also if you can find your save file that would also be helpful 11:50:00 I think you have to log in via other methods to get access to that though 11:50:54 NP7: okay - I'll ask the server admin about that 11:51:30 I'm sure one of the developers will be able to help at some point today :) 12:05:25 Unstable branch on crawl.akrasiac.org updated to: 0.23-a0-713-ge59bbb1 (34) 12:22:38 -!- amalloy is now known as amalloy_ 12:44:12 aidanh: no, I think windows stack traces have been one of those long-term wishlist items that no one quite knows how to do. See https://crawl.develz.org/mantis/view.php?id=98 12:44:40 looks like johnny0 had something working in 2014 though 12:46:03 ebering also, just to be clear, s/taking it easy/burnt out from toxic community interactions 12:51:19 03NormalPerson702 07https://github.com/crawl/crawl/pull/938 * 0.23-a0-707-g0299340: Add a struct to use for MiscastEffect::special_source 10(35 minutes ago, 15 files, 89+ 80-) 13https://github.com/crawl/crawl/commit/02993409fd52 12:51:19 03NormalPerson702 07https://github.com/crawl/crawl/pull/938 * 0.23-a0-708-g60bf322: Convert `enum miscast_source` to an enum class 10(17 minutes ago, 15 files, 100+ 90-) 13https://github.com/crawl/crawl/commit/60bf322a75f7 12:51:19 03NormalPerson702 07https://github.com/crawl/crawl/pull/938 * 0.23-a0-709-g4378faa: Convert `enum recall_t` to an enum class 10(8 days ago, 3 files, 10+ 10-) 13https://github.com/crawl/crawl/commit/4378faa36c76 12:51:19 03NormalPerson702 07https://github.com/crawl/crawl/pull/938 * 0.23-a0-710-g1be2c94d: Add a note to docs/obsolete/cut_spells.txt 10(3 days ago, 1 file, 9+ 1-) 13https://github.com/crawl/crawl/commit/1be2c94d171c 12:54:26 advil: dang 12:54:38 it can be pretty grating 12:56:20 indeed 12:56:39 btw, it looks like you might need to keep around a function named callback.spider_trap_stepped for save compat 12:57:11 oh you saw it 13:02:14 yep 13:05:58 and change pushed 13:07:01 03ebering02 07* 0.23-a0-714-g00c90a0: Save compat for c89a89d9 (11829) 10(5 minutes ago, 1 file, 7+ 0-) 13https://github.com/crawl/crawl/commit/00c90a0c2284 13:11:03 should that also set `triggered=false`? [I don't know what this actually does I just notice it's there in the other calls, including the old one] 13:11:14 no 13:11:23 the old one sets up a callback which stores the data object 13:11:31 which gets called with the data object as argument each time 13:11:39 so triggered is already set 13:11:45 ah, I see 13:11:46 and already set to true or false as need be 13:12:31 if you're itching to put some eyes on some crawl changes I'd like someone else to take a peek at pr-599-rebase 13:12:54 I could do that 13:13:04 I think I'm going to merge it tonight leaving invis as is until I can hear from some others about player invis being made self-target only 13:13:24 I couldn't think of any reason why it wouldn't be self-target only 13:13:32 maybe some people make their orcs invis or something? 13:14:17 re tournament date, has anyone talked about revisiting the various (possibly non-trivial) rules changes that were under discussion a few months ago? 13:17:10 Unstable branch on crawl.kelbi.org updated to: 0.23-a0-714-g00c90a0c22 (34) 13:18:58 -!- amalloy_ is now known as amalloy 13:55:53 advil: I was using it to invis my dryad on my BaIE occasionally 14:19:03 that seems too tedious to allow 14:19:32 the special case doesn't seem that ugly to me in the scale of ugliness 14:19:34 though 14:25:43 there might already be some other ranged logic that should take care of this 14:26:11 anything with SPFLAG_SELFENCH doesn't count as ranged, in some places in the code 14:33:11 well, the logic in mons_has_ranged_spell wouldn't do exactly what this wants (since some escape spells should show range), but I'm comfortable preventing anything with SPFLAG_SELFENCH from showing range 14:35:48 ebering: do you know if there a case where range = 0? 14:38:13 would've thought no 14:38:31 (for monsters at least) 14:47:59 03advil02 07[pr-599-rebase] * 0.23-a0-721-gaa49765: Various small tweaks 10(5 minutes ago, 1 file, 25+ 10-) 13https://github.com/crawl/crawl/commit/aa49765ae990 14:50:44 oh, I guess the webtiles logic is distinct 15:40:50 03advil02 07[pr-599-rebase] * 0.23-a0-722-g9668541: Generalize range string logic for webtiles 10(22 minutes ago, 2 files, 23+ 23-) 13https://github.com/crawl/crawl/commit/9668541f803e 15:40:50 03advil02 07[pr-599-rebase] * 0.23-a0-723-g12dd782: Handle ?/ better, generalize range "key" to webtiles 10(5 minutes ago, 2 files, 9+ 9-) 13https://github.com/crawl/crawl/commit/12dd7828f167 15:42:25 are there any hexes without range cast by monsters? 15:54:30 advil: Mennas has Mass Confusion, a hex spell without range 15:54:41 thanks 16:06:26 i am so close to getting a local Sequell instance running. all of its seqdb commands succeed: it fetches logfiles, adds them to postgres, and adds the necessary indexes 16:06:31 but when i run scripts/runcmd.pl, it seems to be trying to use the wrong psql authenticatoin mechanism, and thus fails to connect: "FATAL: Peer authentication failed for user "sequell". i think i just need to tell it to connect over localhost instead of unix sockets, but supposedly that's the default already so i don't know how to change it. any ideas? 16:14:41 -!- aidanh_ is now known as aidanh 16:15:03 advil: re range 0 spells: I didn't see that when I read through the enum; spells were either positive range or range -1 16:15:12 -1 for "not applicable" 16:16:23 03advil02 07[pr-599-rebase] * 0.23-a0-724-g41c0029: Fix a bug in util.formatted_string_to_html 10(2 minutes ago, 1 file, 3+ 0-) 13https://github.com/crawl/crawl/commit/41c002983f31 16:16:38 amalloy, that message seems to mean unix sockets and authentication based on OS user are in use; it will probably help to create a database user for "sequell" or to adjust the user name maps 16:16:43 ok...just wondering because it was implicitly treating 0 as "has a range", which I guess is conceivably right 16:16:54 41c0029 is pretty amazing for anyone who enjoys crawl bugs 16:17:01 I have no idea how it lasted that long 16:17:21 jilles: yes, there is a database user named "sequell", with a password of "sequell", as directed in https://github.com/crawl/sequell 16:18:11 ebering can I rebase that branch to fix a typo in that last commit message? 16:18:38 well, maybe i misunderstand. there is no OS user "sequell", but there is a postgres user named "sequell". like, it's using unix sockets and peer auth, but it's "supposed" to use TCP and password auth 16:24:32 if the database is on the same machine there seems little reason to use password auth 16:24:54 user name maps can deal with the situation where OS and database user must be named differently 16:25:33 hm, that seems like a reasonable workaround 16:27:07 03NormalPerson702 07https://github.com/crawl/crawl/pull/938 * 0.23-a0-711-gc7a21f0: fixup! Convert `enum miscast_source` to an enum class 10(7 minutes ago, 1 file, 2+ 0-) 13https://github.com/crawl/crawl/commit/c7a21f0e8e66 16:35:38 advil: sure 16:35:39 thanks, jilles. installing a usermap let me successfully run a query 16:36:20 now i just need to wait like 10 hours while it repopulates the database; i tried dropping it to let seqdb recreate the database in case i'd created it wrong 16:37:33 03advil02 07[pr-599-rebase] * 0.23-a0-724-g5357f6d: Fix a bug in util.formatted_string_to_html 10(23 minutes ago, 1 file, 3+ 0-) 13https://github.com/crawl/crawl/commit/5357f6d0a92b 16:42:08 ebering: that's everything I found that I wanted to adjust 17:12:58 https://i.imgur.com/SX0ixC5.png am i an idiot, or does this callfn() call make no sense 17:14:48 what is the purpose of the keyword "activate" in the callfn() call 17:14:56 why is args 4 17:15:39 nevermind, it makes sense, it's just super convoluted 17:58:35 erf (L5 FoFi) ASSERT(you.props.exists(sac_vec_key)) in 'ability.cc' at line 1094 failed. (D:3) 18:22:51 Unstable branch on underhound.eu updated to: 0.23-a0-714-g00c90a0c22 (34) 18:37:42 03Sandman25DCSS02 {ebering} 07* 0.23-a0-715-gab26df7: Display range for spells in monster description (aidanh) 10(1 year, 5 months ago, 1 file, 15+ 15-) 13https://github.com/crawl/crawl/commit/ab26df78391d 18:37:42 03Aidan Holm02 {ebering} 07* 0.23-a0-716-g6d93017: Add hex-chance and spell-range explanatory text 10(12 months ago, 1 file, 4+ 0-) 13https://github.com/crawl/crawl/commit/6d93017843b8 18:37:42 03Aidan Holm02 {ebering} 07* 0.23-a0-717-gd70207d: Use colour to differentiate in/out-of-range 10(12 months ago, 1 file, 5+ 2-) 13https://github.com/crawl/crawl/commit/d70207d22c89 18:37:42 03Aidan Holm02 {ebering} 07* 0.23-a0-718-gc9aa113: Right-align hex-chance in monster spell list 10(12 months ago, 1 file, 3+ 6-) 13https://github.com/crawl/crawl/commit/c9aa113a4fe5 18:37:42 03ebering02 07* 0.23-a0-719-ge15ba66: Improve colour handling for spell ranges 10(18 hours ago, 2 files, 8+ 4-) 13https://github.com/crawl/crawl/commit/e15ba6666cf8 18:37:42 03ebering02 07* 0.23-a0-720-g32e1316: Only display ranges in monster descriptions 10(17 hours ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/32e13164b622 18:37:42 03ebering02 07* 0.23-a0-721-g8ac9f62: Add spell ranges to monster info in webtiles 10(17 hours ago, 2 files, 12+ 0-) 13https://github.com/crawl/crawl/commit/8ac9f62d1dd1 18:37:42 03ebering02 07* 0.23-a0-722-g99592df: Remove ranges from Haste and Might 10(17 hours ago, 2 files, 5+ 7-) 13https://github.com/crawl/crawl/commit/99592df15c72 18:37:42 03advil02 {ebering} 07* 0.23-a0-723-g4d4c92e: Various small tweaks 10(4 hours ago, 1 file, 25+ 10-) 13https://github.com/crawl/crawl/commit/4d4c92e218ad 18:37:42 03advil02 {ebering} 07* 0.23-a0-724-g0b8caf7: Generalize range string logic for webtiles 10(3 hours ago, 2 files, 23+ 23-) 13https://github.com/crawl/crawl/commit/0b8caf7e586b 18:37:42 ... and 2 more commits 18:39:28 COMMITSTORM 18:41:49 !crashlog erf 18:41:51 1. erf, XL5 FoFi, T:3360 (milestone): https://crawl.kelbi.org/crawl/morgue/erf/crash-erf-20190115-225834.txt 18:42:02 advil: thanks for the assist on getting that finished 18:42:09 no problem 18:42:10 I like the selfench solution 18:46:12 the only slightly odd thing is that a bunch of summons are marked with that, not sure why 18:46:16 but they don't have a range, so 18:46:35 (haunt is fine) 18:46:50 they probably have that flag for ai reasons 18:47:08 monsters prefer trying to selfench in the second attempt at rolling a spell to cast if the first failed 18:47:20 Unstable branch on crawl.kelbi.org updated to: 0.23-a0-726-g9fa157057a (34) 18:49:56 03Aidan Holm02 {advil} 07[stone_soup-0.22] * 0.22.1-6-g1668fe2: Limit extra Ru ability info to followers (amalloy) 10(10 weeks ago, 2 files, 6+ 0-) 13https://github.com/crawl/crawl/commit/1668fe2c400a 18:49:57 yeah, selfench is part of what determines ranged for keep ranged ai 18:52:57 one odd thing I noticed about this change is that the spell descriptions still use the arrow thing, and don't have an in-range indicator, so we possibly should include this there, too 21:06:50 -!- amalloy is now known as amalloy_ 21:30:16 -!- amalloy_ is now known as amalloy 22:04:12 advil: hmm yeah 22:08:06 there are a bunch of good UI improvement ideas on the old docu wiki I'm going to migrate to GH and put in an easier to find place 22:32:15 New branch created: pull/944 (6 commits) 13https://github.com/crawl/crawl/pull/944 22:32:15 03bjobae02 {ebering} 07https://github.com/crawl/crawl/pull/944 * 0.23-a0-714-gee4500f: Add support for an 'Extra' pane in religion screen 10(4 weeks ago, 1 file, 29+ 8-) 13https://github.com/crawl/crawl/commit/ee4500f2134e 22:32:15 03bjobae02 {ebering} 07https://github.com/crawl/crawl/pull/944 * 0.23-a0-715-g57b4047: List named followers for Beogh worshippers 10(2 days ago, 3 files, 52+ 2-) 13https://github.com/crawl/crawl/commit/57b404755c4b 22:32:15 03bjobae02 {ebering} 07https://github.com/crawl/crawl/pull/944 * 0.23-a0-716-g9cf7415: Future proof Beogh gift monster props 10(2 days ago, 1 file, 18+ 0-) 13https://github.com/crawl/crawl/commit/9cf741518bae 22:32:15 03ebering02 07https://github.com/crawl/crawl/pull/944 * 0.23-a0-717-g22364f8: God Extra panes for Ash, Gozag, and Hep 10(2 days ago, 1 file, 66+ 60-) 13https://github.com/crawl/crawl/commit/22364f85a77e 22:32:15 03ebering02 07https://github.com/crawl/crawl/pull/944 * 0.23-a0-718-g38bffda: Add a god Extra pane for Nemelex 10(8 hours ago, 4 files, 34+ 14-) 13https://github.com/crawl/crawl/commit/38bffda84c5f 22:32:15 03ebering02 07https://github.com/crawl/crawl/pull/944 * 0.23-a0-719-g0d2dbcc: Add a god Extra pane for Wu Jian 10(7 hours ago, 2 files, 18+ 12-) 13https://github.com/crawl/crawl/commit/0d2dbcc9c9d0 22:36:51 I can't log in to cao to trigger a rebuild, but one should be done so that CAO gets 0.23-a0-714 22:36:59 it seems I'm not yet a dgl admin there 22:37:09 gammafunk: |amethyst 22:37:47 ??rebuild 22:37:48 rebuild[1/2]: http://crawl.akrasiac.org/rebuild/ http://underhound.eu:81/rebuild/ http://crawl.berotato.org/crawl/rebuild/ http://crawl.xtahua.com/rebuild/ https://crawl.kelbi.org/rebuild/ Bug gammafunk, advil, |amethyst, or Nap.Kin for CDO. Use your powers wisely. 22:38:02 started 22:38:57 thanks 22:41:53 Unstable branch on crawl.akrasiac.org updated to: 0.23-a0-726-g9fa1570 (34) 22:46:21 advil: I don't know how familiar with webtiles you are, but adding 944 to webtiles seems a bit involved 22:48:10 anyone used a c++ language server with crawl? I'm playing with cquery