01:17:58 <Rotatell> ziggurnaut (L13 HOSu)  (Lair:5)
02:19:46 <Gretell> Unstable branch on crawl.develz.org updated to: 0.20-a0-910-gb611367 (34)
02:58:12 <Gretell> Windows builds of master branch on crawl.develz.org updated to: 0.20-a0-910-gb611367
03:53:34 <Gretell> Monster database of master branch on crawl.develz.org updated to: 0.20-a0-910-gb611367
04:43:47 -!- amalloy_ is now known as amalloy
05:37:29 -!- amalloy is now known as amalloy_
06:10:56 <Rotatell> ziggurnaut (L13 HOSu)  (Lair:5)
09:42:22 -!- TAS-2012v is now known as TAS_2012v
10:33:22 <Cheibriados> New branch created: pull/500 (1 commit) 13https://github.com/crawl/crawl/pull/500
10:33:22 <Cheibriados> 03Gregory D. Chereshnev02 07https://github.com/crawl/crawl/pull/500 * 0.20-a0-911-ga4f45dd: invert conditional 10(12 minutes ago, 1 file, 2+ 6-) 13https://github.com/crawl/crawl/commit/a4f45dd6d942
13:44:40 -!- amalloy_ is now known as amalloy
14:00:49 -!- amalloy is now known as amalloy_
14:10:17 <murtidash> so looking at the bug i ran into yesterday
14:10:29 <murtidash> with mutation.cc
14:10:39 <murtidash> while (rc == 1 && ++count < 100)         rc = _handle_conflicting_mutations(which_mut, true, reason);     ASSERT(rc == 0);
14:10:52 <murtidash> was the failure, because handle_conflicting_mutations wasn't handling the temporary mutation
14:11:15 <murtidash> if you look at line 1256 there's another call to that function
14:11:15 <murtidash> int rc = _handle_conflicting_mutations(mutat, god_gift || force_mutation,                                            reason,                                            mutclass == MUTCLASS_TEMPORARY);
14:11:36 <murtidash> and the function itself on 852 takes that 4th parameter
14:11:46 <murtidash> i think the call at 2106 just needs that 4th parameter passed to it
14:12:09 <murtidash> to handle temporary ones and ignore them and just grant the perma mutation
14:14:37 -!- amalloy_ is now known as amalloy
14:17:03 <Rotatell> murtidash (L18 DsCK) ASSERT(rc == 0) in 'mutation.cc' at line 2107 failed. (Abyss:3)
14:32:58 <Brannock> !lg * ikiller~~green_death
14:32:59 <Sequell> 979. Rrauwl the Skirmisher (L1 FoAK of Lugonu), slain by a green death on Abyss:1 on 2017-03-24 11:23:06, with 0 points after 65 turns and 0:00:32.
14:33:07 <Brannock> !lg * ikiller~~green_death xl>5
14:33:08 <Sequell> 465. Jerry the Severer (L12 DDFi of Makhleb), blasted by a green death (bolt of poison) on Abyss:2 on 2017-03-20 22:57:08, with 15316 points after 7140 turns and 0:37:19.
14:33:10 <gammafunk> murtidash_: the 4th parameter defaults to false, which would be correct
14:33:18 <gammafunk> for said call
14:33:35 <gammafunk> that parameter is to indicate to that function that a temp mutation is being performed
14:43:14 <murtidash_> nod
14:43:34 <murtidash_> i'm wild ass guessing, i dont' know the codebase that well :)
14:43:59 <murtidash_> so that being said, handle conflict is calling delete_mutation
14:44:11 <murtidash_> but delete mutation on line 1617 ignores it since it's a temporary mutation
14:46:03 <gammafunk> looks like our resolution logic is borked
14:47:05 <murtidash_> it's just passing in the mutation type, not the mutation itself right?
14:47:36 <murtidash_> i.e. you can't see inside handle_conflicting if it's trying to compare to a temp based on the two mutations it's comparing
14:48:16 <gammafunk> that's not really the problem, the problem is that the call to delete basically isn't condition on it being temporary
14:48:35 <gammafunk> the problem here is that your robust mut isn't temp
14:48:40 <gammafunk> so temp is false, as it should be
14:48:46 <gammafunk> since we're giving you a perm mutation
14:48:53 <gammafunk> but the mutation it's conflicting with is temp
14:49:01 <murtidash_> yea, but other mut is temp
14:49:35 <gammafunk> it shouldn't be trying to delete the temp at all
14:49:48 <gammafunk> this took a long time to trigger since
14:50:34 <gammafunk> hrm, well I guess you had to have innate rebust and temp frail
14:50:41 <gammafunk> but that seems like it'd have been triggered pretty quickly
14:50:52 <murtidash_> specifically i have non innate robust 2
14:50:57 <murtidash_> temp frail 3
14:51:02 <murtidash_> and i'm bein granted robust 1
14:51:05 <murtidash_> by levelling
14:51:25 <murtidash_> being granted innate robust 1
14:51:50 <murtidash_> which from what i've read through, is handled by making it permanent
14:53:16 <murtidash_> but yea, i misinterpreted that parameter as acting on the conflicting mutation, not as a value of the mutation you're being granted
14:55:43 <murtidash_> also, i'm not expecting a turn around or anything. it just interests me and i know enough c to be dangerous without actually knowing what i'm doing :P
14:57:13 <gammafunk> yeah, I can see that what's happening is that
14:57:44 <gammafunk> 1) it's incorrectly trying to delete your transient mutation and 2) the function to do the deletion won't ever delete anything if the transient mutation level is higher than the innate level
14:57:52 <gammafunk> which as you say was the case for your char
14:58:18 <gammafunk> so your char that the right configuration (transient conflict level higher than the innate level, just granted a higher innate level)
14:59:32 <gammafunk> between 1) and 2) I should add that the delete function thinks the mutation it's trying to delete is non-transient (due to not being given that arg)
14:59:36 <gammafunk> basically it's kind of a hot mess
15:00:13 <gammafunk> I think all we need to do is extend that if(temp) early return to the case when the second mutation is transient
15:01:08 <gammafunk> but I actually don't know how temp and perm mutations co-exist
15:04:48 <Gregory_> Hi guys! I recently opened a pull request for minor changes (https://github.com/crawl/crawl/pull/500/files). Does it make sense to open it again?
15:05:04 <murtidash_> the thing that confused me about that temp parameter
15:05:09 <murtidash_> is line 1255 saying
15:05:18 <murtidash_> God gifts and forced mutations clear away conflicting mutations.
15:07:05 <amalloy> Gregory_: why was it closed?
15:09:14 <amalloy> i mean, that PR seems like a reasonable one to have open
15:10:13 <rumflump> ah, the mystery of the legendary disappearing/reappearing pull request can now be solved!
15:13:52 <Gregory_> The problem was that I deleted my old account, but can't come up with a satisfactory new username :) But I think I came up with.
15:15:10 <Gregory_> And I was not sure that my patch demanded.
15:17:42 <rumflump> a good username can be generated by pushing buttons rapidly!
15:17:56 <rumflump> %git awkward
15:17:56 <Cheibriados> Could not find commit awkward (git returned 128)
15:18:09 <rumflump> %git Make Pikel
15:18:09 <Cheibriados> Could not find commit Make Pikel (git returned 128)
15:18:25 * rumflump gives up
15:18:28 <Gregory_> It's a good idea :)
15:19:19 <gammafunk> that's not a search
15:19:24 <gammafunk> to do a search you prefix with :/
15:19:30 <gammafunk> %git :/awkward
15:19:30 <Cheibriados> 07Yer mivvaggah02 {Brannock} * 0.20-a0-812-g3c07459: Make Pikel less awkward (#484) 10(3 weeks ago, 1 file, 16+ 5-) 13https://github.com/crawl/crawl/commit/3c0745970ba2
15:19:42 <gammafunk> otherwise it's looking for a branch with that name
15:19:55 <rumflump> dang, I've used that before too
15:20:04 <rumflump> not frequently enough to learn, apparently
15:20:06 <gammafunk> can also use !gitgrep to search for nth match
15:20:10 <gammafunk> if you forget that syntax
15:20:15 <gammafunk> !gitgrep 1 awkward
15:20:15 <Sequell> %git HEAD^{/awkward}
15:20:15 <Cheibriados> 07Yer mivvaggah02 {Brannock} * 0.20-a0-812-g3c07459: Make Pikel less awkward (#484) 10(3 weeks ago, 1 file, 16+ 5-) 13https://github.com/crawl/crawl/commit/3c0745970ba2
15:20:17 <Gregory_> My pull requests have become legendary? :)
15:20:19 <amalloy> the syntax i always forget is how to use gitgrep
15:20:27 <gammafunk> lol
15:20:29 <gammafunk> why?
15:20:48 <gammafunk> the syntax I always forget is the one gitgrep itself uses to search only HEAD
15:20:51 <gammafunk> that one is a monster
15:21:05 <gammafunk> esp if searching back n for n>1
15:21:09 <gammafunk> !gitgrep 3 awkward
15:21:10 <Sequell> %git HEAD^{/awkward}^^{/awkward}^^{/awkward}
15:21:10 <Cheibriados> 07MarvinPA02 * 0.20-a0-412-gcccf842: Don't allow zapping wands while confused 10(3 months ago, 1 file, 7+ 2-) 13https://github.com/crawl/crawl/commit/cccf84277269
15:21:40 <gammafunk> the :/ search I used initially will search all branches, which is usually not what people want
15:21:43 <gammafunk> but it's easy to type
15:21:48 <rumflump> well they're very spooky prs, grego. showing up in the dead of night, stealing away before the murder rooster crows winter blight.
15:21:50 <amalloy> gammafunk: parameter order
15:22:11 <gammafunk> amalloy: make a version that's a single-match search maybe?
15:22:18 <gammafunk> equiv to gitgrep 1 TERM
15:22:23 <gammafunk> !cmd gitgrep
15:22:24 <Sequell> Command: !gitgrep => .echo $(ignore $1)$(let (n $1 s $*) (concat "%git HEAD^{/" $s "}" (repeat (concat "^^{/" $s "}") (- (int $n) 1))))
15:22:31 <amalloy> but that's just :/, which is "easy" to remember
15:22:50 <gammafunk> I suppose so
15:22:52 <Gregory_> lol I'm sorry for them
15:26:44 <murtidash_> would it be as simple as something like
15:27:13 <murtidash_> if (temp || you.mutation.b.mutclass == MUTCLASS_TEMPORARY) or whatever the syntax would be
15:28:01 <murtidash_> on 893
15:45:09 <Cheibriados> 03elliptic02 07* 0.20-a0-911-gc068411: Add an option to replace the noise bar with an equipment bar. 10(30 minutes ago, 4 files, 44+ 11-) 13https://github.com/crawl/crawl/commit/c068411295f4
15:52:21 <rumflump> cool, elliptic !
16:09:30 <Jorgrell> Unstable branch on crawl.jorgrun.rocks updated to: 0.20-a0-911-gc068411 (34)
16:17:18 <Brannock> oh, interesting
16:17:20 <Brannock> that's a good option
16:22:00 <rumflump> could it be toggleable in-game, or is rejiggering the UI a dangerous endeavor?
16:22:24 <|amethyst> rumflump:  you can try it out, if you set a keybinding for the clua console
16:23:02 <|amethyst> I use (in my .rc)   bindkey = [~] CMD_LUA_CONSOLE
16:23:04 <|amethyst> then you can try
16:23:26 <|amethyst> ~crawl.setopt("equip_bar = true")
16:23:46 <|amethyst> (macros are still available on ctrl-d)
16:24:28 <rumflump> now you're playing with power
16:24:30 <|amethyst> I kind of want to give the clua console a default binding, but am torn between ~ and ctrl-d.... probably more people use ~ for macros
16:24:51 <Brannock> I use ctrl-d, I didn't know about ~
16:24:52 <|amethyst> but ~ for the console is kind of mnemonic for people who have played quake
16:24:55 <Brannock> yes
16:25:03 <Brannock> counter-strike and team fortress as well
16:25:13 <Brannock> or was that `?
16:25:19 <Brannock> same key anyway
16:25:21 <MarvinPA> those are ` i think
16:25:42 <Dracunos> console in game is great
16:25:48 <MarvinPA> which is still eternally buggy and would be fine to remove imo
16:26:03 <MarvinPA> (the current ` command that is)
16:26:33 <Cheibriados> New branch created: pull/501 (1 commit) 13https://github.com/crawl/crawl/pull/501
16:26:33 <Cheibriados> 03Gregory D. Chereshnev02 07https://github.com/crawl/crawl/pull/501 * 0.20-a0-912-g5aa625e: Invert conditional 10(5 minutes ago, 1 file, 2+ 6-) 13https://github.com/crawl/crawl/commit/5aa625edda52
16:26:35 <Dracunos> I used the other binding for macros, since it's bound twice
16:26:57 <Dracunos> there's pipe too
16:27:19 <amalloy> on the one hand i agree that ` is eternally buggy. on the other, i'm fairly sure i use it somewhat regularly anyway and would be sad if it were gone
16:27:24 <amalloy> Dracunos: | is bound
16:28:14 <gammafunk> yeah I use ` a lot in wiz mode
16:28:35 <gammafunk> sometimes it works well, sometimes it doesn't
16:29:25 <amalloy> i think i use ` for stuff like zbf or whatever
16:31:06 -!- |amethys1 is now known as |amethyst
16:32:43 <rumflump> I used ` primarily for enchant scrolls
16:32:51 <|amethyst> other commands with two or more bindings:  ./s/Del, ,/g
16:33:15 <rumflump> p is unbound now
16:33:18 <|amethyst> changing any of those is sure to make some people sad
16:33:22 <rumflump> ] is unbound last I checked
16:33:34 <|amethyst> hmm
16:33:57 <|amethyst> wonder why ] doesn't give "Unknown command."
16:34:14 <rumflump> it was always odd to me that ] wasn't jewel display next to [ being the armour display. I can never remember the actual jewel display key
16:34:23 <MarvinPA> it does for me
16:34:29 <MarvinPA> "
16:34:31 <|amethyst> "
16:34:44 <|amethyst> oh duh
16:34:49 <|amethyst> I have a macro on ]
16:35:12 <rumflump> ah, because that's the glyph?
16:35:28 <|amethyst> one of the two glyphs
16:35:34 <|amethyst> " is amulets, = rings
16:37:16 <|amethyst> 0 is one that I don't use much outside of wizmode
16:37:39 <|amethyst> since it's rare that you both want to repeat a command and know in advance how many times
16:38:12 <|amethyst> I also like the idea of making , chop corpses
16:38:18 <|amethyst> or g or whichever
16:38:26 <|amethyst> which could free up c
16:38:29 <MarvinPA> rip the one true use of 0, quaffing fountains for nutrition
16:43:12 <rumflump> circus animals had a brief controversy over whether g or , was the proper key to keep
16:47:15 <ontoclasm> what does 0 even do
16:47:21 <|amethyst> ontoclasm:  0<number><command> repeats the command <number> times
16:47:26 <ontoclasm> oh right
16:47:44 <ontoclasm> brings back memories of victory dancing
16:47:46 <|amethyst> !lg * recent !wandijyb / ckiller=ijyb
16:47:51 <Sequell> 6823/1405619 games for * (recent !wandijyb): N=6823/1405619 (0.49%)
16:47:52 <|amethyst> !lg * recent wandijyb / ckiller=ijyb
16:47:53 <Sequell> 42/4375 games for * (recent wandijyb): N=42/4375 (0.96%)
16:47:56 <ontoclasm> and hacking down bushes
16:47:58 <|amethyst> success!
16:48:12 <|amethyst> !lg * current !wandijyb / ckiller=ijyb
16:48:14 <Sequell> 2915/605652 games for * (current !wandijyb): N=2915/605652 (0.48%)
16:48:30 <|amethyst> ijybe with a wand and armour is now twice as deadly
16:48:34 <|amethyst> s/ybe/yb/
16:52:22 <Cheibriados> 03Gregory D. Chereshnev02 07https://github.com/crawl/crawl/pull/501 * 0.20-a0-913-gc84d9aa: add curly braces 10(5 minutes ago, 1 file, 2+ 0-) 13https://github.com/crawl/crawl/commit/c84d9aad4e9d
16:53:40 <amalloy> rumflump: i used to use ` for enchant scrolls, but now i just do like 4rze instead of rze```
16:54:38 <dpeg> amalloy: since when is this possible?
16:54:49 <amalloy> !lg . 1
16:54:50 <Sequell> 1/414. amalloy the Chiller (L3 FeIE), slain by a worm on D:2 on 2014-08-08 02:41:12, with 51 points after 2342 turns and 0:11:24.
16:54:58 <amalloy> dpeg: at least 2 1/2 years
16:55:03 <|amethyst> you need a 0 in front, but "forever" from my POV
16:55:20 <gammafunk> |amethyst: your numerator is too small! not statistically significant!
16:55:37 <amalloy> oh, i think i have a macro binding 4 to 04, and so on
16:55:49 <amalloy> no, i guess not, because 5 still exists
16:55:54 <|amethyst> I guess amalloy uses vikeys
16:56:30 <amalloy> yeah i guess it is 04rze
16:57:03 <Cheibriados> 03Mnoleg02 {|amethyst} 07* 0.20-a0-912-g0498d72: Invert conditional (#501) 10(10 seconds ago, 1 file, 3+ 5-) 13https://github.com/crawl/crawl/commit/0498d72a55fa
16:58:08 <ontoclasm> // TODO: invert this conditional is good crawlcode
16:59:03 <|amethyst> I was tempted to demorganize that but decided not to
17:02:20 <|amethyst> FR: unique pan lord "Demorgan" that turns other panlords into magical staves and vice versa
17:02:32 <amalloy> |amethyst: imo it should be a potion, not a pan lord
17:02:48 <|amethyst> I suggested pan lord because of Demogorgon
17:03:09 <|amethyst> should be a Xom effect
17:03:15 <amalloy> but the potion is a !
17:03:19 <|amethyst> haha
17:03:21 <amalloy> glyphs uber alles
17:06:46 <rumflump> demorganize?
17:07:06 <|amethyst> rumflump:  https://github.com/crawl/crawl/commit/6c36bf5211ad
17:07:10 <|amethyst> rumflump:  err, wrong link
17:07:11 <|amethyst> rumflump:
17:07:16 <|amethyst> https://en.wikipedia.org/wiki/De_Morgan%27s_laws
17:07:17 <rumflump> rumflump:
17:07:42 <|amethyst> lets you push ! down through && and ||
17:07:50 <rumflump> oh boy
17:07:51 <|amethyst> or push ~ down through | and &
17:09:19 <Jorgrell> Unstable branch on crawl.jorgrun.rocks updated to: 0.20-a0-912-g0498d72 (34)
17:16:22 <amalloy> i never really thought about the ~|& stuff being the same as demorgan's law
17:19:10 <|amethyst> bitfields are a boolean lattice
17:20:01 <|amethyst> In fact, any boolean algebra can be represented as a (perhaps infinite) bitfield: https://en.wikipedia.org/wiki/Stone%27s_representation_theorem_for_Boolean_algebras
17:21:18 <amalloy> speaking of conditionals that should be inverted, |amethyst, https://github.com/crawl/crawl/commit/1a4ba8038832#diff-8846ab41245b32954599989c1d25edcaR135 makes me sad. any objection to my flipping it to like bool give_wand = ...; if (!give_wand) return;?
17:22:35 <|amethyst> no objections... I guess it's a little neater (and less likely to require reformatting) than   if (give_wand) { rest of the function; }
17:23:10 <amalloy> right, i'd write it that way in my own personal style, but crawl style prefers early exits to large blocks
17:23:45 <|amethyst> do remember to re-add the () around the maurice || !once_change_in
17:23:50 <|amethyst> s/change/chance/
17:24:08 <|amethyst> I think that was the reason I didn't put this one in negation-normal form
17:24:26 <|amethyst> since I was going to need one set of parentheses either way
17:24:36 <MarvinPA> i feel like the maurice thing should go away anyway
17:24:43 <MarvinPA> or be guaranteed like ijyb maybe
17:24:46 <|amethyst> or go the Ijyb...
17:24:47 <|amethyst> yeah
17:25:05 <|amethyst> then it could become M_ALWAYS_WAND
17:26:33 <MarvinPA> true, could use that to somewhat clean up the next check too
17:26:55 <MarvinPA> in that checking ALWAYS_WAND would be neater than checking ijyb directly again
17:30:35 <|amethyst> MarvinPA:  since it wouldn't affect Maurice one way or the other
17:30:37 <|amethyst> ?
17:30:58 <MarvinPA> yeaho
17:31:03 <MarvinPA> -o
17:33:20 <|amethyst> !learn add dissertation https://www.mathematik.hu-berlin.de/~ploog/PAPERS/Ploog_phd.pdf
17:33:20 <Sequell> dissertation[1/1]: https://www.mathematik.hu-berlin.de/~ploog/PAPERS/Ploog_phd.pdf
17:33:25 <|amethyst> !learn add dissertation http://sites.krieger.jhu.edu/rawlins/wp-content/blogs.dir/28/files/2013/11/rawlins_dissertation.pdf
17:33:26 <Sequell> dissertation[2/2]: http://sites.krieger.jhu.edu/rawlins/wp-content/blogs.dir/28/files/2013/11/rawlins_dissertation.pdf
17:34:06 <|amethyst> !learn add dissertation http://uknowledge.uky.edu/cgi/viewcontent.cgi?article=1005&context=cs_etds
17:34:06 <Sequell> dissertation[3/3]: http://uknowledge.uky.edu/cgi/viewcontent.cgi?article=1005&context=cs_etds
17:37:03 <|amethyst> !tell amalloy BTW, have you seen Geoff Pullum's verse version of Turing's proof of the undecidibility of the halting problem?  http://www.lel.ed.ac.uk/~gpullum/loopsnoop.html
17:37:03 <Sequell> |amethyst: OK, I'll let amalloy know.
17:37:06 <|amethyst> err
17:37:07 <|amethyst> oop
17:37:14 <|amethyst> !tell advil BTW, have you seen Geoff Pullum's verse version of Turing's proof of the undecidibility of the halting problem?  http://www.lel.ed.ac.uk/~gpullum/loopsnoop.html
17:37:14 <Sequell> |amethyst: OK, I'll let advil know.
17:37:39 <|amethyst> amalloy:  sorry about that, stupid me for pressing tab instead of typing... though I guess you'd probably find it amusing too
17:38:30 <amalloy> so wait, _give_wand's policy is weird to me. is this an accurate summary? (and if so, why is it so weird). "only consider giving wands to unique monsters who don't have the M_NO_WAND flag. if the monster is ijyb, always give it a wand; if it's maurice, give it a wand 7/15ths of the time; for other unique monsters, give them a wand 1/5 of the time"
17:38:30 <Sequell> amalloy: You have 1 message. Use !messages to read it.
17:39:08 <|amethyst> yes, though I'm sure the "7/15" is not exactly intentional
17:42:50 <amalloy> i didn't realize uniques got wands so often (or that non-uniques get wands never)
17:47:19 <|amethyst> amalloy:  apparently it's been that way literally foreven
17:47:23 <|amethyst> s/even/ever/
17:47:33 <|amethyst> +    //mv - give wand
17:47:33 <|amethyst> +    if (mons_is_unique( menv[mid].type ) && one_chance_in(5))
17:47:38 <amalloy> a common explanation for why things are so weird i guess
17:47:47 <|amethyst> in 673bdae7 "Initial revision"
17:52:28 <Cheibriados> 03amalloy02 07* 0.20-a0-913-gd12c4a9: Clarify boolean algebra in _give_wand 10(5 minutes ago, 1 file, 8+ 9-) 13https://github.com/crawl/crawl/commit/d12c4a9f6378
17:52:28 <Cheibriados> 03amalloy02 07* 0.20-a0-914-ge5852ff: Satisfy PleasingFungus's old TODO by deduplicating 10(77 seconds ago, 1 file, 8+ 13-) 13https://github.com/crawl/crawl/commit/e5852ffc234c
18:09:22 <Jorgrell> Unstable branch on crawl.jorgrun.rocks updated to: 0.20-a0-914-ge5852ff (34)
18:30:57 <|amethyst> !learn add dissertation http://math.mit.edu/~apixton/papers/tautological.pdf
18:30:58 <Sequell> dissertation[4/4]: http://math.mit.edu/~apixton/papers/tautological.pdf
18:34:44 <travis-ci> The build was broken. (master - e5852ff #8128 : Alan Malloy): https://travis-ci.org/crawl/crawl/builds/214818091
18:35:26 <dpeg> |amethyst: is this what I think it is?
18:36:41 <dpeg> funny, this happens to be my area in maths, too :)
18:56:17 <gammafunk> did amalloy break fulldebug builds
18:56:19 <gammafunk> looks like it
18:56:49 <amalloy> fulldebug too strong
18:57:13 <amalloy> languages that require commas are too confusing
18:57:41 <|amethyst> dpeg:  I added yours too
18:57:45 <|amethyst> ??dissertation[1]
18:57:45 <Sequell> dissertation[1/4]: https://www.mathematik.hu-berlin.de/~ploog/PAPERS/Ploog_phd.pdf
18:58:16 <|amethyst> If you know anyone else on the devteam with a PhD, feel free to add to the entry
18:58:25 <|amethyst> I have you, advil, me, and elliptic
18:58:56 <Cheibriados> 03amalloy02 07* 0.20-a0-915-gb278767: Un-break FULLDEBUG builds 10(74 seconds ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/b2787677adba
18:59:25 <ontoclasm> |amethyst: give me a few years :Y
19:00:02 <|amethyst> ontoclasm:  I took 9 years, so I'm sure you'll beat me
19:01:07 <amalloy> huh. TIL if you make a yred enslaved soul out of a monster with a holy weapon, it gets converted to draining
19:01:37 <dpeg> TDTTOE!
19:02:02 <amalloy> i wonder what happens if you enslave-soul a DEBM who's wielding a holy wrath weapon in his offhand. it doesn't look like the code checks for that
19:02:26 <gammafunk> it converts the weapon to unholy wrath
19:02:54 <gammafunk> which not coincidentally is the name of my new death metal band
19:03:09 <|amethyst> FR: turn disruption into a brand
19:03:14 <dpeg> gammafunk: <3
19:04:00 <|amethyst> oh, that reminds me, I recently had a great (not death metal) quote for some spell or ability or something, but now I can't remember it
19:04:12 <|amethyst> so I guess it doesn't remind me very well
19:04:14 <Floodkiller> what happens if you enslave soul a pearl dragon?
19:04:36 <|amethyst> pearl dragon is I_ANIMAL
19:04:53 <Floodkiller> ah
19:05:07 <|amethyst> also, I think enslave soul requires MH_NATURAL
19:05:25 <Floodkiller> can't you enslave demons though?
19:05:27 <MarvinPA> it works on holies and demons now
19:05:30 <|amethyst> aha
19:05:42 <MarvinPA> holies lose any holy spells, although i forget if innate holy branding is handled
19:05:42 <|amethyst> forgot about that
19:06:33 <gammafunk> hrm
19:06:38 <gammafunk> @?spectral pearl dragon
19:06:38 <Gretell> spectral pearl dragon (03Z) | Spd: 12 | HD: 18 | HP: 91-129 | AC/EV: 12/10 | Dam: 2813(drain), 1609(claw)13(drain) | 07undead, evil, see invisible, fly, unbreathing | Res: 13magic(immune), 02cold, 09poison+++, 12drown, 04rot+++, 13neg+++, 13torm, 04napalm | Vul: 08holy | XP: 1169 | Sz: Giant | Int: brainless.
19:06:42 <amalloy> hm, the offhand holy-wrath scimitar is just destroyed, while the main-hand one converts to draining
19:06:42 <MarvinPA> oh i guess they always get draining melee brand because of the base monster
19:06:45 <gammafunk> is that what it basically turns into?
19:07:08 <MarvinPA> or since spectrals don't keep brands, rather
19:07:23 <gammafunk> @? pearl dragon
19:07:23 <Gretell> pearl dragon (08D) | Spd: 12 | HD: 18 | HP: 89-136 | AC/EV: 10/15 | Dam: 3508(holy), 2009(claw)08(holy) | 08holy, see invisible, fly | Res: 06magic(160), 12drown, 04rot, 13neg+++, 08holy+++ | XP: 3722 | Sp: holy breath (3d18) [11!AM, 06!sil, 08breath] | Sz: Giant | Int: animal.
19:07:35 <gammafunk> Int: animal to Int: brainless, poor guy
19:07:43 <amalloy> oh, it's not destroyed but dropped. interesting
19:07:45 <MarvinPA> gammafunk: if it worked on pearl dragons that's what it'd make, yeah
19:07:47 <Floodkiller> are ophans eligible?
19:08:07 <MarvinPA> except it has some special handling for spells and other things that monster doesn't know about
19:08:19 <Floodkiller> I'm trying to think of something that can generate holy using an ability
19:08:20 <|amethyst> MarvinPA:  I guess holy as in MON_SPELL_MAGICAL?
19:08:27 <gammafunk> I should try a 15 rune yred run at some point I guess
19:08:31 <|amethyst> MarvinPA:  since the SPTYP_ doesn't exist anymore
19:08:40 <Floodkiller> gammafunk: don't forget to enslave cerebov
19:08:51 <gammafunk> hrm
19:08:55 <gammafunk> self-firestorm, no thanks
19:08:57 <|amethyst> and MON_SPELL_DEMONIC (== divine/angelic) and MON_SPELL_MAGICAL were merged
19:09:22 <MarvinPA> |amethyst: there's an SPFLAG_HOLY
19:09:22 <Jorgrell> Unstable branch on crawl.jorgrun.rocks updated to: 0.20-a0-915-gb278767 (34)
19:09:22 <gammafunk> I bet enslaved mnoleg would be really hilarious
19:09:45 <gammafunk> @?mnoleg
19:09:45 <Gretell> Mnoleg (09&) | Spd: 13 | HD: 17 | HP: 290-395 | AC/EV: 10/25 | Dam: 4009(claw), 3509(mutation), 30, 2306(blink self) | 05demonic, 10doors, fighter, evil, see invisible | Res: 13magic(immune), 05fire, 10elec, 03poison, 04rot, 13neg+++, 13torm | Vul: 08holy, 11silver | XP: 10673 | Sp: b.random (3d22 / 3d20 / 3d24 / 3d21 / 3d26) [06!sil], malign gateway [06!sil], sum.eyeballs [06!sil]..
19:09:47 <|amethyst> MarvinPA:  aha
19:09:53 <|amethyst> oh, that somehow reminds me
19:09:56 <Floodkiller> do friendly summoned giant eyeballs paralyze enemies properly?
19:09:57 <gammafunk> yeah, probably really fun times
19:10:06 <gammafunk> I don't believe so, no
19:10:14 <gammafunk> I guess the eyes wouldn't do too much
19:10:18 <gammafunk> but there's always the tentacle!
19:10:20 <|amethyst> should we rename spl-*.{h,cc} to spell-*.{h,cc} ?
19:10:32 <|amethyst> maybe similarly for art-*
19:10:50 <|amethyst> and l-* to lua-*
19:11:09 <MarvinPA> art-* to artefact-*? that seems kinda long
19:11:17 <|amethyst> perhaps
19:11:34 <MarvinPA> the others seem fine, especially since all my muscle memory for file names is already recently ruined :P
19:11:42 <|amethyst> spell and lua are I hope less controversial because it's just two characters
19:14:03 <minmay> have to accomodate the DOS users
19:15:40 <Cheibriados> 03amalloy02 07* 0.20-a0-916-g1c438e7: Convert monster's alt weapons when Yred enslaves its soul 10(63 seconds ago, 1 file, 7+ 5-) 13https://github.com/crawl/crawl/commit/1c438e7a2999
19:15:40 <Cheibriados> 03amalloy02 07* 0.20-a0-917-g46c167e: Omit some redundant checks 10(32 seconds ago, 1 file, 2+ 5-) 13https://github.com/crawl/crawl/commit/46c167e02af2
19:15:57 <Lantell> Unstable branch on underhound.eu updated to: 0.20-a0-915-gb278767 (34)
19:17:00 <Cheibriados> 03MarvinPA02 07* 0.20-a0-918-g7688923: Always give Maurice a wand 10(16 minutes ago, 3 files, 11+ 7-) 13https://github.com/crawl/crawl/commit/7688923efa51
19:24:17 <Rotatell> murtidash (L18 DsCK) ASSERT(rc == 0) in 'mutation.cc' at line 2107 failed. (Abyss:3)
19:27:26 <gammafunk> that mutation issue, I haven't had time to fix
19:27:26 <|amethyst> oh, is that trunk?
19:27:27 <gammafunk> it happens because player had temp frail 3
19:27:38 <|amethyst> !crashlog
19:27:39 <Sequell> 16791. murtidash, XL18 DsCK, T:39280 (milestone): http://crawl.berotato.org/crawl/morgue/murtidash/crash-murtidash-20170324-232404.txt
19:27:40 <gammafunk> what's happening is
19:27:49 <gammafunk> he has temp frail 3, gets robust having robust 2
19:28:03 <gammafunk> the codepath tries to remove the temp mutation
19:28:24 <|amethyst> I had thought it was yiufcrawl because I know rumflump has been messing with innate mutations, but I guess that would be Bearkin not Ds
19:28:25 <gammafunk> that conditional in the mutation resolution only checks that the first mutation in the pair is a temp mut
19:28:40 <gammafunk> *the pair of the mut conflict
19:29:06 <gammafunk> bit it's being called with the second mutation as a temp mut (by order of specifications in conflicts array)
19:29:40 <gammafunk> then the code to delete mutations is being called as if the frail mut is permanent
19:29:53 <gammafunk> it shouldn't be trying to delete the temp mutation at all though
19:30:10 <gammafunk> the specific place of the crash is only triggered because his frail has higher levels than his robust
19:30:22 <gammafunk> but it shouldn't even be in that call to delete the mutation, from what I can tell
19:31:35 <gammafunk> !source mutation.cc:897
19:31:35 <Sequell> https://github.com/crawl/crawl/blob/master/crawl-ref/source/mutation.cc#L897
19:32:07 <gammafunk> it's trying to delete his temp mutation and failing to do so, then always returning 1
19:32:21 <gammafunk> and here a is robust, b is frail
19:32:50 -!- amalloy is now known as amalloy_
19:33:31 <gammafunk> and is going from delete_mutation -> _delete_single_mutation_level, but now having transient = false, when the mutation is in fact transient
19:33:50 <gammafunk> so at 1458 it's returning
19:34:00 <gammafunk> since frail 3 has more levels than robust 2
19:34:11 <gammafunk> er not that line
19:34:18 <gammafunk> 3 lines down
19:35:03 <gammafunk> but it shouldn't ever call delete_mutation in the first place afaik
19:35:13 <gammafunk> basically seems like our mutation resolution is kind of borked?
19:37:03 <|amethyst> It's not designed to handle temp mutations
19:37:59 <|amethyst> so if it's being called on them at all (which I guess makes sense bcause of conflicting temp mutations) it should only delete temp mutations, and should probably ignore innate mutations
19:38:54 <|amethyst> and should treat/be passed force as true
19:47:36 <travis-ci> The build was fixed. (master - b278767 #8129 : Alan Malloy): https://travis-ci.org/crawl/crawl/builds/214836679
19:55:09 -!- amalloy_ is now known as amalloy
19:56:55 <Cheibriados> 03|amethyst02 07* 0.20-a0-919-g6893888: Checkwhite. 10(31 minutes ago, 1 file, 1+ 1-) 13https://github.com/crawl/crawl/commit/689388829fd8
19:56:55 <Cheibriados> 03|amethyst02 07* 0.20-a0-920-gb18bc0f: Data-ify demon<->holy weapon upgrades and downgrades. 10(31 minutes ago, 1 file, 20+ 14-) 13https://github.com/crawl/crawl/commit/b18bc0fa4020
20:09:22 <Jorgrell> Unstable branch on crawl.jorgrun.rocks updated to: 0.20-a0-920-gb18bc0f (34)
20:31:02 <travis-ci> The build was fixed. (master - 46c167e #8130 : Alan Malloy): https://travis-ci.org/crawl/crawl/builds/214840692
20:57:46 -!- amalloy is now known as amalloy_
21:13:05 <travis-ci> The build was fixed. (master - 7688923 #8131 : Chris Campbell): https://travis-ci.org/crawl/crawl/builds/214840957
23:18:08 -!- md_ is now known as Guest57414
23:18:18 -!- Guest57414 is now known as murtidash_
23:45:34 <hellmonk> alexjurkiewicz: how do you feel about minotaur headbutt retaliation triggering while swimming in deep water
23:45:52 <hellmonk> I am undecided on whether or not it's worth blocking
23:54:40 <Cheibriados> 03Floodkiller02 07https://github.com/crawl/crawl/pull/487 * 0.20-a0-906-ga74cde3: Modify Cyno skill system to +4/decay by 2 at 7 to -6 10(22 minutes ago, 2 files, 19+ 34-) 13https://github.com/crawl/crawl/commit/a74cde327a4b