00:29:00 <alexjurkiewicz> !source size_type
00:29:00 <Sequell> 1/4. https://github.com/crawl/crawl/blob/master/crawl-ref/source/fixedarray.h#L21
00:29:08 <alexjurkiewicz> what
00:29:09 <alexjurkiewicz> !source size_type 2
00:29:10 <Sequell> 2/4. https://github.com/crawl/crawl/blob/master/crawl-ref/source/fixedarray.h#L21
00:29:12 <alexjurkiewicz> !source size_type 3
00:29:12 <Sequell> 3/4. https://github.com/crawl/crawl/blob/master/crawl-ref/source/fixedvector.h#L22
00:29:14 <alexjurkiewicz> !source size_type 4
00:29:14 <Sequell> 4/4. https://github.com/crawl/crawl/blob/master/crawl-ref/source/size-type.h#L4
00:50:42 <Cheibriados> New branch created: pull/589 (1 commit) 13https://github.com/crawl/crawl/pull/589
00:50:42 <Cheibriados> 03johnny002 07https://github.com/crawl/crawl/pull/589 * 0.21-a0-179-g0f305af: initfile.cc: Make sure unistd.h is included for Cygwin targets. 10(5 days ago, 1 file, 2+ 2-) 13https://github.com/crawl/crawl/commit/0f305af31222
00:55:54 <Cheibriados> 03johnstein02 07https://github.com/crawl/crawl/pull/586 * 0.21-a0-175-g0329265: Make some docstrings more consistent 10(4 hours ago, 3 files, 15+ 11-) 13https://github.com/crawl/crawl/commit/0329265dc8a9
00:55:54 <Cheibriados> 03johnstein02 07https://github.com/crawl/crawl/pull/586 * 0.21-a0-176-gafb6893: Refactor some warning msgs into a new function 10(13 minutes ago, 3 files, 25+ 6-) 13https://github.com/crawl/crawl/commit/afb68935f565
00:55:54 <Cheibriados> 03johnstein02 07https://github.com/crawl/crawl/pull/586 * 0.21-a0-177-g23b7658: Simplify and fix some formatting 10(9 minutes ago, 1 file, 3+ 3-) 13https://github.com/crawl/crawl/commit/23b76589f318
01:06:22 <Cheibriados> 03alexjurkiewicz02 07https://github.com/crawl/crawl/pull/569 * 0.21-a0-154-gda359ed: Add new vaults and a serial 10(2 minutes ago, 3 files, 333+ 0-) 13https://github.com/crawl/crawl/commit/da359edb7029
01:06:22 <Cheibriados> 03alexjurkiewicz02 07https://github.com/crawl/crawl/pull/569 * 0.21-a0-155-g587e544: Tweak decor vault visuals 10(2 minutes ago, 1 file, 6+ 6-) 13https://github.com/crawl/crawl/commit/587e544a4394
01:06:22 <Cheibriados> 03alexjurkiewicz02 07https://github.com/crawl/crawl/pull/569 * 0.21-a0-156-g8dba6ba: Various fixes (gammafunk) 10(2 minutes ago, 2 files, 21+ 35-) 13https://github.com/crawl/crawl/commit/8dba6ba772b5
01:38:08 <johnstein> !bug 10969
01:38:08 <Sequell> https://crawl.develz.org/mantis/view.php?id=10969
01:44:42 <johnstein> I was looking at this, and unless I'm doing something really wrong, in addition to inner flame, the following spells also don't seem to be triggering dith annoyance.   ignite poison
01:44:42 <johnstein> bolt of magma
01:44:42 <johnstein> ignition
01:44:42 <johnstein> firestorm
01:44:50 <johnstein> but that can't be right, can it?
01:45:30 <johnstein> I tested all the other fire spells and they are pissing dith off
01:54:52 <alexjurkiewicz> !tell hellmonk given Wz starts with an air spell, conjure flame in the starting book is too good/strong IMO
01:54:52 <Sequell> alexjurkiewicz: OK, I'll let hellmonk know.
01:56:40 <Gretell> Windows builds of master branch on crawl.develz.org updated to: 0.21-a0-178-g0b87683
02:00:49 -!- amalloy_ is now known as amalloy
02:26:38 <johnstein> yea looks like there's no check for is_spell_firey(DID_FIRE...) in _spellcasting_god_conduct
02:27:08 <johnstein> but spells like Fireball get checked and punished elsewhere in _handle_your_gods_response
02:27:56 <johnstein> so adding the is_spell_fiery(DID_FIRE...) does start punishing bolt of magma, ignition, firestorm, and inner flame
02:28:22 <johnstein> but now the spells that already punished are punishing twice, making me question whether I really understand the intent here
02:37:44 <johnstein> ??firestorm
02:37:45 <Sequell> fire storm[1/3]: L9 fire/conj spell. Deals 55% resistible fire damage in a large radius and leaves behind fire clouds and {fire vortices}. Like glaciate, it makes a godawful racket when cast. Smite targeted.
02:37:59 <johnstein> ??bolt of magma
02:38:01 <Sequell> bolt of magma[1/2]: 55% resistible (by rF) L5 Conj/Fire/Earth bolt spell. Range 5. To-hit: 8+power/25.
02:52:45 <Gretell> Monster database of master branch on crawl.develz.org updated to: 0.21-a0-178-g0b87683
02:56:01 <johnstein> looks like DID_FIRE is checked elsewhere for the other spells. like beam.cc or cloud.cc, so fireball and conjure_flame get DID_FIRE checked there. but bolt of magma, ignition, firestorm, and inner_flame don't fit any of those situations.  what's the right way to handle this?
02:58:25 <johnstein> (also, should I be rambling here? or in the mantis item? wasn't sure if advertising that dith seems to allow some fire magic is a good idea or if it really matters that much)
02:58:38 -!- amalloy is now known as amalloy_
03:06:09 <|amethyst> johnstein:  probably apply_beam_conducts could do it?
03:06:35 <johnstein> and just add extra cases for the ones missing? even if they aren't beams?
03:06:45 <|amethyst> johnstein:  bolt of magma and firestorm both use BEAM_LAVA, and inner flame uses BEAM_INNER_FLAME
03:06:52 <johnstein> ah ok
03:07:00 <johnstein> I suppose I need educated on what a beam is
03:07:22 <|amethyst> ignition uses BEAM_FIRE so should be fine?
03:07:23 <johnstein> even though firestorm is smite targeted it's considered a beam?
03:07:38 <|amethyst> it uses a beam for the explosion
03:07:54 <|amethyst> see setup_fire_storm
03:08:10 <johnstein> ah ok. ignition doesn't seem to be triggering. checked it a few times and just now again
03:09:02 <|amethyst> ah, the beam ignition uses isn't actually fired
03:09:20 <|amethyst> do_fire() is what calls apply_beam_conducts
03:10:26 <|amethyst> probably in cast_ignition after initialising beam_actual, call beam_actual.apply_beam_conducts()
03:10:53 <|amethyst> hm
03:11:20 <|amethyst> fire storm might need similar handling, since it calls bolt::explode() directly
03:11:20 <|amethyst> rather than using do_fire
03:11:37 <Rotatell> Unstable branch on crawl.beRotato.org updated to: 0.21-a0-178-g0b87683 (34)
03:11:59 <|amethyst> I guess that what lets fire storm be smite targetted
03:12:04 <|amethyst> s/that/that's/
03:14:35 <johnstein> would you expect ignite_poison should also make dith mad? seems like it should
03:20:28 <|amethyst> probably... does it already when it creates a cloud?
03:21:07 <|amethyst> hm, I guess probably not
03:21:22 <|amethyst> which is good, it means you don't have to worry about double charging
03:24:20 <johnstein> does spell failure kick in before or after the beam is initialised?
03:25:46 <|amethyst> johnstein:  ignition's failure check is at the top, "fail_check();"
03:26:18 <|amethyst> (that macro really should be all caps, since it's a macro that is not at all function-like)
03:26:25 <johnstein> |amethyst, yea, that looked good. I was trying to find the right place in cast_fire_storm
03:26:37 <johnstein> which has a slightly different setup that I'm absorbing
03:27:05 <johnstein> wondering if I do tempbeam.apply_beam_conducts() after the fail_check()
03:27:35 <|amethyst> you definitely want to do whatever after that   if (tempbeam.beam_cancelled)
03:28:02 <|amethyst> probably after the fail_check() too
03:28:57 <|amethyst> I don't think refine_for_explosion does anything that's relevant to the penalty
03:29:47 <johnstein> I tried looking for games where someone won a dith game with firestorm, but my sequell-fu failed me
03:30:12 <|amethyst> !lg * dith skill=fire_magic
03:30:14 <Sequell> 34. Tungsten the Firebug (L7 HOFE of Dithmenos), quit the game on D:4 on 2017-07-22 22:28:31, with 721 points after 6667 turns and 0:17:28.
03:30:21 <|amethyst> !lg * dith skill=fire_magic max=xl
03:30:22 <Sequell> 34. Ceann the Infernalist (L27 DEFE of Dithmenos), escaped with the Orb and 15 runes on 2015-12-06 00:01:47, with 12502077 points after 131029 turns and 13:37:57.
03:30:25 <|amethyst> !lg * dith skill=fire_magic max=sklev
03:30:27 <Sequell> 34. Ceann the Infernalist (L27 DEFE of Dithmenos), escaped with the Orb and 15 runes on 2015-12-06 00:01:47, with 12502077 points after 131029 turns and 13:37:57.
03:30:31 <|amethyst> !lg * dith skill=fire_magic max=sklev -log
03:30:33 <Sequell> 34. Ceann, XL27 DEFE, T:131029: http://dobrazupa.org/morgue/Ceann/morgue-Ceann-20151206-000147.txt
03:30:56 <|amethyst> nah, joined dith on the orb run on D:3
03:31:01 <|amethyst> !lg * dith skill=fire_magic max=sklev -2 -log
03:31:03 <Sequell> 33/34. lread714, XL8 DEFE, T:5838: https://webzook.net/soup/morgue/trunk/lread714/morgue-lread714-20161128-041411.txt
03:31:11 <johnstein> yea, that's the one I saw earlier (D:3)
03:31:25 <|amethyst> !lg * dith fifteenskills~~fire max=xl
03:31:26 <Sequell> 1. Ceann the Infernalist (L27 DEFE of Dithmenos), escaped with the Orb and 15 runes on 2015-12-06 00:01:47, with 12502077 points after 131029 turns and 13:37:57.
03:31:46 <|amethyst> that's the only dith game where someone got fire magic up to level 15
03:32:12 <johnstein> whew! integrity preserved!
03:32:58 <johnstein> what's the cleanest/best way to make apply_beam_conducts() public?
03:33:26 <johnstein> my brain is fixating on javaisms and it's been a long time since I worried about public/private functions in C++
03:33:46 <johnstein> is it just moving it in beam.h to the public section?
03:35:45 <|amethyst> yeah
03:36:17 <johnstein> bummer, I bet beam is used everywhere.  this might be a longer compile
03:36:30 <|amethyst> yeah :)
03:37:19 <johnstein> ty for the help. this was a neat one
03:40:32 <|amethyst> oh, double-check that you don't get penalties for monsters casting the spells
03:40:58 <|amethyst> shouldn't be an issue since I think they go through a different code path, but worth checking
03:41:35 <johnstein> that would be humorous
03:58:41 <johnstein> finally
04:26:28 <johnstein> |amethyst, looks like letting Cerebov beat me up for a hundred turns doesn't cause Dith any grief
04:28:13 <johnstein> and looks like all red fire spells are making dith mad. I'm still not quite sure how the penance vs wrath works. looks like maybe if the penance is high enough, and you are still a follower, you get wrathed.
04:34:29 <Cheibriados> New branch created: pull/590 (1 commit) 13https://github.com/crawl/crawl/pull/590
04:34:29 <Cheibriados> 03johnstein02 07https://github.com/crawl/crawl/pull/590 * 0.21-a0-179-gffd08df: Make Dith hate all fire spells 10(10 minutes ago, 4 files, 10+ 1-) 13https://github.com/crawl/crawl/commit/ffd08df0c522
04:39:41 <Cheibriados> 03johnstein02 07https://github.com/crawl/crawl/pull/590 * 0.21-a0-179-gd111ce5: Make Dith hate all fire spells 10(16 minutes ago, 4 files, 10+ 1-) 13https://github.com/crawl/crawl/commit/d111ce54135c
05:25:41 -!- amalloy_ is now known as amalloy
05:31:45 -!- illusion is now known as Guest22884
05:41:30 -!- amalloy is now known as amalloy_
09:24:48 <shmup> I wish FooTV was XomTV
09:31:02 <gammafunk> what would xomtv entail?
09:46:52 -!- Mayoi is now known as erkin
10:11:19 <johnstein> !lg * killer=ogre ikiller!=the_rage_of_trog min=turn x=ikiller
10:11:22 <Sequell> 215753. [ikiller=an ogre] Leissi the Skirmisher (L1 TrBe), demolished by an ogre (a +0,+0 giant spiked club) on D:1 on 2007-12-28 23:39:37, with 9 points after 2 turns and 0:00:09.
10:11:24 <johnstein> !lg * killer=ogre ikiller!=the_rage_of_trog min=turn -log
10:11:27 <Sequell> 215753. Leissi, XL1 TrBe, T:2: http://crawl.develz.org/morgues/trunk/Leissi/morgue-Leissi-20071229-003937.txt
10:11:48 <johnstein> it's a wrath kill after abandonment. but ikiller isn't giving trog the assist. is that correct?
10:43:52 <shmup> gammafunk: Xom as an anchor man is just amusing to me, not to mention I'm usually watching FooTV to recap how someone died (and possibly how they could have avoided it)
10:44:20 <shmup> And I associate failing to win with the essence Xom sometimes, I don't know man
10:44:49 <shmup> Possibly I thought of it because you're only shifting an F to an X and an o to an m
10:45:13 <shmup> s/essence/essence of
14:04:14 -!- amalloy_ is now known as amalloy
14:31:25 <gammafunk> ??changelog
14:31:25 <Sequell> changelog[1/3]: https://github.com/crawl/crawl/blob/master/crawl-ref/docs/changelog.txt - often incomplete and/or out of date, see {changelog[3]} for an exhaustive list of changes.
14:45:38 <amalloy> !lg killer=ogre ikiller=the_rage_of_trog
14:45:39 <Sequell> No games for amalloy (killer=ogre ikiller=the_rage_of_trog).
14:45:47 <amalloy> !lg * killer=ogre ikiller=the_rage_of_trog
14:45:48 <Sequell> 178. THUG4L1F3 the Cleaver (L11 VSBe of Okawaru), mangled by an ogre (a +0 giant club) (created by the rage of Trog) on D:9 on 2017-07-22 18:41:38, with 11186 points after 7325 turns and 0:21:13.
14:46:23 <amalloy> !lg * killer=ogre ikiller=the_rage_of_trog 1
14:46:25 <Sequell> 1/178. Kyrris the Ducker (L1 SpBe), demolished by an ogre (a +0,+0 giant spiked club) (summoned by the rage of Trog) on D:1 on 2009-11-19 22:21:05, with 20 points after 4 turns and 0:00:27.
14:46:38 <amalloy> johnstein: it looks like we didn't start recording trog as ikiller until 2009
14:46:46 <amalloy> the game you linkes is from 2007
15:06:55 <johnstein> that's kinda what I was assuming
15:14:37 -!- jfcaron_ is now known as jfcaron
17:03:54 <Cheibriados> 03advil02 07https://github.com/crawl/crawl/pull/585 * 0.21-a0-166-g9983550: Refactor skill_level_to_diffs slightly based on feedback in IRC 10(7 minutes ago, 3 files, 37+ 27-) 13https://github.com/crawl/crawl/commit/99835501abb8
17:46:46 <advil> amalloy: fwiw most of the things you asked questions about are things that I inherited from the old code
17:46:51 <advil> not that they shouldn't be fixed
17:48:19 <amalloy> at the moment you have the unfortunate position of being the only person who understands them, so you attract suggestions
17:48:23 <advil> heh
17:48:25 <advil> reasonable
17:49:52 <advil> (though "understand" is a bit strong)
17:52:10 <advil> also I'm feeling the pull to rewrite the whole thing using this skill_state struct that otherwise isn't used much
17:52:25 <advil> probably unwise
17:52:36 <gammafunk> I swear this used to have entries
17:52:38 <gammafunk> ??refactor
17:52:38 <Sequell> I don't have a page labeled refactor in my learndb.
17:52:54 <gammafunk> maybe it was censored
17:53:40 <advil> ??pleasingfungus[23
17:53:40 <Sequell> pleasingfungus[23/27]: <PleasingFungus> I don't think it'd even be that much work <PleasingFungus> unless you wanted to refactor it to be good code
18:30:51 -!- Kramin42 is now known as Kramin
18:53:43 <gammafunk> ?/refactor
18:53:43 <Sequell> Matching entries (3): pleasingfungus[13]: <wheals> the refactoring brigadier general | pleasingfungus[23]: <PleasingFungus> I don't think it'd even be that much work <PleasingFungus> unless you wanted to refactor it to be good code | sdl2[1]: TODO: zooming (base initial scale on display density), refactor WME_ framework to line up with SDL2's framework
19:00:37 <amalloy> !cmd !refactor .echo $nick refactors ${1:-$nick} right out of existence!
19:00:37 <Sequell> Defined command: !refactor => .echo $nick refactors ${1:-$nick} right out of existence!
19:02:05 <gammafunk> imo make it like !remove
19:02:08 <gammafunk> !cmd !remove
19:02:08 <Sequell> Command: !remove => !commit Remove ${*}
19:02:32 <Cheibriados> 03amalloy02 07* 0.21-a0-179-gd2bfb5f: Simplify some expressions in skills.cc 10(3 weeks ago, 1 file, 3+ 7-) 13https://github.com/crawl/crawl/commit/d2bfb5fa0b8c
19:03:52 <gammafunk> hey you used the !refactor command
19:04:30 <amalloy> !remove
19:04:31 <Sequell> 03amalloy ⛐ 0.21-a0-2033-gbcc26a4: Remove 10(in the future, 46 files, 302+ 751-) 13http://s-z.org/neil/git/commit.png?p=crawl.git;a=commitdiff;h=bcc26a4
19:05:20 <amalloy> that is a dubious emoji
19:11:54 <gammafunk> you don't drive like that?
19:26:44 <ProzacElf> hah
19:29:36 -!- Fixer_ is now known as Fixer
19:47:28 <alexjurkiewicz> advil: when you rewrite the skill/xl stuff please simplify it on the game mechanics side too
20:09:16 <Jorgrell> Unstable branch on crawl.jorgrun.rocks updated to: 0.21-a0-179-gd2bfb5f (34)
20:40:11 <bh> !seen dpeg
20:40:11 <Sequell> bh: You have 1 message. Use !messages to read it.
20:40:11 <Sequell> I last saw dpeg at Thu Jun  1 23:15:43 2017 UTC (9w 6d 1h 24m 28s ago) quitting, saying 'Quit: nights'.
20:40:30 <bh> !tell dpeg dammit dpeg, we need to finish the book
20:40:30 <Sequell> bh: OK, I'll let dpeg know.
20:48:10 <bh> in case anyone was wondering, Lasty is not an axe murderer
20:48:37 <amalloy> maybe he's a patient axe murderer
20:59:08 <bh> speak of the devil
20:59:17 <bh> I hope you didn't get lost going home
21:20:11 -!- amalloy is now known as amalloy_
21:24:58 <ProzacElf> Lasty--known axe murderer.
21:41:01 <Lasty> bh: haha, it would have been hard :D
21:41:38 <bh> Lasty: you did go the wrong way, I think
21:41:42 <Lasty> lol, looked at logs, now I know why everyone is referencing axe murdering
21:42:01 <Lasty> bh: I wasn't sure where the other direction let out. Could have been a dead end. :)
21:42:04 <Lasty> I should have asked tho
21:43:57 <Cheibriados> New branch created: pull/591 (1 commit) 13https://github.com/crawl/crawl/pull/591
21:43:57 <Cheibriados> 03alexjurkiewicz02 07https://github.com/crawl/crawl/pull/591 * 0.21-a0-174-gcaa902c: Improve bear berserk 10(5 minutes ago, 1 file, 2+ 1-) 13https://github.com/crawl/crawl/commit/caa902c54fea
21:55:26 <Lasty> bh: man, looking at the map, it would have been _much_ shorter to go the other way
22:00:32 <bh> yep.
22:24:16 <advil> in 2014 there was an entry for refactoring, guess it got deleted
22:24:36 <advil> <Sequell> gammafunk[11] -> refactoring[2/2]: <gammafunk> next time I get the idea to refactor code <gammafunk> someone just stop me
22:24:50 <gammafunk> hah, nice, yeah that was the one I think
22:24:52 <advil> that's probably what you were remembering gf
22:25:10 <gammafunk> probably I deleted it in my great purge of gammafunk in the learndb
22:25:19 <gammafunk> ?/gammafunk
22:25:19 <Sequell> Matching terms (2): gammafunk, gammafunkrc; entries (46): Basil[12] | beem[1] | beem[3] | bh[4] | Cerebot[3] | coolplayers[2] | devsprint[12] | devteam[25] | doom_hound[2] | doom_hound[3] | dracoomega[5] | dumb[5] | gammafunk[1] | gammafunk[4] | gammafunkrc[1] | gammafunkrc[2] | grunt[13] | grunt[24] | grunt[25] | hangedman[18] | hilarious_deaths[109] | jeanjacques[5] | johnstein[2] | lomlobot[1] ...
22:25:36 <advil> if you do just "!learn add x" does it delete x?
22:25:39 <gammafunk> it adds it to the end of the topic
22:25:40 <gammafunk> as a new entry
22:25:47 <advil> well with nothing else on the line
22:25:57 <gammafunk> oh it might give an error
22:26:00 <gammafunk> !learn add x
22:26:00 <Sequell> Syntax is: !learn add TERM TEXT or !learn add TERM[n] TEXT
22:26:04 <advil> ah
22:26:06 <gammafunk> yeah it wants some content
22:26:17 <advil> I guess one can edit these things in the sequell channel
22:26:20 <gammafunk> yeah
22:26:25 <gammafunk> !learn add gammafunk[2] hi
22:26:25 <Sequell> gammafunk[2/9]: hi
22:26:28 <gammafunk> cool
22:26:33 <gammafunk> good way to add at a specific place
22:26:38 <gammafunk> !learn del gammafunk[2]
22:26:38 <Sequell> Deleted gammafunk[2/9]: hi
22:28:51 <gammafunk> I was thinking about wand stacking
22:28:58 <gammafunk> there's an issue with the identification aspect
22:29:48 <gammafunk> wand stacking would work so that any time you picked up a wand when you have a wand of that type, the charges of the new wand go into the one you have, and the new one is destroyed
22:30:04 <gammafunk> this could happen regardless of any identification
22:30:13 <gammafunk> but if you want to not waste charges, you have to id the wand
22:30:28 <gammafunk> if you want to not waste charges and know how many you have left
22:30:50 <ProzacElf> are wands no longer going to ID on pickup if stacking goes in?
22:31:07 <gammafunk> we're not bringing back the wand id minigame
22:31:15 <gammafunk> so yes they'd still ID
22:31:22 <gammafunk> the type would, that is
22:31:36 <gammafunk> my issue is how there's less pressure on your id scrolls
22:31:43 <alexjurkiewicz> generate fewer
22:31:44 <Sequell> alexjurkiewicz: You have 1 message. Use !messages to read it.
22:31:44 <gammafunk> since you only have to id a wand of a given type once
22:32:04 <gammafunk> well I shouldn't say less pressure, but more less relevance
22:32:08 <gammafunk> it's pretty vestigal
22:32:17 <nikheizen> id scrolls are pretty vestigal yes
22:32:25 <gammafunk> one of the whole points of that change was that id scrolls were more relevant
22:32:47 <gammafunk> this commit is not about removing the identification process
22:33:01 <gammafunk> so you can save any hot takes in that regard
22:33:32 <alexjurkiewicz> hellmonk: yeah I agree this problem exists in vanilla too
22:33:36 <gammafunk> I've thought about how you could maybe continue to have id on wands be of the same frequency in terms of wanting to use them
22:33:45 <hellmonk> k, just wondering
22:33:45 <gammafunk> but it doesn't seem like it'd work super well
22:34:23 <gammafunk> so that's my initial plan but I'll think about whether there's a way to maybe not remove that aspect of id scrolls quite so much
22:34:41 <hellmonk> check how dcss-ca did it if you haven't already
22:35:01 <gammafunk> you could have non-ided wands not stack, but have them merge with an ided wand automatically
22:35:13 <hellmonk> you know how i feel about identification so i'm not gonna argue on that point, but maybe dcss-ca did something with it
22:35:38 <alexjurkiewicz> from memory, dcs-ca just merged the wands even if the identification was inconsistent
22:35:57 <Cheibriados> 03alexjurkiewicz02 07https://github.com/crawl/crawl/pull/581 * 0.21-a0-151-g96bad19: Rework strength damage modifier 10(4 minutes ago, 1 file, 3+ 9-) 13https://github.com/crawl/crawl/commit/96bad19bb9fd
22:36:01 <gammafunk> that preserves the relevance of id scrolls but is sort of adds an inventory slot aspect
22:36:12 <gammafunk> that's maybe not great to connect to id scrolls
22:36:27 <gammafunk> dpeg would probably like that approach I'm guessing
22:36:38 <gammafunk> Has the old german been around recently
22:36:41 <alexjurkiewicz> i don't think it was a good approach. it means once you identify your initial wand of each type, each new one you pick up just gets merged and implicitly identified
22:36:42 <gammafunk> !seen dpeg
22:36:42 <Sequell> I last saw dpeg at Thu Jun  1 23:15:43 2017 UTC (9w 6d 3h 20m 59s ago) quitting, saying 'Quit: nights'.
22:36:53 <alexjurkiewicz> which was mechanically clunky
22:37:16 <gammafunk> well how do you mean mechanically clunky
22:37:33 <gammafunk> it's probably mostly what players would expect
22:37:51 <gammafunk> now that I type it out I'm sort of warming to the idea of non-ided wands not stacking until they're ided
22:37:57 <alexjurkiewicz> why do you have to identify the first wand to know charges, but you learn that information for free for subsequent wands?
22:38:26 <gammafunk> alexjurkiewicz: it's exactly how potions and scrolls work
22:39:19 <gammafunk> I agree that the entire thing becomes a bit murky if you think too hard about any aspect of it
22:39:37 <hellmonk> it would encourage you to carry around or go back for 0 charge wands if you found another one of the same type, i guess
22:39:57 <hellmonk> unless you destroy 0 charge wands automatically, which might be good to do anyway
22:39:59 <gammafunk> my thought was that when ..yes
22:40:06 <gammafunk> I should have mentioned that
22:40:23 <gammafunk> but again it does sort of introduce some strange juggling when your wand runs low on charges
22:40:41 <gammafunk> you'd theoretically wand to save a charge I guess sort of if you were lowish on id
22:40:46 <hellmonk> i assume that ?recharge is gonna go away as well?
22:40:48 <alexjurkiewicz> one other thing is that the lore about wand merging is reallllly weird. Why do wands merge? What happens?
22:41:13 <gammafunk> the ganja lord
22:41:15 <gammafunk> he tokes up the wand charges and breaths wand smoke onto the wand
22:41:22 <alexjurkiewicz> hellmonk already copyrighted that name for a god
22:41:26 <gammafunk> dang
22:41:30 <gammafunk> no I'm not too concerned about lore
22:41:41 <hellmonk> weedl0rd the dank confirmed for crawl 0.21????
22:41:51 <gammafunk> wandl0rd the dank?
22:41:54 <gammafunk> recharge is sort of its own thing here
22:42:06 <gammafunk> I was thinking maybe recharge could be extended to work on evocables
22:42:12 <gammafunk> the xp ones
22:42:26 <gammafunk> use one scroll to recharge the elemental evokers? I guess that has a problem of wasting xp
22:42:55 <alexjurkiewicz> are elemental evokers strong enough to justify that sort of usage?
22:43:03 <gammafunk> it doesn't necessarilly have to go with stacking; I don't see them as terribly related other than that theortically it's easier to have more charges that you want
22:43:22 <gammafunk> well if they aren't then wands aren't really
22:44:10 <gammafunk> 4 charges of lightning rod certainly packs quite a punch, but it depends on whether the player cares about evokers (do they have a bunch of evo)
22:44:52 <gammafunk> recharge could be good to remove, but I'd maybe like to hear some feedback from others on the dweeb team
22:44:59 <gammafunk> I mean dev team.
22:45:19 <alexjurkiewicz> remove some of my prs. by merging them
22:45:28 <gammafunk> make good prs and I will
22:45:49 <gammafunk> make bad prs and they get the red close button
22:46:01 <gammafunk> I need a gif of clicking close on a PR I can send to alex at times like this
22:46:25 <gammafunk> I have more PRs to go through, we'll get to them, don't worry
22:46:54 <alexjurkiewicz> #521 opened on 19 Apr by alexjurkiewicz
22:47:02 <alexjurkiewicz> soon this PR will be legal drinking age
22:47:22 <gammafunk> you think you're special? We have open bugs on mantis
22:47:29 <gammafunk> with patches for things like "Snail Knight" I bet
22:47:41 <gammafunk> and a crypt end that's basically a big box with sphinx in it
22:47:54 <gammafunk> there's a PR for a new Veh zealot even
22:47:57 <gammafunk> which I should close
22:55:53 <hellmonk> merge it and remove all the elementalist backgrounds imo
22:56:03 <hellmonk> true pro feature right there
23:01:43 <Floodkiller> so I just made a thing for a potential gnoll addition, but I don't think I've actually sat down and justified it to the design concept yet
23:02:00 <Floodkiller> I just did it because I wanted to figure out how to mess around with making an inventory sort category
23:02:06 <Floodkiller> https://github.com/Floodkiller/crawl/commit/1dd04c5338ffa14a2321dbbb477f9366c818789c
23:03:32 <hellmonk> please don't make players eat trash items for attributes
23:04:32 <gammafunk> seems like it's just a way to undo their conduct
23:05:10 <gammafunk> I guess I'm not sure how many artes they would actually get on average
23:05:47 <gammafunk> imagine the hilarity of accidentally eating your wielded weapon tho
23:05:52 <geekosaur> thought they were gnolls, not jellies
23:06:00 <Floodkiller> you can only eat unequipped artifacts
23:06:02 <gammafunk> it's like old sticks to snakes blunder
23:06:20 <Floodkiller> so you can only eat your swap weapon
23:06:33 <gammafunk> but imagine eating your swap weapon!
23:06:39 <gammafunk> guess you better inscribe !e on it
23:06:51 <gammafunk> or like
23:06:57 <gammafunk> you temporarily wield a stave
23:07:13 <gammafunk> and in that moment, disaster strikes....
23:07:14 <hellmonk> imo if you want to give them randomized stats just give them randomized level-up stats like every other species and don't tie it to artifact gen
23:07:43 <gammafunk> no, but in all seriousness yeah it's a pretty convoluted way of just given stats based on exploration which is similar to stats based on xl
23:07:53 <gammafunk> it obviously has more.....flavor........
23:08:23 <gammafunk> I suppose it's sort of interesting on some level in how you can go into item rich areas for permanent upgrades
23:08:26 <gammafunk> like zigs
23:09:05 <gammafunk> it's kind of bold, I'll grant you that
23:09:07 <Floodkiller> that was more of the intention behind why I made it (again without thinking about how much it affects the conduct)
23:09:38 <johnstein> I can't think of any good comments for my dith fire spells PR. I feel like that means I'm not getting creative enough
23:09:40 <Floodkiller> since the species isn't really encouraged to take as many risks for XP
23:10:05 <Floodkiller> as for average artifact gen, I think it was around 30 from what I counted up in objstat
23:10:21 <gammafunk> well that's like 10 pts on average added to each stat
23:10:43 <gammafunk> unless you make it a chance to gain a point or something
23:11:04 <Floodkiller> a chance would probably be fine to add in
23:17:42 <johnstein> I'm trying to find all the monsters that use ignite_poison, inner_flame, and ignition. I think the first two are monster spells too, but not sure about the last. I can't find these 3 in mon-spell.h, but inner flame and ignite_poison_single are in mon-cast.
23:19:25 <hellmonk> ignition isnt a monster spell afaik
23:20:02 <gammafunk> making ignition a monster spell would make minmay madder than that time r-i put the coctagon in cocytus
23:21:41 <ProzacElf> have a monster use ignition in the coctagon
23:21:48 <ProzacElf> for maximum anger
23:22:19 <Floodkiller> if you drag a monster to the coctagon in cocytus to case ignition, your mummy can drink potions
23:22:25 <Floodkiller> cast*
23:22:53 <ProzacElf> mm
23:27:56 <Cheibriados> 03advil02 07https://github.com/crawl/crawl/pull/585 * 0.21-a0-167-gfccdb0a: Further code cleanups and commenting 10(3 minutes ago, 1 file, 21+ 10-) 13https://github.com/crawl/crawl/commit/fccdb0a44f8d
23:30:46 <gammafunk> code cleanups? *and* comments? A bit contraversial, not sure if we can merge
23:33:11 <advil> it's all amalloy's fault, really
23:33:50 <gammafunk> it always is...
23:35:42 <alexjurkiewicz> Floodkiller: it would allow scumming abyss for 120 120 120 stat
23:40:46 <alexjurkiewicz> hellmonk: i think gozag abilities need a rework for hellcrawl. well, at least call merchant
23:40:57 <hellmonk> yeah, agree
23:41:17 <hellmonk> some sort of shop rework would be good
23:41:52 <alexjurkiewicz> for shops in general i was thinking of just reducing prices globally
23:41:58 <alexjurkiewicz> maybe halving
23:59:39 <Eksell> Unstable branch on CRAWL.XTAHUA.COM updated to: 0.21-a0-179-gd2bfb5f (34)