00:16:45 <Cheibriados> New branch created: pull/1090 (1 commit) 13https://github.com/crawl/crawl/pull/1090
00:16:45 <Cheibriados> 03alexjurkiewicz02 07https://github.com/crawl/crawl/pull/1090 * 0.24-a0-459-g843278e: Recompress all tiles with zopfli 10(17 hours ago, 4785 files, 0+ 0-) 13https://github.com/crawl/crawl/commit/843278e4923a
01:05:59 <gammafunk> I'm reading that zopfli has been integrated into advpng?
01:06:30 <gammafunk> -4, --shrink-insane
01:06:31 <gammafunk> Set the compression level to "insane" using the zopfli compressor. You can define the compressor iterations with the -i, --iter option.
01:07:39 <gammafunk> alexjurkiewicz: we already support use of advpng and that has support for zopfli apparently, so we'd likely want to use that
01:38:26 <alexjurkiewicz> this compressor seems to produce smaller files than advpng -4
01:39:22 <alexjurkiewicz> partly because this compressor can be tweaked to use much slower compression zopfli settings than advpng -4 (recompressing all those files took about 32 CPU hours on my laptop) but it also seems to drop the colour space if possible which advpng won't do
01:40:33 <alexjurkiewicz> the latter part means the only part I'm worried about is maybe this will cause some weird incompatibility with SDL? I didn't see any issues compiling locally on macOS
01:42:09 <alexjurkiewicz> i'm trying to compress the title images too, realised I missed them
01:49:25 -!- amalloy_ is now known as amalloy
03:06:00 <alexjurkiewicz> is it a bug to shaft into no_tele_into area? https://cdn.discordapp.com/attachments/99797890309840896/593696635088011285/unknown.png
03:21:56 <Cheibriados> New branch created: pull/1091 (1 commit) 13https://github.com/crawl/crawl/pull/1091
03:21:56 <Cheibriados> 03wjchen02 {GitHub} 07https://github.com/crawl/crawl/pull/1091 * 0.24-a0-459-gc520147: Allow travelling to altars in dungeon overview 10(9 minutes ago, 1 file, 115+ 3-) 13https://github.com/crawl/crawl/commit/c52014747a7b
04:25:29 <amalloy> alexjurkiewicz: yes. what vault is that? my first guess would be that this area is not no_tele_into but should be
04:26:57 -!- amalloy is now known as amalloy_
05:10:08 <simbs38> hi there, i want to run the beem bot locally from my machine, but i cannot set it to watch my player, can anyone help me out please?
05:15:32 <simbs38> nevermind, i got it, thanks anyway
06:23:30 <Royaljello> hey all
08:10:25 <alexjurkiewicz> amalloy_: looks like it's minmay_hidden_floor
08:10:47 <alexjurkiewicz> minqmay: is minmay_hidden_floor meant to be fully enclosed? There's a runed door but it seems like you could walk around it
08:11:37 <alexjurkiewicz> oh no you can't, I misready it
08:24:16 <Cheibriados> New branch created: pull/1092 (1 commit) 13https://github.com/crawl/crawl/pull/1092
08:24:16 <Cheibriados> 03alexjurkiewicz02 07https://github.com/crawl/crawl/pull/1092 * 0.24-a0-459-g0f7d979: Mark some runed door vaults as no_tele_into 10(4 minutes ago, 2 files, 3+ 1-) 13https://github.com/crawl/crawl/commit/0f7d97973b1e
09:44:09 <advil> man, the stuff that qw gets away with
09:44:39 <advil> in fairness, this instance does have a +10 vamp broad axe
11:22:41 <Lantell> Stable (0.23) branch on underhound.eu updated to: 0.23.1-82-g5129a83184
12:48:43 <Cheibriados> 03wjchen02 07https://github.com/crawl/crawl/pull/1091 * 0.24-a0-460-g2ad161e: Use toupper_safe() to handle altar selection 10(25 minutes ago, 1 file, 1+ 26-) 13https://github.com/crawl/crawl/commit/2ad161e6d53a
13:39:16 -!- amalloy_ is now known as amalloy
13:41:48 <Cheibriados> 03alexjurkiewicz02 {amalloy} 07* 0.24-a0-459-g225fefd: Mark some runed door vaults as no_tele_into (#1092) 10(12 seconds ago, 2 files, 3+ 1-) 13https://github.com/crawl/crawl/commit/225fefd5308d
13:47:30 <Kelbell> Unstable branch on crawl.kelbi.org updated to: 0.24-a0-459-g225fefd530 (34)
14:51:47 <travis-ci> The build has errored. (master - 225fefd #11644 : Alex Jurkiewicz): https://travis-ci.org/crawl/crawl/builds/551429914
18:22:11 <Lantell> Unstable branch on underhound.eu updated to: 0.24-a0-459-g225fefd530 (34)
20:10:42 <Cheibriados> 03alexjurkiewicz02 07https://github.com/crawl/crawl/pull/1090 * 0.24-a0-459-gf890ff5: Recompress all PNGs with zopfli 10(13 hours ago, 4815 files, 0+ 0-) 13https://github.com/crawl/crawl/commit/f890ff5fc7c6
20:12:12 <alexjurkiewicz> crunched title PNGs now
20:20:45 -!- amalloy is now known as amalloy_
20:22:10 <gammafunk> did you see what I mentioned about advpng?
20:22:18 <gammafunk> it has support for zopfli and we're already using it
20:22:37 <gammafunk> by using it, I mean we're already using advpng
20:23:00 <gammafunk> and could possibly just call it with the flags to enable use of zopfli instead
20:51:39 <alexjurkiewicz> It seems advpng's zopfli configuration gives larger files than the binary I used
21:01:47 <alexjurkiewicz> hm actually on the title PNGs it can save a few more bytes
21:01:51 <alexjurkiewicz> 18625       18617  99% title_froggy_goodgod_tengu_gold.png
21:16:53 <gammafunk> yeah, not sure what all options it has
21:17:06 <gammafunk> but that's obviously a lot more convenient than requiring people to install a new tool
21:18:04 <gammafunk> anyhow, I didn't really investigate it much, but the zopfli wikipedia page mentioned how advpng had support for it
21:27:16 <alexjurkiewicz> so I'm just recompressing the static assets in the repo. I'm not quite sure what you mean by requiring people to install a new tool. You mean in order to ensure new tiles are compressed optimally?
21:27:55 <gammafunk> oh, this isn't a build time thing at all?
21:29:05 <gammafunk> as in, don't you have to compress the tile sheets themselves in order to see the savings? which would imply it happens at build time and would add a dependency, but yeah maybe I don't understand what you're doing
21:29:19 <alexjurkiewicz> that's right. I just performed an offline recompression of all the content currently in the repo. The build process for webtiles where the PNGs are compiled into spritesheets is unchanged
21:29:37 <alexjurkiewicz> This will improve offline tiles package size by a megabyte or so
21:30:46 <gammafunk> well, does this recompression of the tiles (that go into the tilesheets) actually translate into less file size for the tilesheets?
21:30:48 <alexjurkiewicz> I don't expect any significant change in webtiles spritesheet size (probably no change). The sprite sheets are compressed with advpng -z3 as you probably know. It used to be -z4 but it's a LOT slower and IMO not worth doing for daily trunk rebuilds (maybe another story for builds of stable relases)
21:30:59 <gammafunk> ok
21:31:56 <gammafunk> so is there much point to recompressing the repo tiles (that go into tilesheets)? the savings there is basically only for source packages, right?
21:32:14 <alexjurkiewicz> offline tiles doesn't use tilesheets, does it?
21:32:17 <gammafunk> and the savings to e.g. tiles packages is just coming from recompression of all the non-tile stuff
21:32:22 <gammafunk> yes it does
21:32:57 <gammafunk> it uses the same sort of tilesheet setup and those sheets are assembled with the same tilegen tool
21:33:16 <alexjurkiewicz> oh huh TIL. In that case there's no real point to recompressing the raw PNGs. It wouldn't even reduce the repo size, it would do the opposite since you now have to store two versions of the tile in git history
21:33:18 <gammafunk> the difference is just how that tilesheet is handled/rendered (via sdl)
21:33:49 <gammafunk> right
21:33:51 <alexjurkiewicz> I'll change the PR to only recompress the title PNGs and icons, since they aren't put into sheets
21:33:59 <gammafunk> yeah that would make sense
21:34:17 <gammafunk> recompressing tiles would decrease the size of non-git copies of the source
21:34:34 <gammafunk> but that's probably not worth the effort
21:35:05 <alexjurkiewicz> i wonder how big that is now and if 1mb is significant
21:35:16 <gammafunk> can see download page
21:35:38 <gammafunk> 28.8 mb for dependencies
21:35:48 <gammafunk> 10.5 for without dependencies
21:36:21 <gammafunk> I think one issue though is that we'd need to start uploading all new tiles after zopfli compression
21:36:43 <gammafunk> otherwise we'd have some recompressed some not; I guess that's just a periodic thing one could do
21:36:50 <gammafunk> but not really sure if it's wise to do stuff like this
21:37:20 <alexjurkiewicz> you could probably write a travis test to check new/modified PNGs are optimally compressed
21:37:48 <gammafunk> I suppose so, yeah; I'd leave it to someone far more qualified to comment about this sort of image management stuff
21:38:20 <gammafunk> I don't know if other projects do this or how they keep image sizes down or whatnot
21:38:43 <alexjurkiewicz> thanks for the offline tiles info. I'll update the PR later today and try to include that travis test
21:59:52 -!- amalloy_ is now known as amalloy
23:24:51 <Cheibriados> Switching weapon inventory slots causes mismatch in displayed wielded weapon 13https://crawl.develz.org/mantis/view.php?id=12015 by kitchen_ace