00:00:59 <Eksell> Unstable branch on CRAWL.XTAHUA.COM updated to: 0.23-a0-455-ge12115f (34)
01:27:48 <Gretell> Unstable branch on crawl.develz.org updated to: 0.23-a0-455-ge12115f (34)
01:58:22 <Gretell> Windows builds of master branch on crawl.develz.org updated to: 0.23-a0-455-ge12115f
02:39:15 <Henzell> Blackmore (L16 MiFi) ERROR in 'tileweb.cc' at line 219: Socket write error: Resource temporarily unavailable (D (Sprint))
02:58:43 <Gretell> Monster database of master branch on crawl.develz.org updated to: 0.23-a0-455-ge12115f
04:44:08 <alexjurkiewicz> congrats on cko!
05:42:28 <Cheibriados> New branch created: pull/907 (1 commit) 13https://github.com/crawl/crawl/pull/907
05:42:28 <Cheibriados> 03NormalPerson702 07https://github.com/crawl/crawl/pull/907 * 0.23-a0-451-gd621765: Don't let Ru offer mummies the no potion heal mutation (11750) 10(6 minutes ago, 1 file, 4+ 0-) 13https://github.com/crawl/crawl/commit/d62176515c34
11:28:15 <Cheibriados> 03NormalPerson702 {ebering} 07* 0.23-a0-456-gc514d67: Don't let Ru offer mummies the no potion heal mutation (11750) 10(6 hours ago, 1 file, 4+ 0-) 13https://github.com/crawl/crawl/commit/c514d6719830
12:06:27 <Henzell> Unstable branch on crawl.akrasiac.org updated to: 0.23-a0-456-gc514d67 (34)
12:10:57 <Jorgrell> Unstable branch on crawl.jorgrun.rocks updated to: 0.23-a0-456-gc514d67198 (34)
15:31:00 <switch__> ebering: regarding 900, I originally actually wrote the feature you asked for (the boolean option), but rewriting 1035-1037 in delay.cc to accomodate it seemed hackier than i wanted it to look
15:32:09 <switch__> i'm still not really sure how to write it in a way that isn't... excessive
15:32:38 <switch__> since if it's separate from the ai_heal and ai_mana calls, it has to check for those in some way
15:32:49 <ebering> yes
15:32:55 <ebering> It's a new interrupt and a new option
15:33:08 <ebering> It should take some checking
15:33:37 <ebering> one moment
15:41:22 <switch__> oh, regarding "some players might prefer not to have resting wait for the ancestor." i considered this, but discounted it
15:41:43 <switch__> in my testing the ancestor heals super quickly, significantly faster than the player
15:41:49 <ebering> switch__: I added a comment on the pr suggesting a target behavior
15:42:06 <switch__> looking at it now, thanks
15:42:18 <ebering> and an implementation
15:56:20 <Eksell> Gruud (L14 DsNe) ASSERT(!invalid_monster(&mons)) in 'mon-death.cc' at line 1904 failed. (Orc:2)
16:04:26 <switch__> ebering what did you mean by your last sentence? (regarding the double message)
16:06:01 <ebering> switch__: if you're resting and your ancestor re-spawns you get a message
16:06:11 <ebering> you should not also get a "ancestor at full health" message
16:07:40 <switch__> doesn't seem to happen
16:15:35 <switch__> but only because i didn't actually change all the functions from "anc" to "ancestor" when i was testing, re-resting now
16:17:46 <switch__> actually, i can tell you from the code that it won't happen, because the healing message only procs from the new interrupt, and the new interrupt only procs if the monster regenerates one HP into full hp
16:51:12 <gammafunk> !crash gruud
16:51:13 <Sequell> 5. Gruud, XL14 DsNe, T:19153 (milestone): https://crawl.xtahua.com/crawl/morgue/Gruud/crash-Gruud-20181124-205601.txt
17:01:38 <switch__> ebering, any advice? https://pastebin.com/raw/EEmPFbSG
17:02:12 <switch__> i'd do mons->god == hep, but I imagine there are wrath summons with those tags
17:09:40 <switch__> https://github.com/crawl/crawl/blob/master/crawl-ref/source/tags.cc#L4348 just crashed to this
17:09:51 <switch__> while loading yermak's old bugged save
17:09:56 <switch__> and playing around with the PR
17:10:02 <switch__> this probably shouldn't be a fatal crash
17:10:23 <ebering> switch__: there's a utility function
17:10:23 <switch__> unless you want it to be i guess
17:10:30 <ebering> heplask_ancestor(mons)
17:10:32 <ebering> use thaat
17:10:40 <switch__> got it, thanks
17:11:08 <floraline> is it important that /ttyrec and /morgue are indexed
17:14:47 <gammafunk> floraline: not the directory of usernames but the individual directories themselves probably should be
17:15:04 <gammafunk> assuming by "indexed" you mean the directory listing
17:15:52 <gammafunk> for ttyrecs, footv needs the directory listing, and in general it's best to have a listing to help find the desired ttyrec in case the footv listing is off
17:15:55 <floraline> the directory of usernames specifically
17:16:03 <gammafunk> yeah, that does not need to be indexed
17:16:16 <gammafunk> I think that's disabled on CAO
17:16:18 <floraline> ok, great
17:16:56 <floraline> that is a custom script that is serving ttyrec and morgue directories and i don't know how well it will perform when the number of items is very large
17:18:58 <floraline> i was thinking i might turn off the user list but i will have to find out if that is a problem anyway for the file list
17:19:54 <ebering> ??pacification[3
17:19:55 <Sequell> pacification[3/3]: The god power has about 25% chance of pacifying a hydra with 5 invo. With 10 invo, 69%. See http://www.niestu.com/caracal/pacify.html for a table of approximate probabilities.
17:20:00 <gammafunk> floraline: the user list?
17:20:28 <floraline> the first level under /ttyrec and /morgue
17:20:56 <gammafunk> I thought that's what you were initially asking about
17:21:14 <floraline> yeah that's what i mean, i would like to turn that off since it sounds like it's not necessary
17:21:44 <gammafunk> gotcha; yeah footv does need the user's ttyrec dir itself to be indexed, but it doesn't need the index of users
17:22:00 <gammafunk> as far as morgues go, technically no directory listing at all is necessary
17:28:57 <alexjurkiewicz> floraline: are you putting morgues onto s3 as well? what is your script for listing them?
17:40:05 <floraline> it's DO 'spaces', not s3 but yes morgue is on object storage as well
17:40:33 <floraline> i altered some existing script, let me see what it was
17:45:52 <floraline> i'm certain that i edited only a small part of an existing dgl script but i can't find the original
17:46:31 <floraline> it's compress-ttyrecs.sh and -morgue on my end, i don't know if that's the original name
17:46:57 <floraline> oh nevermind that's not what you were asking
17:48:08 <floraline> to generate the directory listing i wrote a python script which runs under cgi-bin, based on your parameters it fetches the files from both 's3' and local disk and just draws them together
17:48:16 <floraline> i can put it on pastebin or something if you want a copy of it
17:55:47 <Cheibriados> 03stenella02 07https://github.com/crawl/crawl/pull/900 * 0.23-a0-419-gb269562: Changes requested 10(2 minutes ago, 13 files, 42+ 31-) 13https://github.com/crawl/crawl/commit/b269562ed710
17:57:14 <floraline> alexjurkiewicz: here is the script i'm using: https://pastebin.com/nQNarV0j
17:57:38 <floraline> it's really hacky and it probably only works with DO spaces
17:58:00 <switch__> ebering i think that should do it. from my testing it all works as expected, but i need to clean up my environment
17:58:12 <switch__> so you may want to quickly sanity check the obvious stuff
18:22:33 <Lantell> Unstable branch on underhound.eu updated to: 0.23-a0-456-gc514d67198 (34)
18:31:20 <switch__> speaking of cleaning up my environment
18:31:47 <switch__> if there's no problems, i'd appreciate a merge on the newest PR ASAP <3
18:32:54 <Cheibriados> New branch created: pull/908 (1 commit) 13https://github.com/crawl/crawl/pull/908
18:32:54 <Cheibriados> 03stenella02 07https://github.com/crawl/crawl/pull/908 * 0.23-a0-457-gbe97a06: MSVC gitignore 10(4 minutes ago, 1 file, 5+ 1-) 13https://github.com/crawl/crawl/commit/be97a069337f
18:33:12 <floraline> ahh yes that was bothering me, i was wondering if that was intentional?
18:34:17 <switch__> what, the vcproj.*.?
18:36:23 <gammafunk> hrm, weird that there aren't more constriction crashes
18:36:30 <floraline> actually my problem was with the whole MSVC folder
18:36:42 <floraline> git threw a fit about that yesterday
18:36:50 <switch__> wdym
18:37:04 <switch__> oh, as in, you're also using MSVC?
18:37:53 <floraline> no, i was not able to update because the files in MSVC were tracked remotely but not locally
18:38:26 <floraline> i don't know exactly what happened but something must have changed in that folder
18:38:30 <switch__> I don't think the PR changes anything about that
18:38:44 <switch__> you're gonna have to be a bit more specific
18:38:53 <floraline> well i thought they ought to be added to gitignore and it looks like you are doing that
18:39:15 <switch__> I only added the .vs folders to the gitignore, which you shouldn't have in your directory if you aren't using MSVC
18:39:49 <geekosaur> suspect that happened is you have different msvc versions, so merge conflicts because the local version will have rewritten the files to suit itself
18:40:17 <floraline> no i'm not using msvc, it was just git complaining about something
18:40:38 <floraline> i forget what the problem was exactly
18:40:58 <gammafunk> advil: Have you seen any other weird constriction crashes recently, possibly involving summons dying? Looking at the code, I'm sort of surprised there aren't more
18:41:05 <gammafunk> !crash gruud
18:41:07 <Sequell> 5. Gruud, XL14 DsNe, T:19153 (milestone): https://crawl.xtahua.com/crawl/morgue/Gruud/crash-Gruud-20181124-205601.txt
18:41:11 <gammafunk> this one is a good example
18:43:08 <gammafunk> let's see if I can recreate this
19:02:22 <gammafunk> heh, bvc isn't reflectable because it's an "explosion"
19:30:04 <bh> how's crawlcode these days?
19:33:35 <switch__> getting better every week
19:41:27 <gammafunk> bh: what's up
19:41:55 <switch__> gamma, could you merge the newest PR I put up?
19:41:59 <gammafunk> bh: Do you have any grand plans for a return
19:42:18 <switch__> #908
19:45:11 <Cheibriados> 03gammafunk02 07* 0.23-a0-457-g6f6daef: Add more checks for invalid constrictions 10(16 minutes ago, 1 file, 7+ 5-) 13https://github.com/crawl/crawl/commit/6f6daefbf7df
19:46:25 <ebering> hey team, aiming at one of our long running implementables I've put up a new pr
19:46:39 <ebering> didn't just merge it because it changes some formulas, but I think I maatched the distributions pretty well
19:46:54 <ebering> will probably merge some time tomorrow if there aren't any major hangons
19:47:47 <gammafunk> nice
19:48:17 <gammafunk> so no more pacifying player ghosts or pan lords
19:48:47 <switch__> return (scale * (sides - target) * (sides - target)) / (sides * sides);
19:48:48 <switch__> jesus
19:50:39 <Cheibriados> New branch created: pull/909 (4 commits) 13https://github.com/crawl/crawl/pull/909
19:50:40 <Cheibriados> 03ebering02 07https://github.com/crawl/crawl/pull/909 * 0.23-a0-455-g6954b01: Change pacification to depend only on monster type 10(7 hours ago, 1 file, 6+ 6-) 13https://github.com/crawl/crawl/commit/6954b01559b0
19:50:40 <Cheibriados> 03ebering02 07https://github.com/crawl/crawl/pull/909 * 0.23-a0-456-g4c20a16: Simplify Heal Other formula 10(3 hours ago, 3 files, 30+ 25-) 13https://github.com/crawl/crawl/commit/4c20a16a88a6
19:50:40 <Cheibriados> 03ebering02 07https://github.com/crawl/crawl/pull/909 * 0.23-a0-457-gf1c1ca4: Display pacification chance in Heal Other targeter 10(2 hours ago, 1 file, 31+ 12-) 13https://github.com/crawl/crawl/commit/f1c1ca41e0c2
19:50:40 <Cheibriados> 03ebering02 07https://github.com/crawl/crawl/pull/909 * 0.23-a0-458-gb11bff2: Make ghost demons immune to pacification 10(71 minutes ago, 2 files, 14+ 5-) 13https://github.com/crawl/crawl/commit/b11bff2b74a2
19:51:11 <gammafunk> ebering: slight problem, you've made ugly things and very ugly things immune as well
19:51:39 <gammafunk> those are ghost demons due to how they're set up
19:54:26 <gammafunk> but it's easy to allow them as an exception; you might just want to specifically rule out player ghosts and pan lords since those are the only two you have with specific concerns
19:54:30 <gammafunk> ??pacify
19:54:31 <Sequell> pacification[1/3]: Followers of Elyvilon can use a (smite-targeted) ability to attempt to pacify a monster. Success rate depends on invocation skill, the monster's mhp, and the monster's holiness (see [2]). Pacifying a monster gives half of the normal xp.
19:54:36 <gammafunk> ??pacify[2
19:54:36 <Sequell> pacification[2/3]: Monster's maxhp is multiplied by 2 (holy), 3 (other), 4 (undead), 5 (demonic), and then divided by 3 for animal or lower intelligence. The check is whether 1d((invo+1)*power) > modified_maxhp. Power is randomized and depends on your invocations skill, with average 19+2*invo/3.
19:54:46 <gammafunk> ok, and you also exclude dancing weapons
19:57:11 <gammafunk> left a comment on the PR to that effect
20:09:31 <switch__> https://github.com/crawl/crawl/pull/909/commits/f1c1ca41e0c26e0b1b986a15ed13008ed27dfdc5#diff-c237b22d6b4afba66d4e3d33f20c1301L191 is this change intentional?
20:10:43 <Jorgrell> Unstable branch on crawl.jorgrun.rocks updated to: 0.23-a0-457-g6f6daefbf7 (34)
20:10:57 <geekosaur> given the associated wording, I'd say yes
20:11:18 <geekosaur> its' aborting out in the case // monster avg hp too high to ever be pacified with your invo skill.
20:11:26 <switch__> ofc, but the math doesn't seem to have been reversed
20:11:57 <geekosaur> why woudl it? it's taking a "this can never work" and aborting instead of falsely claiming something appened
20:12:27 <geekosaur> hence wording I metioned: "is" (something happened) -> "would be" (nope)
20:13:07 <switch__> oh, i understand
20:13:12 <switch__> that makes sense
20:46:25 <gammafunk> floraline: by the way, if you wanted to try out hosting an experimental, the known-traps branch is one we'd like to see playtested more
20:47:10 <gammafunk> no worries if you don't want to mess with that now (or at all), of course
20:48:35 <floraline> sure, i'll see if i can figure out what it takes to set that up
21:22:57 <floraline> gammafunk: is the experimental branch present in the official repository?
21:23:10 <floraline> or are they typically only in forks?
21:23:44 <gammafunk> well, technically speaking, an experimental branch is just a repo+branch install as a game on your server
21:23:48 <gammafunk> but yes it has a branch
21:23:57 <gammafunk> %git known-traps
21:23:57 <Cheibriados> Could not find commit known-traps (git returned 128)
21:24:05 <gammafunk> oh is it missing a hyphen
21:24:12 <gammafunk> or did he host it in his fork
21:24:18 <floraline> i see it has a branch in ebering's fork
21:24:21 <floraline> but not in the original repo
21:24:32 <gammafunk> yeah, I guess he's put it in his fork
21:24:42 <gammafunk> typically it would be hosted in the main repo
21:25:27 <gammafunk> I know cbro hosts forks just fine, and I don't think it being in another repo causes a problem for you
21:25:34 <gammafunk> you need his repo as a remote, of course
21:25:47 <gammafunk> but if it is a problem, ebering would probably be willing to just make it a branch in crawl:crawl
21:25:47 <floraline> i'm trying to figure out what the out-of-the-box scripts are capable of
21:29:47 <gammafunk> I think they just need a local branch
21:29:53 <floraline> ahh ok, i'm looking at cbro branch on dgl-config, i see how they are doing it
21:30:07 <gammafunk> so you can have one that's a checkout of any remote, doesn't have to be crawl:crawl
21:30:31 <floraline> you're saying one local repo that can checkout any branch on any remote?
21:30:32 <gammafunk> s/remote/remote's branch/
21:31:14 <gammafunk> well your repo is its own entity; it knows about the remote 'crawl' (if that's what you've called it) that corresponds to crawl:crawl on github
21:32:27 <gammafunk> when you make a branch that tracks crawl:master, it's just a checkout of that remote's branch where git pull defaults to fetching+merge from the crawl remote's master branch
21:32:46 <gammafunk> so it's important to understand that your repo is its own independent repository
21:33:23 <gammafunk> and yeah, your repo is perfectly happy to use any remote that's a fork of repository yours is based on
21:33:37 <gammafunk> and yours is based on the crawl:crawl github repository
21:34:08 <gammafunk> as is ebering's, so you can fetch from his github repo as a remote and check out its branches as a new local branch in your repo
21:35:23 <gammafunk> the only issue could be if various scripts have repository names hard coded in some place for git operations they might perform (e.g. some kind of git pull)
21:35:35 <gammafunk> but I don't think they would do that
21:36:00 <gammafunk> I believe they just do 'git pull' under the assumption you are already tracking the right remote branch
21:36:26 <gammafunk> so your local checkout of known-traps will track the known-traps branch of ebering:crawl
21:42:44 <Henzell> Airwolf (L21 VSFi) ERROR in 'tileweb.cc' at line 219: Socket write error: Resource temporarily unavailable (Vaults:3)
21:42:45 <Henzell> linjason (L13 GnAE) ERROR in 'tileweb.cc' at line 219: Socket write error: Resource temporarily unavailable (IceCv)
22:13:57 <Cheibriados> 03stenella02 07https://github.com/crawl/crawl/pull/908 * 0.23-a0-457-gc432555: MSVC gitignore 10(4 hours ago, 1 file, 6+ 1-) 13https://github.com/crawl/crawl/commit/c4325554b2a3
22:24:54 <switch__> aidanh:
22:26:05 <aidanh> switch__: the removal of the .*. is intentional, yeah?
22:26:18 <switch__> i'm not sure why it's there in the first place, to be honest
22:26:19 <switch__> but yes
22:26:25 <switch__> it didn't hit my user files
22:27:01 <Cheibriados> 03stenella02 {Aidan Holm} 07* 0.23-a0-458-g93a6ea9: MSVC gitignore 10(4 hours ago, 1 file, 6+ 1-) 13https://github.com/crawl/crawl/commit/93a6ea952288
22:27:14 <aidanh> cheers, merged
22:27:20 <switch__> awesome, thanks
22:27:28 <switch__> i can pull directly into my dev environment now
22:27:29 <switch__> wow
22:28:37 <switch__> ebering: travis' debug branch seems to disagree with your PR
22:33:58 -!- isha is now known as werekitten
22:44:49 <Yermak> %git b11bff2b74a2
22:44:49 <Cheibriados> 07ebering02 * 0.23-a0-458-gb11bff2: Make ghost demons immune to pacification 10(4 hours ago, 2 files, 14+ 5-) 13https://github.com/crawl/crawl/commit/b11bff2b74a2
22:45:04 <Yermak> ebering, line 363: 'to' is missing
22:48:04 <switch__> "to unintelligent to be pacified"
23:07:38 <floraline> gammafunk: thanks for the explanation, i think i have it all understood. known-traps is available now
23:10:39 <Jorgrell> Unstable branch on crawl.jorgrun.rocks updated to: 0.23-a0-458-g93a6ea9522 (34)
23:12:24 <floraline> i should have no problem adding experimental branches in the future
23:15:07 <gammafunk> floraline: right on, thanks a ton
23:51:49 <Cheibriados> The other floor is shown on X-map when trying to travel to unreachable stairs. 13https://crawl.develz.org/mantis/view.php?id=11771 by Yermak