⏴ go to previous day
01:51:19 *** HerzogDeXtEr has joined #openttd
02:05:39 *** NGC3982 has quit IRC (Ping timeout: 480 seconds)
02:13:49 *** NGC3982 has joined #openttd
02:21:28 *** Flygon has joined #openttd
02:45:17 *** Wormnest has quit IRC (Quit: Leaving)
02:46:07 *** Compu has joined #openttd
02:46:39 *** Compu has quit IRC ()
02:49:37 *** D-HUND has joined #openttd
02:53:10 *** debdog has quit IRC (Ping timeout: 480 seconds)
03:00:45 *** felix has quit IRC ()
03:02:41 *** felix has joined #openttd
03:36:05 *** keikoz has joined #openttd
03:45:53 <DorpsGek> [OpenTTD/OpenTTD] glx22 opened pull request #10848: Fix #10846: [Squirrel] Ensure sqvector size does not overflow https://github.com/OpenTTD/OpenTTD/pull/10848
03:53:40 *** D-HUND is now known as debdog
04:48:16 *** keikoz has quit IRC ()
06:30:18 *** keikoz has joined #openttd
06:55:38 *** gelignite has joined #openttd
07:24:18 *** _aD has joined #openttd
07:35:20 <andythenorth> o7
07:55:46 *** nielsm has joined #openttd
08:02:02 *** _aD has quit IRC (Ping timeout: 480 seconds)
09:21:30 *** Wolf01 has joined #openttd
09:22:17 <DorpsGek> [OpenTTD/OpenTTD] fmang opened pull request #10849: Add: Shift-click Rename on VehicleView to rename the group https://github.com/OpenTTD/OpenTTD/pull/10849
09:41:14 <DorpsGek> [OpenTTD/website] auge8472 opened pull request #254: Change: make even short pages taking the whole viewport height https://github.com/OpenTTD/website/pull/254
09:58:49 <DorpsGek> [OpenTTD/team] legoscia opened issue #420: [eo_EO] Translator access request https://github.com/OpenTTD/team/issues/420
10:02:12 <DorpsGek> [OpenTTD/team] glx22 commented on issue #420: [eo_EO] Translator access request https://github.com/OpenTTD/team/issues/420
10:24:26 <DorpsGek> [OpenTTD/OpenTTD] fmang updated pull request #10849: Add: Shift-click Rename on VehicleView to rename the group https://github.com/OpenTTD/OpenTTD/pull/10849
10:27:24 *** gelignite has quit IRC (Quit: Stay safe!)
10:32:29 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #10850: Update: fmt library to 10.0.0 https://github.com/OpenTTD/OpenTTD/pull/10850
11:19:04 <petern> Hi
11:19:25 <andythenorth> hi
11:19:43 <TrueBrain> hI
11:23:52 <petern> Legs need a rest, they are knackered
11:50:43 <TrueBrain> and guess what ... GitHub's Container Registry also doesn't have IPv6
11:50:46 <TrueBrain> what the hell guys ..
11:51:52 <TrueBrain> github.com doesn't even have AAAA records .. sadddd
11:52:20 <petern> Weird
11:52:56 <TrueBrain> especially as it would make an AWS deployment drastically more expensive ... hmm .. can I bounce this via a Cloudflare Worker .. most likely I can .. feels weird .. but does solve the issue 😛
11:55:22 <TrueBrain> https://github.com/orgs/community/discussions/10539 always nice to read 😄
11:56:15 <TrueBrain> their docs at least are IPv6 enabled 😛
12:25:22 <petern> Damn, the constructor inheritance stuff (whatever the correct term for it is) needs to be in the declaration 😦
12:33:38 *** birdjj6 has joined #openttd
12:39:22 *** birdjj has quit IRC (Ping timeout: 480 seconds)
12:39:22 *** birdjj6 is now known as birdjj
12:52:49 <andythenorth> uuuf Horse is now 24.8 MB
12:54:00 <andythenorth> 50 MB of nfo
12:54:52 *** Ahyangyi has joined #openttd
12:54:52 <Ahyangyi> 😮
12:54:58 <Ahyangyi> The one on BaNaNaS is still just 2.77M
12:55:01 <Ahyangyi> What did you add
12:55:28 <andythenorth> I don't think there's a 2.77 MB Horse for many years 🙂
12:55:36 <andythenorth> v2 was 20 MB
12:55:44 <andythenorth> so was V3
12:56:40 <Ahyangyi> I see
12:56:58 <Ahyangyi> the "download size" of the in-game downloader seems wrong
12:57:05 <andythenorth> yeah adding 2815 vehicles has added 4.8 MB to the grf
12:57:15 <andythenorth> petern: ^ did we do a wrong thing? 😛
12:57:25 <pickpacket> 2815...
12:57:32 <pickpacket> how many are there in total now?
12:57:39 <glx[d]> it's so annoying, there's always a CI target with a missing implicit include
12:58:00 <andythenorth> 4001 vehicles in total right now
12:58:15 <pickpacket> andythenorth: not to be contrarian but who needs that many?
12:58:35 <andythenorth> nobody
12:58:36 *** birdjj has quit IRC (Read error: Connection reset by peer)
12:58:43 <andythenorth> it's not about need 😛
12:58:47 <pickpacket> :D
12:58:53 <pickpacket> this is true
12:58:55 <andythenorth> nobody needed 1186 vehicles in v3
12:59:04 <pickpacket> is it a popular newgrf?
12:59:12 <andythenorth> allegedly
12:59:28 <pickpacket> then it's all needed. Obviously
12:59:35 <pickpacket> it's what the fans crave!
13:02:36 <Ahyangyi> Andy, will you also make the hog bigger 😄
13:03:30 <andythenorth> I hope to yes
13:04:04 <pickpacket> andythenorth: what kind of vehicles are added?
13:05:44 <Rubidium_> glx[d]: expect that CI target to complain about missing fmt/format.h more often, as it's the one without precompiled headers
13:06:08 <glx[d]> oh yeah that may explain 🙂
13:07:38 <andythenorth> variants of wagons are what's added
13:09:14 <glx[d]> a lot of variants
13:10:12 <pickpacket> andythenorth: what kind of vehicles were already in it?
13:10:19 <andythenorth> should have used the livery refit spec as intended eh
13:10:34 <andythenorth> pickpacket it's just many more colours
13:10:45 <pickpacket> ah
13:11:02 <glx[d]> and the only useful one is random
13:11:30 <glx[d]> but still needs all the other internally
13:11:46 <andythenorth> well...kinda
13:12:02 <andythenorth> one day, RGB company colours, with presets, and the ability to set them per vehicle 😛
13:12:12 <andythenorth> could delete a lot of variants
13:12:27 <pickpacket> I guess different colours are useful for colour coding cargo wagons, but what are the random ones for then?
13:12:52 <glx[d]> cute non uniform trains
13:13:27 <glx[d]> but still with the same wagon type
13:16:13 <andythenorth> lol, we could do buyable variants, but they're all the same ID, with a different action 2 var
13:16:35 <andythenorth> saves some action 0 bytes 😛
13:19:25 <andythenorth> I think it's the per-variant name handling switches that eat bytes most right now
13:19:51 <andythenorth> I gave up trying to proceduralise them, I can't get string IDs to be reliably handled
13:20:07 <andythenorth> so they use about 3500 varact 2 chains
13:20:27 <andythenorth> instead of about 7
13:24:29 <petern> So explicit cargo subtypes before purchase? 😛
13:24:50 <andythenorth> such
13:25:09 <andythenorth> dare you call them 'liveries'? 😛
13:25:36 <andythenorth> also map an optional recolour sprite to them 😛
13:30:31 <Ahyangyi> andythenorth: I think I found it
13:30:33 <Ahyangyi> https://github.com/OpenTTD/BaNaNaS/blob/main/newgrf/43411222/versions/20230416T132052Z.yaml
13:30:50 <Ahyangyi> Somehow the file size in the metadata is wrong
13:32:45 <Ahyangyi> Alright, I'll create an issue just in case someone is going to fix it
13:34:20 <Rubidium_> is the size wrong, or is it the compressed size?
13:34:40 <Ahyangyi> Good point
13:35:26 <Ahyangyi> I ran zip with that tarball, and it's indeed 2.77 MB
13:35:50 <Ahyangyi> One fewer embarrassing Github issue 😄
13:38:24 <Rubidium_> that's probably why it's called download size in OpenTTD's content UI, and since it saves it compressed on the server it's the file size there
13:38:29 *** _aD has joined #openttd
13:46:09 <Ahyangyi> Yep, I only uploaded heightmaps before, which are generally incompressible PNGs. so somehow when I saw a size difference I wasn't thinking of compression...
13:48:23 <andythenorth> do we transmit grfs compressed on bananas?
13:48:27 * andythenorth should know this, it's basics
13:49:22 <andythenorth> oh the compression saving with zip is huge
13:50:02 <andythenorth> wonder why the grf makefiles default to tar, which seems to be uncompressed
13:50:22 <glx[d]> because basesets
13:50:41 <glx[d]> and the game expects uncompressed files anyway
13:52:30 <andythenorth> the compression does mean that the filesize isn't much
13:52:55 <andythenorth> 3.1 MB currently for dev Horse
13:53:01 <andythenorth> zipped
13:53:21 <glx[d]> yes but openttd can't use it 🙂
13:54:14 <andythenorth> wonder what the RAM use implication is
14:01:48 <Eddi|zuHause> why did we never allow reading zip/gz/whateverz files? can't be that complicated
14:02:04 <DorpsGek> [OpenTTD/OpenTTD] glx22 updated pull request #10848: Fix #10846: [Squirrel] Ensure sqvector size does not overflow https://github.com/OpenTTD/OpenTTD/pull/10848
14:04:54 <petern> We don't load all sprites into memory, IIRC we rely on random file access (possibly mmaped but probably not). This doesn't play will with compression.
14:04:56 <glx[d]> I know music can't even work from inside tar
14:05:25 <petern> Music could, but external players can't do it and for ages it was all extenral.
14:05:39 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #10848: Fix #10846: [Squirrel] Ensure sqvector size does not overflow https://github.com/OpenTTD/OpenTTD/pull/10848#pullrequestreview-1435410072
14:06:40 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler requested changes for pull request #10849: Add: Shift-click Rename on VehicleView to rename the group https://github.com/OpenTTD/OpenTTD/pull/10849#pullrequestreview-1435406784
14:07:38 <glx[d]> without looking at the comment I can say it's coding style 🙂
14:07:53 <petern> It's not.
14:08:08 <glx[d]> oh, but style is wrong everywhere
14:08:11 <Eddi|zuHause> ah right, it was random access
14:08:27 <petern> Okay, it is style and more 🙂
14:10:26 <andythenorth> hmm Horse compile is 1s per MB
14:10:31 <andythenorth> that's so meaningless correlation 🙂
14:10:41 <TallTyler> I like the feature, just not which button was hijacked to add it 🙂
14:10:53 <glx[d]> "show all vehicles that share this schedule" is so microscopic
14:11:34 <glx[d]> but it fits nicely between resize and scrollbar
14:11:39 <Eddi|zuHause> and why shift click and not ctrl click like normal people?
14:15:32 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #10849: Add: Shift-click Rename on VehicleView to rename the group https://github.com/OpenTTD/OpenTTD/pull/10849#pullrequestreview-1435411219
14:19:11 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #10849: Add: Shift-click Rename on VehicleView to rename the group https://github.com/OpenTTD/OpenTTD/pull/10849#pullrequestreview-1435411345
14:22:13 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #10849: Add: Shift-click Rename on VehicleView to rename the group https://github.com/OpenTTD/OpenTTD/pull/10849#pullrequestreview-1435411828
14:43:26 <DorpsGek> [OpenTTD/OpenTTD] glx22 merged pull request #10848: Fix #10846: [Squirrel] Ensure sqvector size does not overflow https://github.com/OpenTTD/OpenTTD/pull/10848
14:43:29 <DorpsGek> [OpenTTD/OpenTTD] glx22 closed issue #10846: [Crash]: script executing `array(2**62)` overflows allocated memory counter and crashes the game https://github.com/OpenTTD/OpenTTD/issues/10846
14:47:44 *** _aD has quit IRC (Ping timeout: 480 seconds)
14:48:00 <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #10847: Codechange: use fmt::format_to to format the help message https://github.com/OpenTTD/OpenTTD/pull/10847#pullrequestreview-1435414156
14:50:07 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #10847: Codechange: use fmt::format_to to format the help message https://github.com/OpenTTD/OpenTTD/pull/10847
14:51:32 *** Tirili has joined #openttd
14:51:34 <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #10844: Cleanup: rename smallvec_type.hpp https://github.com/OpenTTD/OpenTTD/pull/10844#pullrequestreview-1435414470
14:51:39 *** Tirili has quit IRC ()
14:53:14 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #10844: Cleanup: rename smallvec_type.hpp https://github.com/OpenTTD/OpenTTD/pull/10844
14:54:48 <DorpsGek> [OpenTTD/OpenTTD] fmang commented on pull request #10849: Add: Shift-click Rename on VehicleView to rename the group https://github.com/OpenTTD/OpenTTD/pull/10849#issuecomment-1555929513
15:07:10 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #10849: Add: Shift-click Rename on VehicleView to rename the group https://github.com/OpenTTD/OpenTTD/pull/10849#issuecomment-1555931945
15:16:41 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10849: Add: Shift-click Rename on VehicleView to rename the group https://github.com/OpenTTD/OpenTTD/pull/10849#pullrequestreview-1435416776
15:50:41 <DorpsGek> [OpenTTD/OpenTTD] fmang commented on pull request #10849: Add: Shift-click Rename on VehicleView to rename the group https://github.com/OpenTTD/OpenTTD/pull/10849#pullrequestreview-1435419698
16:00:36 *** Wormnest has joined #openttd
16:25:07 <DorpsGek> [OpenTTD/OpenTTD] michicc commented on pull request #10849: Add: Shift-click Rename on VehicleView to rename the group https://github.com/OpenTTD/OpenTTD/pull/10849#pullrequestreview-1435422647
16:30:43 *** Smedles has quit IRC (Quit: http://quassel-irc.org - Chat comfortably. Anywhere.)
16:31:01 *** Smedles has joined #openttd
16:41:15 *** Flygon has quit IRC (Quit: A toaster's basically a soldering iron designed to toast bread)
16:53:59 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #10849: Add: Shift-click Rename on VehicleView to rename the group https://github.com/OpenTTD/OpenTTD/pull/10849#issuecomment-1555951279
17:00:15 <petern> Why does industry "pct transported" exist, can it not be derived from production and transported?
17:13:22 <frosch> yes, cs wanted to save a division
17:16:01 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler opened pull request #10851: Change: Use "Via-Destination-Source" as default station cargodist display https://github.com/OpenTTD/OpenTTD/pull/10851
17:16:36 <TallTyler> ^^ Could use some opinions from people who play with Cargo Distribution enabled 🙂
17:16:53 <TallTyler> (Besides myself, of course)
17:19:51 <TallTyler> This was spurred by a recent conversation (https://discord.com/channels/142724111502802944/142725338831519744/1109509518871777372) as well as past recommendations
17:25:23 <DorpsGek> [OpenTTD/OpenTTD] EmperorJake commented on pull request #10851: Change: Use "Via-Destination-Source" as default station cargodist display https://github.com/OpenTTD/OpenTTD/pull/10851#issuecomment-1555956765
17:34:36 *** Compu has joined #openttd
17:38:18 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10851: Change: Use "Via-Destination-Source" as default station cargodist display https://github.com/OpenTTD/OpenTTD/pull/10851#issuecomment-1555958917
17:39:07 <petern> ^ "too many settings" vs "we have a setting for it, but you don't get to see it in the UI" is a bit pants really.
17:40:25 <DorpsGek> [OpenTTD/grfcodec] SoapGentoo commented on pull request #16: Gentoo patches https://github.com/OpenTTD/grfcodec/pull/16#issuecomment-1555959296
17:41:30 *** JGR has joined #openttd
17:41:30 <JGR> The setting is set whenever you change the mode using the dropdown, it doesn't need to be in the settings windows as well
17:41:51 <JGR> It's just a way of persisting the UI state
17:43:03 <petern> Hmm
17:43:43 <petern> Deleted.
17:44:00 <petern> I know I've changed it but it was on the default for whatever reason...
17:47:47 <glx[d]> Hmm but we don't have release workflow for grfcodec IIRC
17:49:23 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #10852: Codechange: replace seprintf with fmt::format for crashlog generation https://github.com/OpenTTD/OpenTTD/pull/10852
18:04:54 *** Compu has quit IRC (Quit: https://www.youtube.com/watch?v=5FjWe31S_0g)
18:15:19 *** _aD has joined #openttd
18:25:30 <andythenorth> orudge do the interstitial ads on the forums make much money? 🙂
18:25:33 <andythenorth> quite annoying 🙂
18:31:18 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #10852: Codechange: replace seprintf with fmt::format for crashlog generation https://github.com/OpenTTD/OpenTTD/pull/10852
18:33:57 <Rubidium_> TrueBrain: something to check for your survey thing... I think there won't be a survey sent for Windows upon a crash. As far as I can see it does not call MakeCrashLog which contains the call for sending the survey
18:34:17 <andythenorth> Horse 25.2 MB
18:34:20 <andythenorth> giant grf is giant
18:34:29 <petern> Well
18:34:40 <petern> Probably smaller than me.
18:34:54 <petern> Probably smaller than this branch which will probably never see the light of day.
18:36:28 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #10852: Codechange: replace seprintf with fmt::format for crashlog generation https://github.com/OpenTTD/OpenTTD/pull/10852
18:39:36 <DorpsGek> [OpenTTD/OpenTTD] DorpsGek pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/98dffc31577d89351d7b7e3b00dea6f68ed9ceb7
18:39:37 <DorpsGek> - Update: Translations from eints (by translators)
18:41:46 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #10852: Codechange: replace seprintf with fmt::format for crashlog generation https://github.com/OpenTTD/OpenTTD/pull/10852
18:46:59 <andythenorth> 25.2 MB of diff? 😛
18:57:55 <petern> Okay, only 48KB so far.
19:10:32 *** Wormnest has quit IRC (Ping timeout: 480 seconds)
19:16:47 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #10852: Codechange: replace seprintf with fmt::format for crashlog generation https://github.com/OpenTTD/OpenTTD/pull/10852
19:45:55 *** Wormnest has joined #openttd
19:57:40 <DorpsGek> [OpenTTD/OpenTTD] github-code-scanning[bot] commented on pull request #10852: Codechange: replace seprintf with fmt::format for crashlog generation https://github.com/OpenTTD/OpenTTD/pull/10852#pullrequestreview-1435441064
20:38:40 *** Wormnest has quit IRC (Ping timeout: 480 seconds)
20:51:25 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #10852: Codechange: replace seprintf with fmt::format for crashlog generation https://github.com/OpenTTD/OpenTTD/pull/10852#pullrequestreview-1435445617
21:16:22 *** Wormnest has joined #openttd
21:19:27 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
21:27:48 <DorpsGek> [OpenTTD/OpenTTD] anatolyeltsov updated pull request #10541: Feature: Industry production graph https://github.com/OpenTTD/OpenTTD/pull/10541
21:34:22 *** keikoz has quit IRC (Ping timeout: 480 seconds)
21:43:25 <petern> Hmm
22:03:38 *** tokai|noir has joined #openttd
22:03:38 *** ChanServ sets mode: +v tokai|noir
22:06:16 *** ralph[m] has left #openttd
22:09:02 *** Wormnest has quit IRC (Ping timeout: 480 seconds)
22:10:19 *** tokai has quit IRC (Ping timeout: 480 seconds)
22:25:22 <petern> Hmm, is SLE_ARR with size 1 the same as SLE_VAR?
22:31:58 <glx[d]> array store the number of elements I think
22:35:02 <petern> Hmm, well, this is quite a restructure. Hmm.
22:43:53 *** Wormnest has joined #openttd
22:45:21 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #10853: Change: Reorganise industry accept/produce arrays. https://github.com/OpenTTD/OpenTTD/pull/10853
22:47:57 <DorpsGek> [OpenTTD/OpenTTD] James103 commented on pull request #10853: Change: Reorganise industry accept/produce arrays. https://github.com/OpenTTD/OpenTTD/pull/10853#pullrequestreview-1435453193
22:48:45 <petern> Such saveload code
22:51:57 *** _aD has quit IRC (Ping timeout: 480 seconds)
23:26:16 <DorpsGek> [OpenTTD/OpenTTD] github-code-scanning[bot] commented on pull request #10853: Change: Reorganise industry accept/produce arrays. https://github.com/OpenTTD/OpenTTD/pull/10853#pullrequestreview-1435455207
23:33:57 *** nielsm has quit IRC (Ping timeout: 480 seconds)
23:37:02 *** Wormnest has quit IRC (Ping timeout: 480 seconds)
23:39:21 *** VincentKadar[m]1234567 has joined #openttd
continue to next day ⏵