00:09:31 -!- Marvin is now known as Guest57751 00:22:13 -!- mikee___ is now known as mikee_ 00:28:55 Unstable branch on CRAWL.XTAHUA.COM updated to: 0.21-a0-476-gd103f4b (34) 00:42:12 -!- mikee___ is now known as mikee_ 00:43:28 gammafunk: did you see https://www.reddit.com/r/dcss/comments/7elrbn/ijyb_had_no_wand/ ? i think they're right: i just spawned ijyb and maurice in wizmode and neither one of them got a wand. in 0.20, they do get a wand when spawned in wizmode. cc |amethyst since he responded to the post already 00:43:52 oh, could be a bug 00:44:02 I tested by giving monsters a wand with x -> o 00:44:06 and they zapped it fine 00:44:17 perhaps they don't see the wand as valid? 00:45:29 <|amethyst> err 00:45:30 <|amethyst> yeah 00:45:36 <|amethyst> !source monster::likes_wand 00:45:36 1/1. https://github.com/crawl/crawl/blob/master/crawl-ref/source/monster.cc#L777 00:45:43 <|amethyst> !source wand_charge_value 00:45:44 1/1. https://github.com/crawl/crawl/blob/master/crawl-ref/source/item-prop.cc#L1580 00:45:49 <|amethyst> those numbers don't match up 00:46:20 ah yeah, I changed the meaning of that function 00:47:04 @??ijyb 00:47:04 Ijyb (02g) | Spd: 10 | HD: 3 | HP: 21-28 | AC/EV: 6/11 | Dam: 4 | 10weapons, 10items, 10doors | Res: 06magic(10) | XP: 36 | Sz: small | Int: human. 00:47:14 @??maurice 00:47:14 Maurice (03@) | Spd: 10 | HD: 5 | HP: 49-71 | AC/EV: 2/13 | Dam: 910(steal) | 10weapons, 10items, 10doors, spellcaster | Res: 06magic(20) | XP: 468 | Sp: sprint, invisibility, blink, teleport self [04emergency] | Sz: Medium | Int: human. 00:47:30 <|amethyst> just divide by 3 probably 00:47:43 <|amethyst> or I guess multiply HD by 6 instead of 2 and compare against 72 00:49:16 <|amethyst> looks like currently no monster will ever use a wand other than clouds, scattershot, iceblast, and acid? 00:49:21 i'm confused, though. shouldn't ijyb and maurice both like clouds, scattershot, iceblast, and acid, as written? is the problem that we don't offer them those wands so it doesn't matter whether they like it? 00:49:24 <|amethyst> err, will ever pick one up 00:49:38 <|amethyst> amalloy: it probably doesn't reroll 00:50:08 ah, indeed 00:50:15 on the fourth try, ijyb got a wand of iceblast 00:54:24 fr: instead of this weird shit about multiplying hit dice and comparing against the number of charges a wand starts with, define a wand_quality enum that's STRONG for eg /clouds, WEAK for /flame, and so on. then this monster::likes_wand can just compare hd against wand quality in a reasonable way 00:55:27 (and use that enum to determine how many charges a wand starts with, of course) 00:57:14 <|amethyst> struct wand_def 00:57:23 <|amethyst> (doesn't exist but maybe should) 00:57:30 yeah 00:57:37 this stuff should be a propery of wands 00:57:44 <|amethyst> also, min HD 00:57:46 or rather, of wand types 00:57:54 <|amethyst> i.e. the "high-power" wand thing 00:58:07 <|amethyst> and is_offensive_wand 00:58:28 what min hd thing? 01:00:04 <|amethyst> monsters with <5 HD, and those with a special flag (just Grinder) won't take a "high-tier" wand, which includes all the high-value ones, as well as enslavement, paralysis, and disintegration 01:01:02 wild. yes, more junk to cram in struct wand_def 01:01:27 <|amethyst> (which leaves flame, digging, polymorph, and randeff; the last of which they won't use) 01:01:40 03Yermak02 {amalloy} 07* 0.21-a0-477-gafc4b18: Add size to % screen 10(12 days ago, 4 files, 34+ 23-) 13https://github.com/crawl/crawl/commit/afc4b183e5b9 01:01:48 <|amethyst> maybe that should now be "low-HD monsters who aren't ijyb don't get wands at all" 01:02:27 03Yermak02 {amalloy} 07* 0.21-a0-478-g6b4cb37: Remove Haste reference 10(12 days ago, 1 file, 3+ 3-) 13https://github.com/crawl/crawl/commit/6b4cb3753bfd 01:05:57 !source is_excluded 01:05:57 1/2. https://github.com/crawl/crawl/blob/master/crawl-ref/source/exclude.cc#L259 01:07:09 !source travel_exclude 01:07:10 1/3. https://github.com/crawl/crawl/blob/master/crawl-ref/source/exclude.cc#L135 01:09:20 03gammafunk02 07* 0.21-a0-479-g139aec4: Changelog through 0.21-a0-478-g6b4cb3753b 10(2 minutes ago, 1 file, 40+ 17-) 13https://github.com/crawl/crawl/commit/139aec4fbc70 01:10:00 changelog version doesn't match that of commit title, oops 01:10:27 but if anyone wants to make corrections, feel free; i'll do the post tomorrow 01:10:36 <|amethyst> FR: find a commit hash that does make that work 01:10:48 <|amethyst> oh, never mind, misunderstood 01:11:18 <|amethyst> FR: find a commit hash that does make what I thought you meant work 01:11:48 <|amethyst> i.e. have the changelog commit's own shorthash in its commit message 01:12:12 |amethyst: there are a number of vanity-hash generating tools that could probably do this 01:12:54 also if people want to do cherry-picking for any 0.20.2 01:13:33 I guess that being easier is one good byproduct of slower development 01:17:48 03amalloy02 {GitHub} 07* 0.21-a0-480-g5da0f0d: Mild copy-editing of changelog.txt 10(6 seconds ago, 1 file, 2+ 2-) 13https://github.com/crawl/crawl/commit/5da0f0de552f 01:20:28 Unstable branch on crawl.develz.org updated to: 0.21-a0-479-g139aec4 (34) 01:22:03 !source is_bad_food 01:22:03 1/1. https://github.com/crawl/crawl/blob/master/crawl-ref/source/food.cc#L758 01:34:08 -!- amalloy is now known as amalloy_ 01:48:45 -!- amalloy_ is now known as amalloy 01:58:35 Windows builds of master branch on crawl.develz.org updated to: 0.21-a0-480-g5da0f0d 02:35:54 The build passed. (master - 5da0f0d #8947 : Alan Malloy): https://travis-ci.org/crawl/crawl/builds/305647921 02:41:03 03Aidan Holm02 07https://github.com/crawl/crawl/pull/624 * 0.21-a0-397-g7c3ae8b: Don't (auto)travel via stairs into exclusions 10(5 weeks ago, 3 files, 37+ 0-) 13https://github.com/crawl/crawl/commit/7c3ae8bb4e0c 02:54:53 Monster database of master branch on crawl.develz.org updated to: 0.21-a0-480-g5da0f0d 02:59:09 -!- amalloy is now known as amalloy_ 03:14:26 !messages 03:14:27 No messages for TZer0. 03:18:43 10/10 autoexplore: https://2ch.hk/vg/src/24341112/15113164928840.png 03:21:46 Unstable branch on crawl.beRotato.org updated to: 0.21-a0-480-g5da0f0d (34) 03:51:21 Aiming: Paralyse (will waste charges) 03:58:18 -!- mikee___ is now known as mikee_ 03:59:03 Game warns about wasting charges. 13https://crawl.develz.org/mantis/view.php?id=11289 by Yermak 04:42:49 -!- aditya1 is now known as aditya 07:13:21 ? 07:13:26 exit 07:14:46 exit 10:06:36 03Aidan Holm02 {amalloy} 07* 0.21-a0-481-ge9ed347: Don't (auto)travel via stairs into exclusions 10(5 weeks ago, 3 files, 37+ 0-) 13https://github.com/crawl/crawl/commit/e9ed347cad6f 10:23:25 -!- amalloy_ is now known as amalloy 10:24:02 03advil02 07* 0.21-a0-482-g31fed6f: Clean up a case of charge id conditioning on pakellas (Yermak, 11289) 10(2 minutes ago, 1 file, 1+ 6-) 13https://github.com/crawl/crawl/commit/31fed6faf02a 10:38:01 advil: there's no way to waste charges anymore (and if there is there shouldn't be) 10:38:20 yeah, I'm removing that message altogether 10:43:00 03advil02 07* 0.21-a0-483-gd70176a: Remove pakellas device charge id (everyone gets this now) 10(4 minutes ago, 3 files, 0+ 26-) 13https://github.com/crawl/crawl/commit/d70176a092bd 10:43:00 03advil02 07* 0.21-a0-484-gcd31912: Remove a few references to wasting charges 10(31 seconds ago, 2 files, 2+ 9-) 13https://github.com/crawl/crawl/commit/cd3191265fad 10:44:39 possibly the whole ISFLAG_KNOW_PLUSES thing for wands could be removed too, but that might have interactions 10:50:25 advil: I believe that's used for things like amulets of reflection etc? 10:50:25 gammafunk: You have 4 messages. Use !messages to read them. 10:52:00 yeah, it's probably used for other stuff (that's what I mean by interactions), but I think it's now vestigial on wands 10:52:20 any wand in inventory should always have that set 10:52:36 or really, any id'd wand 11:10:57 oh, yeah, that could be set upon creation for wands certainly 11:11:10 I'm not sure if it's set for other items 11:11:20 I just removed (most) of the bits of code checking it 11:14:37 I'm speedrunning Hunter for the first time. Right now my game looks like this: I fire missiles, monsters die and leave their equipment above my fired missiles. When fight is over I save and load the game. Missiles now are on the top of items piles. I apport them. Do you think this is right situation? 11:16:22 https://crawl.develz.org/mantis/view.php?id=10337 11:19:49 that seems bad, but looking at neil's comment, it would be somewhat painful to fix properly (need to marshall the item grid, or perhaps item order) 11:30:48 -!- wwwn is now known as kekmaster_one 11:30:58 -!- kekmaster_one is now known as wwwn 11:31:12 !source _change_walls_from_centre 11:31:12 1/1. https://github.com/crawl/crawl/blob/master/crawl-ref/source/dgn-labyrinth.cc#L167 11:31:27 some bad va_args stuff here (I think my xcode upgraded and I'm seeing new warnings) 11:43:00 Cold turkey! 11:43:00 dpeg: You have 12 messages. Use !messages to read them. 11:45:30 scroll of brand weapon comes with icon of scroll of enchant weapon 13https://crawl.develz.org/mantis/view.php?id=11290 by Majang 11:58:07 wow, look who's back! 12:00:19 whoa, hi dpeg! 12:02:52 advil: Hi there! :) 12:03:08 gammafunk: It's me, the pernicious dpeg. 12:03:54 I guess we should have known that our german firewall wasn't holding when nap.kin kept sneaking through as well 12:04:33 gammafunk: but it works a little bit: there was a time when three developers came from Berlin 12:04:45 oh, true 12:05:07 Of course, the real question is: Is Crawl great again? 12:05:47 with wand charge merging? we have yuge design...the best you've ever seen 12:05:50 (I get the commit mails and read them: good work!) 12:08:10 yes, we're planning for a release/tourney 1st week of january 12:08:20 and I'm making a "trunk update" post today 12:08:48 %git :/travel 12:08:49 07Aidan Holm02 {amalloy} * 0.21-a0-481-ge9ed347: Don't (auto)travel via stairs into exclusions 10(5 weeks ago, 3 files, 37+ 0-) 13https://github.com/crawl/crawl/commit/e9ed347cad6f 12:09:10 gammafunk: ^ could be good to add to changelog/trunk update as well? i'm not sure what level of detail is expected in those 12:09:11 1st week of January is very good: no teaching for me :) 12:09:25 ah, you can take advil's place... 12:09:43 hi dpeg 12:09:49 Hallo amalloy 12:09:56 amalloy: well, that seems more just a basic bug 12:10:25 we try to put in notable features (and not even all changes that effect gameplay if the effect is minor) 12:10:34 but there has been debate about what is changelog worthy in the past 12:10:47 but I will include it in the trunk update post along with a witty joke 12:10:54 since for that everything is fair game 12:11:00 hah 12:12:22 dpeg! 12:13:22 I was actually just writing to jpeg to make sure you weren't dead 12:14:09 bh: I am not sure jpeg would know if I'm dead :) 12:14:23 and anyway she doesn't have resurrection powers so she couldn't do anything about it 12:14:48 dpeg: have you got a copy of the source code to the book of the dead? I only have fragments of it 12:14:58 03|amethyst02 07* 0.21-a0-485-g2875ce1: Checkwhite. 10(14 minutes ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/2875ce172d4d 12:14:58 03|amethyst02 07* 0.21-a0-486-g0e05f1c: Remove a varargs function and simplify (advil) 10(24 minutes ago, 1 file, 14+ 49-) 13https://github.com/crawl/crawl/commit/0e05f1cc88cb 12:15:52 The var args hunter needed only the most subtle hint to catch his prey..... 12:16:19 heh I thought that might trigger |amethyst 12:16:29 bh: that's one of the reasons why I am here. I cannot log into angband.dis.pl anymore :( 12:16:30 <|amethyst> va_args? Not on my watch! 12:16:46 in that case the non-varargs function even already existed 12:17:51 gammafunk: are you planning to fix the ijyb/maurice thing? i don't know that i'll be able to do any of the struct wand_def stuff |amethyst and i were talking about to simplify it before i go on vacation 12:17:59 oh, sure 12:18:04 I actually assumed someone had already fixed it 12:18:16 I will probably not do any big refactoring though 12:18:39 sure, that's fine. leaves more big refactoring for me 12:20:19 dpeg: christ. this thing has been unfinished for four years 12:20:25 bh: slacking like the best of 'em! 12:20:51 bh: this is one of the (not so many) things that can make me cringe when I close my eyes. Of course, I simply stopped closing my eyes... 12:21:29 @?ijyb 12:21:29 Ijyb (02g) | Spd: 10 | HD: 3 | HP: 19-27 | AC/EV: 7/11 | Dam: 4 | 10weapons, 10items, 10doors | Res: 06magic(10) | XP: 36 | Sz: small | Int: human. 12:22:45 bh: let me look in some more distant and dusty places of my file systems... 12:23:17 oh my, a wild dpeg appears 12:23:17 oh, I just recovered the harddrive from my water damaged laptop, I'll check that 12:24:55 Unstable branch on crawl.akrasiac.org updated to: 0.21-a0-484-gcd31912 (34) 12:24:56 hrm 12:25:01 !source monster::likes_wand 12:25:02 1/1. https://github.com/crawl/crawl/blob/master/crawl-ref/source/monster.cc#L777 12:25:34 why on earth do I have a font called "webdings" that seems to be spiders and pirates 12:25:45 bh: wingdings made webscale 12:25:51 so I see that comment has some faulty math even with the old values (divided by 3) for wand_charge_value() 12:25:53 post-browser, pre-emoji 12:26:04 looks like max of HD 9 would use value 5 wands 12:26:29 *old value 5 12:26:40 @?maurice 12:26:41 Maurice (03@) | Spd: 10 | HD: 5 | HP: 51-70 | AC/EV: 2/13 | Dam: 910(steal) | 10weapons, 10items, 10doors, spellcaster | Res: 06magic(20) | XP: 469 | Sp: sprint, invisibility, blink, teleport self [04emergency] | Sz: Medium | Int: human. 12:26:44 hrmph. yeah, nothing on my old laptop or on my server 12:26:58 I guess this doesn't mean too much in practice, but I wonder if it just became out of date at some point 12:29:56 gammafunk: a part of the natural lifecycle of a comment 12:30:29 oh hi dpeg, bh 12:30:43 -!- mumra1 is now known as mumra 12:31:05 mumra: Hi! I'll be around more often now. Again, that is. 12:31:54 found it 12:32:00 we'd put the source code on gitorious 12:32:11 when github bought them, they shut down in 2015 12:32:34 i've been lurking a bit again lately 12:32:45 Krankykoala (L1 DrAr) (D:1) 12:34:38 !crashlog 12:34:51 18117. Krankykoala, XL1 DrAr, T:116 (milestone): http://crawl.akrasiac.org/rawdata/Krankykoala/crash-Krankykoala-20171122-173243.txt 12:35:55 @?erica 12:35:55 Erica (06@) | Spd: 10 | HD: 9 | HP: 48-75 | AC/EV: 2/11 | Dam: 10 | 10weapons, 10items, 10doors, spellcaster | Res: 06magic(60) | XP: 923 | Sp: b.venom (3d14), mystic blast (3d14), confuse, slow, invisibility | Sz: Medium | Int: human. 12:35:55 @?rupert 12:35:55 Rupert (04@) | Spd: 10 | HD: 16 | HP: 98-138 | AC/EV: 6/8 | Dam: 21 | 10weapons, 10items, 10doors, see invisible | Res: 06magic(100) | XP: 2914 | Sp: paralyse, confuse, berserker rage [06!sil] | Sz: Medium | Int: human. 12:35:55 @?asterion 12:35:55 Asterion (12H) | Spd: 10 | HD: 15 | HP: 102-146 | AC/EV: 10/1 04(headbutt: d20-1) | Dam: 35, 20 | natural, 10weapons, 10items, 10doors, priest, evil, see invisible | Res: 06magic(100) | Vul: 11silver | XP: 2649 | Sp: major destruction (3d6 / 3d27 / 3d20 / 3d24 / 3d28 / 3d25 / 3d22) [11!AM], g.s.makhleb [11!AM], haste [11!AM] | Sz: Medium | Int: human. 12:36:03 @?urug 12:36:04 Urug (05o) | Spd: 10 | HD: 9 | HP: 55-81 | AC/EV: 8/11 | Dam: 25 | 10weapons, 10items, 10doors, fighter | Res: 06magic(40) | XP: 834 | Sz: Medium | Int: human. 12:36:13 oh rip. that PR i merged has a crash in it 12:36:54 <|amethyst> amalloy: the exclusion one? 12:38:03 yeah 12:38:26 another commit for you! 12:38:33 03amalloy02 07* 0.21-a0-487-ga335af4: Revert "Don't (auto)travel via stairs into exclusions" 10(71 seconds ago, 3 files, 0+ 37-) 13https://github.com/crawl/crawl/commit/a335af4bc1dc 12:38:49 gammafunk: *two* more commits! 12:38:58 one revert, and then some happy day in the future, a fix 12:39:12 <|amethyst> what's the crash? 12:39:16 !crashlog 12:39:18 18117. Krankykoala, XL1 DrAr, T:116 (milestone): http://crawl.akrasiac.org/rawdata/Krankykoala/crash-Krankykoala-20171122-173243.txt 12:39:25 |amethyst: i haven't investigated yet, just saw that code in the backtrace 12:42:30 <|amethyst> oh, looks like a simple fix 12:43:02 irqconflict (L2 OgWr) (D:1) 12:44:33 irqconflict (L2 OgWr) (D:1) 12:44:57 ??cbro 12:44:57 cbro[1/4]: Server in Georgia, US: crawl.beRotato.org -- ssh port 22, username crawler, keys at http://crawl.beRotato.org/crawl/keys (no password access). In case of DNS problems, use crawl.boylecraft.net. Webtiles at: http://crawl.beRotato.org. Use glyph/sigil ^ for bots 12:45:01 irqconflict (L2 OgWr) (D:1) 12:45:33 !gameinfo irqconflict 12:45:34 irqconflict the L2 OgWr^ in D:1 (cao console), T:265, defenses: 3/10/0, stats: 14/14/8 12:45:38 ??rebuild 12:45:38 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.jorgrun.rocks/rebuild/ Bug |amethyst or Nap.Kin for CDO. Use your powers wisely. 12:45:57 i'll do a trunk rebuild 12:47:06 <|amethyst> I'm about to push a fix I think 12:47:16 <|amethyst> hmm 12:47:27 <|amethyst> maybe not, go ahead :) 12:47:31 i already did 12:48:16 that's one good habit from SRE: mitigate first, don't count on being able to fix it quickly even if it looks easy 12:49:05 <|amethyst> can you un-close a PR? 12:49:15 good question, I've never tried 12:49:37 you can't 12:49:43 you can still comment on closed PRs 12:50:08 argh, i closed the tab that was doing the cao rebuild. does that make the rebuild stop? 12:50:31 <|amethyst> yes 12:50:36 you can reopen one I think 12:50:43 <|amethyst> hm 12:50:48 <|amethyst> s/yes/I think?/ 12:50:54 FR: don't make it stop the rebuild 12:51:09 or make it warn when closing the tab 12:51:23 well, at least the .o files are still there 12:53:20 3ajku1 (L9 HaWr) (D:6) 12:54:17 (reopen a PR, that is, not a tab) 12:54:24 <|amethyst> is level_info in some weird coordinate system 12:54:25 <|amethyst> ? 12:54:44 <|amethyst> oh, never mind, I see 12:55:50 Unstable branch on crawl.akrasiac.org updated to: 0.21-a0-487-ga335af4 (34) 12:57:01 giant ogre on D:4 -- quality as always! 12:57:46 fr giant ogre 12:57:57 the red ones :) 12:58:59 ?: says it was a two-headed ogre! Am I supposed to read the words on the screen? 12:59:35 Drakenburg (L7 GnFi) (D:3) 12:59:37 @?maurice 12:59:37 Maurice (03@) | Spd: 10 | HD: 5 | HP: 51-70 | AC/EV: 2/13 | Dam: 910(steal) | 10weapons, 10items, 10doors, spellcaster | Res: 06magic(20) | XP: 469 | Sp: sprint, invisibility, blink, teleport self [04emergency] | Sz: Medium | Int: human. 13:00:34 dpeg: no, misreadings lead to lovely feature requests 13:00:49 !gameinfo drakenburg 13:00:50 Drakenburg the L7 GnFi^ in D:3 (cao console), T:2592, defenses: 10/11/0, stats: 18/12/15 13:00:56 !versions 13:01:07 <|amethyst> sthm 13:01:09 <|amethyst> hm 13:01:27 <|amethyst> okay, I have a fix, but noticed something slightly suboptimal about the behaviour 13:01:32 03gammafunk02 07* 0.21-a0-488-gc317228: Fix monster wand assessment (saltwaterterrapin) 10(22 minutes ago, 1 file, 5+ 4-) 13https://github.com/crawl/crawl/commit/c317228d68f1 13:02:00 <|amethyst> or maybe not, I guess it's close to the non-stair case 13:02:39 <|amethyst> but if you try interlevel travelling somewhere and the only paths would go through stairs whose other end is excluded 13:03:26 <|amethyst> you just get "Sorry, I don't know how to get there" without any indication of where the exclusion is 13:03:45 <|amethyst> but I see now that that happens for exclusions blocking within-level travel as well, so is nothing new 13:04:48 yeah 13:05:46 <|amethyst> hmm 13:06:27 <|amethyst> it's also weird that single-tile exclusions on stairs block taking the stairs from that direction but not the other 13:06:52 <|amethyst> but I guess that's for people who are used to blocking stairs in that way 13:08:13 yeah 13:08:21 <|amethyst> hm, what's the proper way to do this 13:08:27 <|amethyst> I could revert then commit my changes 13:08:27 that's almost the only use case for single-tile exclusions anyway 13:08:35 |amethyst: revert what? 13:08:35 <|amethyst> but that leaves another broken version in history 13:08:38 oh 13:08:41 <|amethyst> amalloy: the reversion 13:08:51 <|amethyst> *or* I can revert and apply my patch at the same time 13:08:56 <|amethyst> but that's not really a revert 13:09:34 i'd frame yours as not a revert. checkout the version before my revert, apply your changes, then reset --soft to origin/master 13:09:40 then add/commit 13:12:28 <|amethyst> I did revert then rebase -i instead, but to the same effect 13:12:42 sure 13:13:32 <|amethyst> hm 13:13:48 <|amethyst> and commit --amend so the author is correct 13:15:08 hm, how did you get the author wrong? you weren't modifying any previous commit, so every commit should be (correctly) authored by you 13:15:31 what if: when a stairs gets secluded, automatically create a single-tile exclusion on the stairs at the other end? 13:15:39 <|amethyst> I want it to be authored by Aidan 13:15:51 s/secluded/excluded 13:15:52 |amethyst: i disagree. he has credit for the original change, which he made 13:15:53 <|amethyst> mumra: difficult to do if you haven't visited the opposite level 13:16:04 <|amethyst> amalloy: I mean, this commit is almost entirely his code 13:16:06 the fix wasn't written by him, even though he surely could have figured it out if he'd been around 13:16:33 hm, i see. i guess since it's a single commit to un-revert as well as fix it's not as clear as i made it sound 13:16:37 when an upstairs is generated, check the opposite stair to see if it has an exclusion already 13:16:39 your call i guess 13:18:14 03|amethyst02 07* 0.21-a0-489-g4f54bde: Remove another varargs function. 10(46 minutes ago, 1 file, 188+ 214-) 13https://github.com/crawl/crawl/commit/4f54bde7880d 13:18:14 03Aidan Holm02 {|amethyst} 07* 0.21-a0-490-g87dc6a9: Don't (auto)travel via stairs into exclusions 10(5 weeks ago, 3 files, 47+ 0-) 13https://github.com/crawl/crawl/commit/87dc6a9e9dbd 13:19:08 <|amethyst> Hm 13:19:33 <|amethyst> yeah, now I think maybe I should have done a revert + fix so the changes would be more obvious 13:19:52 <|amethyst> ah well, the tree is the same anyway 13:20:56 <|amethyst> I'm kind of surprised travis didn't catch it 13:21:06 <|amethyst> do none of our tests involve taking stairs? 13:21:52 |amethyst: i don't think every stair causes it. we'd see a lot more crashes 13:22:03 but i honestly know nothing about our tests. i guess they exist but i don't count on them to catch anything at all 13:22:09 <|amethyst> amalloy: not a lot of servers have updated 13:22:20 right but even on cao i watched a player whose game had crashed 13:22:26 and he went through some stairs 13:22:26 <|amethyst> amalloy: in my tests, taking a stair to a brand-new level always crashed 13:22:32 hm 13:23:13 <|amethyst> I guess "unvisited stairs" (in my commit message) was not quite strong enough, it's specifically to an unvisited level 13:23:37 Kenreth (L8 VpEn) (D:7) 13:23:54 <|amethyst> !crashlog 13:23:55 18130. Kenreth, XL8 VpEn, T:4862 (milestone): http://crawl.akrasiac.org/rawdata/Kenreth/crash-Kenreth-20171122-182337.txt 13:24:05 !lg . muwz ktyp=lava 13:24:06 No games for bh (muwz ktyp=lava). 13:24:11 !lg * muwz ktyp=lava 13:24:20 7. ImGanon the Hellbinder (L27 MuWz of Sif Muna), turned to ash by lava in the Abyss on 2012-11-03 22:32:42, with 743187 points after 120570 turns and 12:28:08. 13:24:56 <|amethyst> hmm 13:25:07 !lm kenreth x=v 13:25:07 <|amethyst> in Kenreth's case it was on *leaving* temple 13:25:08 5088. [2017-11-22 18:23:37] [v=0.21.0-a0] Kenreth the Covert (L8 VpEn of Dithmenos) (D:7) 13:25:22 !lg * muwz ktyp=lava min=xl 13:25:23 7. heteroy the Magician (L1 MuWz), turned to ash by lava on D:1 on 2007-11-25 10:02:13, with 39 points after 60 turns and 0:00:28. 13:25:26 |amethyst: you still call *si without checking it 13:25:33 !lg * muwz ktyp=lava min=xl log 13:25:34 7. heteroy, XL1 MuWz, T:60: http://crawl.akrasiac.org/rawdata/heteroy/morgue-heteroy-20071125-050213.txt 13:25:35 !lm kenreth x=vlong 13:25:36 5088. [2017-11-22 18:23:37] [vlong=0.21-a0-484-gcd31912] Kenreth the Covert (L8 VpEn of Dithmenos) (D:7) 13:25:40 <|amethyst> amalloy: where? 13:25:49 https://github.com/crawl/crawl/commit/87dc6a9e9dbd#diff-379a39a03900cbbee78c399131a460d9R84 13:26:06 <|amethyst> amalloy: that's inside if (si) 13:26:11 <|amethyst> well, if (declaration of si) 13:26:27 oh, i see. i missed that because i'm not used to declaring things inside if/etc expressions 13:26:42 (why i missed that but didn't miss the one before it is a mystery) 13:28:15 <|amethyst> Anyway, CPO and CJR people can test this one before the other servers update tonigh :) 13:29:03 yeah 13:32:05 First shop I see in months: Plog's Antique Weapon Shop. Excellent, Crawl! 13:33:15 hm, yeah, it's annoying that the tests wouldn't catch that 13:33:31 it might be that all level changes in the tests involve just level change lua or wizmode 13:33:41 &d etc 13:35:27 <|amethyst> yeah, I think they all use &~ 13:38:30 easy to add &~t< to one of them or something 13:42:25 <|amethyst> or &{X>T> from D:1 (after clearing monsters from the current level I suppose) 13:43:09 <|amethyst> hm, I guess that might take a hatch instead of a stair 13:44:05 <|amethyst> (both would have exhibited the problem in this case, so that's not *terrible*) 13:44:41 FR: being able to set skill breakpoint at the moment manual dusts. 13:45:48 <|amethyst> what does "identify what target for all wands you pick up" mean? 13:49:22 My FoFi just run into the little Nemelex transporter vault. After defeating the crystal golem, the second transporter got me back to the altar (not to the loot room). Is this intended? (Current game on CDO.) 13:52:00 <|amethyst> dpeg: It is intended (you gamble and get either loot or a monster), but that might not be a terrible change 13:52:03 <|amethyst> ohh 13:52:07 <|amethyst> gammafunk broke the vault 13:52:13 <|amethyst> it's not much of a gamble at all now 13:52:30 no? 13:52:38 I'm fine either way, just wanted to know if it works as intended. 13:52:38 <|amethyst> gammafunk: you see the destination 13:52:46 sounds like it's working like intended 13:52:54 you chose one room, didn't get the loot room 13:53:01 you got the punishment room 13:53:10 <|amethyst> gammafunk: originally you didn't know which room you would get until you enter the transporter 13:53:19 gammafunk: yes, I knew in advance... 13:53:25 <|amethyst> gammafunk: now you see the landing site and know whether you'll get the monster or the loot 13:53:37 oh, sure 13:53:54 all you have to do to fix this 13:54:01 is place a landing site feature in both 13:54:09 |amethyst: where's the "identify what target..." text from? 13:54:30 <|amethyst> advil: salty reddit comment: https://www.reddit.com/r/dcss/comments/7elrbn/ijyb_had_no_wand/dq6tb24/ 13:54:47 yeah that comment was pretty funny 13:54:57 or that poster rather; pretty half-hearted troll attempts 13:55:08 I appreciate the more subtle trolls 13:55:27 they were referring to that identify scroll bug 13:55:32 where wands showed up in the listing 13:55:44 even though wands should not be targets of the scroll 13:55:49 since they have their charges known 13:55:54 but that was fixed some commits back 13:55:58 that is the same user who was freaking out in the other thread, with "Major bug in new change, identify still brings up wands to be identified." 13:56:11 yeah I think they're just referring to that again 13:56:12 they're probably referring to that 13:58:49 gammafunk: on the other hand a landing site feature that doesn't have any transporter leading to it may lead to players frantically searching the whole floor for the teleporter that *must* lead to the loot 13:59:05 heh, that's possible 13:59:06 but 13:59:11 that can be fixed with the lua marker 13:59:18 *lua marker function 13:59:22 so yeah, I'll look into improving that 13:59:33 it will simply remove the landing site feature in the cleanup function 13:59:40 ah, sounds good to me 14:00:18 good opportunity for ridiculous flavour text: "As you step onto the transporter, the dual waveform collapses into a single transporter" 14:00:40 <|amethyst> I would complain about anachronism... 14:00:47 <|amethyst> but we have a freaking moon base now, so 14:01:20 the moon always existed! a wizard on the moon made a magic base for an acid troll 14:01:38 nbd 14:01:55 is it an acid troll now? 14:02:03 it's a moon troll that uses acid 14:02:12 |amethyst: Crawl has a knack for pseudo science, a la "metabolic englaciation" etc. 14:02:26 dpeg: we could add homeopathic potions 14:02:43 that's already a nethack feature, essentially... 14:03:27 <|amethyst> gammafunk: you can mix healing and water to get two potions of extra healing? 14:03:52 uh...I'd have to look on the wiki... 14:04:19 I think there are some results from nethack alchemy that would imply something about homeopathy is true... 14:11:31 |amethyst: lol 14:12:06 I might suggest that as a feature for some of the sillier variants 14:12:35 add a Homeopath role 14:12:50 <|amethyst> bhaak: probably it should be only one potion as the result rather than two 14:15:42 03|amethyst02 07* 0.21-a0-491-ga0408e1: Make Nemelex's gamble a gamble again. 10(7 minutes ago, 1 file, 1+ 0-) 13https://github.com/crawl/crawl/commit/a0408e1856a4 14:16:07 <|amethyst> ^^ I did not implement the "can be fixed with the lua marker function" 14:16:14 <|amethyst> so have fun with that, gf :) 14:17:18 <|amethyst> hm 14:18:16 <|amethyst> also, ideally I think the transporter out (N and O) would only appear on the side that you entered, not the other side 14:23:41 |amethyst: what do you mean? i think that already happens, if i understand you correctly 14:26:15 Typo in job creation tutorial 13https://crawl.develz.org/mantis/view.php?id=11291 by Poroing 14:27:13 <|amethyst> amalloy: right now, when you take the entry transporter, both exit transporters appear 14:27:28 <|amethyst> oh wait 14:27:30 <|amethyst> no it doesn't 14:27:33 <|amethyst> aha 14:27:54 <|amethyst> sorry, that was my bad, I teleported into the other side in my test 14:28:06 <|amethyst> err 14:28:11 <|amethyst> no, I'm not sure how it happened 14:28:29 <|amethyst> so never mind me 14:36:58 don't worry, lua marker functions are barely hacky black magic at all 14:37:11 they're almost entirely possibly sane 14:37:38 <|amethyst> are you working on it right now? 14:38:08 oh sure, I can take a look 14:38:21 but if you wanted to, by all means :) 14:38:28 <|amethyst> no, no, go right ahead 14:38:30 you did say "have fun with that" so I assumed not 14:38:34 <|amethyst> I'll learn from the master 14:38:40 <|amethyst> well, from you anyway 14:38:47 I was going to say 14:38:53 "will one be stopping by at some point?" 14:38:58 <|amethyst> !seen grunt 14:38:59 I last saw Grunt at Thu Aug 17 04:50:52 2017 UTC (13w 6d 14h 48m 6s ago) parting ##crawl-dev, saying 'chanpart'. 14:39:06 don't summon him... 14:39:52 <|amethyst> gammafunk: so, just to clarify 14:40:05 <|amethyst> gammafunk: I assume this {{ }} code, because of its position, runs before the shuffle happens? 14:40:19 <|amethyst> gammafunk: because otherwise I can't see how it would give the correct message 14:40:45 yes, that should be the case 14:40:55 <|amethyst> ah, but the MARKER: is after the shuffle 14:41:14 <|amethyst> okay, that part makes sense to me now 14:41:20 the transporter marker is, yeah 14:43:13 <|amethyst> oh, the time I got both exit transporters might have involved placing the vault twice on the same level 14:44:13 <|amethyst> because I infer from those MARKER:s that transporters and destinations are linked by that "the_gamble_entry" parameter being the same, and if you have two of each on the level that leads to fun 14:44:14 ah, yeah, if you were using &L that should at least place transporters initially correctly, but 14:44:17 yes 14:45:03 <|amethyst> hm 14:46:40 <|amethyst> FR: tab-completion on the "Enter map name:" prompt 14:47:02 <|amethyst> (and a lot of other wizmode prompts) 14:47:09 hmm...I see what looks like a bug 14:47:23 oh nm 14:47:34 dang, I need to move this save compat code elsewhere 14:48:04 <|amethyst> any thoughts about spreading the loot out in the prize room? 14:48:36 |amethyst: tab completion would be nice, but at least ^p (or up-arrow) works 14:48:55 <|amethyst> amalloy: yeah, until you save, edit the vault, and restart :) 14:48:56 oh, sure, that would be fine 14:49:07 |amethyst: paste works 14:49:14 where do we put are save compat lua 14:50:13 <|amethyst> gammafunk: dat/des/variable/compat.des, except for temple compatibility maps 14:50:20 cool 14:50:30 now I had noticed that this code seemed not to work when I moved it to this file 14:50:44 namely it seemed to somehow not be available unless it remained in its original position 14:50:48 but I never looked into why 14:51:03 this was seen when testing with saves before/after changing vaults like these 14:51:05 <|amethyst> local variables? 14:51:12 <|amethyst> err, not local, but file-scope 14:51:18 <|amethyst> or whatever it is 14:51:18 yeah, maybe? 14:51:35 but the variables involved aren't local 14:51:41 hrm 14:51:54 function callback.grunt_nemelex_gamble_start(data,triggerable,triggerer,marker,ev) 14:52:14 this shouldn't be file local, right? 14:52:16 it wasn't finding the save compat functions like these 14:52:36 I'm going to move this code to compat, since even if it doesn't work the odds of a save having it and transfering are pretty slim 14:54:44 <|amethyst> yeah, I don't see anything there that would be referring to such a variable 14:55:00 <|amethyst> everything's either a parameter, a local (slave), or dgn.persist 14:55:30 <|amethyst> I think it would be a problem if you defined a variable in the {{ }} block and used that 14:56:00 maybe sometime I can try to test that code again when it's in compat.des 14:58:55 ??dobrazupa 14:58:55 dobrazupa[1/2]: Good soup! Hosts ttyrecs, morgues, etc. for crawl.s-z.org . 14:59:03 is dobrazupa dead? 14:59:05 <|amethyst> oh yeah, I still haven't done that 14:59:12 <|amethyst> let me do that now 14:59:52 <|amethyst> "that" = "file an issue with the ISP because it went down around the time of Irma and didn't come back up 15:00:32 I really like the Elf monster set. 15:00:55 |amethyst: ouch 15:03:07 cool, will be nice to cszo back 15:03:22 s/cszo/dobrazupa/ 15:03:31 ok, so let's see if this actually works... 15:05:36 also, this vault could just use stone 15:05:59 I mean, if someone cheats using LRD..well, nemelex can appreciate a good cheater, no? 15:06:59 <|amethyst> gammafunk: but that means people who expect they might learn LRD need to set a level annotation so they remember to come back to it 15:07:13 <|amethyst> not that that isn't a problem with lots of vaults currently 15:07:17 yeah 15:07:22 I think it will already do that for them, no? 15:07:26 transporters make annotations 15:07:50 not an accurate annotation for what they want, I guess 15:08:00 <|amethyst> yeah, that doesn't distinguish it from other transporter vaults 15:08:21 eventually we probably should see about making permarock not actually be a thing, if we can 15:08:28 but I guess we do still have the hall_of_zot to figure out 15:08:41 <|amethyst> I would go in the other direction 15:08:44 <|amethyst> err 15:08:50 <|amethyst> not "the" other direction 15:09:07 <|amethyst> I would make sure level borders are permarock, not plain rock that just happens to not be diggable 15:09:21 yes, that certainly would be fine 15:09:54 well this works 15:10:08 I'll also randomize the loot placement 15:14:44 <|amethyst> okay, I filed the ticket with my ISP 15:15:46 oh, no no_item_gen tag 15:17:12 lol 15:17:30 good stuff, the 8 and the 9 in one instance of the vault rolled nothing :/ 15:17:45 we really need to fix that population list having NO_MONSTER thing 15:21:05 -!- Marvin is now known as Guest8413 15:21:55 The build passed. (master - a0408e1 #8956 : Neil Moore): https://travis-ci.org/crawl/crawl/builds/305967440 15:26:16 I wish travis would give the build a grade 15:26:24 passed with what? C+? 15:26:43 C++ I hope 15:27:13 gammafunk: re 8 and 9 monster: yes, no monster is better in the vault definition 15:27:32 sorry? 15:28:38 you mean no monster should be rolled only when explicitely indicated in the vault def? 15:28:44 if so, yeah, that's my complaint 15:29:16 8 and 9 can still roll something very trivial, so even that doesn't fix the problem fully, but no monster is certainly a bit of a let down 15:32:58 gammafunk: yes. If you want a random monster placement to possibly give no monster, that should be done in the MAP definition with 8. or somesuch. 15:33:33 we just have to go through and edit the monster population lists 15:33:41 03advil02 07* 0.21-a0-492-gef8ecb5: Refactor high-DPI code 10(2 hours ago, 11 files, 138+ 68-) 13https://github.com/crawl/crawl/commit/ef8ecb54598b 15:33:41 03advil02 07* 0.21-a0-493-g22212c1: Correctly handle dragging between displays with different DPI 10(49 seconds ago, 8 files, 106+ 44-) 13https://github.com/crawl/crawl/commit/22212c17e2ad 15:33:50 to not have entries with MONS_NO_MONSTER 15:33:50 and then figure out what we want to distribute their weight to 15:34:06 dang, advil beat me to a push 15:35:33 "Yes, I realize I'm insane." 15:35:39 <|amethyst> the reason NO_MONSTER is there at all is that those were originally very weak monsters 15:35:49 03gammafunk02 07* 0.21-a0-494-g4b1fdcf: Move some vault save compat code to compat.des 10(17 minutes ago, 2 files, 52+ 53-) 13https://github.com/crawl/crawl/commit/4b1fdcfa0c13 15:35:49 03gammafunk02 07* 0.21-a0-495-g7c12b18: Clean up item placement and generation in a vault (|amethyst) 10(15 minutes ago, 1 file, 10+ 8-) 13https://github.com/crawl/crawl/commit/7c12b189b42c 15:35:49 03gammafunk02 07* 0.21-a0-496-g696f58f: Clean up transporter landing sites in a vault (amalloy) 10(15 minutes ago, 1 file, 10+ 4-) 13https://github.com/crawl/crawl/commit/696f58f53759 15:36:03 <|amethyst> and it was thought that replacing them with another monster would just mean more XP for the player 15:36:19 well, maybe we need to have monster numbers depend on depth/branch 15:36:28 in a way we can control more 15:37:16 <|amethyst> yeah, though I wouldn't want to try reproducing the current numbers 15:37:24 <|amethyst> because they're extremely ad-hoc anyway 16:17:34 Uskayaw's Grand Finale can hurt allies with flying chunks 13https://crawl.develz.org/mantis/view.php?id=11292 by Lavandula 16:19:26 <|amethyst> gammafunk: (looking over commits now) oh, I'd have removed both landing sites 16:19:41 <|amethyst> gammafunk: which I think might be a little simpler? 16:20:50 it's not simpler if you have to look up the pos with a marker 16:21:15 but yeah you could remove more stuff 16:21:27 I think it was more important for the unvisited area 16:21:47 the only other features that remain now are ones in areas the player has actually already been ot 16:21:50 *to 16:21:50 |amethyst: 16:23:02 but yeah, you probably mean just look up both prop markers and remove both sites instead of using an argument 16:23:24 that would be fine, but the main issue was just there was this unvisited area with a landing site 16:23:44 however I suppose the timer could expire and they'd both be unvisited 16:24:10 hence I suppose you'd need to have the exiration function handle this as well 16:24:13 *expiration 16:57:19 -!- KamiKatze_ is now known as KamiKatze 17:02:03 <|amethyst> CSZO is back up 17:04:30 \o/ 17:04:40 -!- |amethys1 is now known as |amethyst 17:12:38 |amethyst: awesome 17:13:22 |amethyst: czso the actual crawl server, or dobrazupa? i thought you decommissioned cszo on purpose and we just wanted dobrazupa back 17:21:10 I think he just means the apache portion 17:21:11 not webtiles 17:38:05 Is anyone here great at qemu+mounted folder permissions? I'm struggling to find a solution. 17:40:45 birkhead (L1 DsMo) (D:1) 17:40:45 Webtiles server stopped. 17:40:45 Basroks (L4 DsWz) (D:3) 17:40:45 tensugalgo (L27 NaGl) (Geh:4) 17:52:20 gammafunk: permission-issues should be resolved any second now 17:52:31 awesome, thanks 17:52:41 can't wait to watch someone die in the wizlab I modified 17:52:56 After trying to fix things using acl, I landed on just running my nginx-server as libvirt-qemu 17:52:58 lol 18:06:43