IRC logs for #openttd on OFTC at 2023-04-25
            
01:06:25 *** _aD has quit IRC (Quit: leaving)
01:08:37 *** TROILUS has quit IRC (Ping timeout: 480 seconds)
01:21:03 *** Flygon has quit IRC (Quit: A toaster's basically a soldering iron designed to toast bread)
02:19:50 *** Wormnest has quit IRC (Quit: Leaving)
02:37:12 *** D-HUND has joined #openttd
02:40:44 *** debdog has quit IRC (Ping timeout: 480 seconds)
02:42:56 *** D-HUND is now known as debdog
03:31:47 *** keikoz has joined #openttd
04:23:43 *** felix has quit IRC ()
04:23:54 *** felix has joined #openttd
05:57:56 <andythenorth> wonder what happened to the nfo -> nml converter
05:58:13 *** WormnestAndroid has quit IRC (Remote host closed the connection)
05:58:14 <andythenorth> nfo FIRS was converted by a script initially, then manual clean up
05:58:16 *** WormnestAndroid has joined #openttd
06:23:12 <petern> It was converted by a tool.
06:31:41 <andythenorth> so maybe I do nest variants 2 levels
06:31:59 <andythenorth> I want 'random - more colours' and 'random - fewer colours' 😛
06:32:28 <andythenorth> random has 2 dimensions, colour, and shape
06:34:35 <petern> So you need a third that includes both types of random...
06:34:42 <andythenorth> yes
06:34:57 <petern> How many random bits are you already using...
06:34:59 <andythenorth> variants of random variant variants
06:35:04 <andythenorth> I use 8 bits currently
06:35:14 <andythenorth> and sometimes the bits overlap, in a way that is known
06:35:54 <andythenorth> but it looks ok
06:36:53 <andythenorth> oh I might be about to free up 1 bit
06:36:57 <andythenorth> by deleting a feature
06:37:37 <andythenorth> nah, that's the one that already overlaps and is used for 2 different switches
06:38:23 <petern> There's a draft PR around to increase that.
06:38:42 <andythenorth> is it a stash?
06:39:07 <petern> No, an actual PR 😛
06:39:14 <andythenorth> I have seen it 🙂
07:28:49 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #10716: Change: Set baseset description & video driver info size after initial sizing. https://github.com/OpenTTD/OpenTTD/pull/10716
08:23:12 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #10716: Change: Set baseset description & video driver info size after initial sizing. https://github.com/OpenTTD/OpenTTD/pull/10716
08:24:08 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10716: Change: Set baseset description & video driver info size after initial sizing. https://github.com/OpenTTD/OpenTTD/pull/10716#issuecomment-1521380269
09:00:34 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #10716: Change: Set baseset description & video driver info size after initial sizing. https://github.com/OpenTTD/OpenTTD/pull/10716#pullrequestreview-1399456990
09:08:57 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #10717: Codechange: Define Date/Year/Month/Day within TimerGameCalendar class https://github.com/OpenTTD/OpenTTD/pull/10717#pullrequestreview-1399460942
09:09:45 <TrueBrain> hmm .. if we would be to add `TimerGameEconomy::Year`, can we make a compiler throw a warning if you assign `TimerGameCalendar::Year` to a variable defined as `TimerGameEconomy::Year`?
09:12:16 <TrueBrain> guess we only can if we make it a struct
09:29:56 <Eddi|zuHause> you mean like a unit conversion error?
09:33:05 <Eddi|zuHause> "static dimensional analysis"
09:34:21 <petern> Yeah, it's annoying that typedefs aren't distinct and conflict. It would probably be annoying if they did too...
09:35:02 <TrueBrain> yeah, I guess ... making it a struct where you define the conversions allowed would be the proper way
09:35:03 <TrueBrain> just a shame ..
09:37:46 <TrueBrain> okay, scaffolding for survey done .. now to collect the actual information 🙂
09:48:30 <TrueBrain> `'settings': {'gui.sg_full_load_any': 'false', '`
09:48:40 <TrueBrain> that seems fine .. just booleans are encoded as strings .. but meh, who cares 😛
09:52:51 <Rubidium_> how would you encode booleans other than as strings?
09:53:03 <TrueBrain> as a .. JSON boolean?
09:53:08 <TrueBrain> like all normal humans would 😄
09:53:33 <Eddi|zuHause> and where would you find a normal human?
09:53:43 <Rubidium_> well, that's still mostly string
09:54:03 <TrueBrain> that is a weird thing to say Rubidium; in that definition, everything is a string 🙂
09:54:11 <petern> Without quotes is the JSON way...
09:54:24 <petern> andy is it lunch yet?
09:55:07 * TrueBrain does away with all types in OpenTTD and changes it all to std::string :P
09:55:16 <Rubidium_> I was thinking in the realm of 0 and 1 for a boolean, but that's essentially undefined
09:55:26 <TrueBrain> JSON has booleans very well defined
09:55:30 <TrueBrain> like numbers
09:55:35 <TrueBrain> which are now encoded as strings too .. but who cares
09:56:10 <TrueBrain> I don't feel like changing the settings variants to allow me to export the correct type .. too much effort, too little gain 🙂
09:58:01 <petern> Awww 😄
09:58:22 <petern> There's the settings that have names AND numbers depending on what the value is.
09:59:54 <TrueBrain> settings are confusing ...
10:00:16 <TrueBrain> I also don't understand how our ini-saveload works correctly 😛
10:00:20 <TrueBrain> let's delve in a bit deeper ...
10:01:03 <petern> I wonder who wrote it.
10:01:23 <TrueBrain> the thing I am trying to understand .. `_network_settings` is mostly in `_settings_client`
10:01:32 <TrueBrain> but the IniLoad function uses `_settings_game` to load it from
10:01:42 <TrueBrain> so something is telling where the actual value is stored ..
10:07:52 <TrueBrain> `SDTG` vs `SDTC` ... and handled by the generator
10:08:03 <TrueBrain> that is ... hidden ... deeeeeeeeeeep insde 😛
10:10:27 <petern> Yeah all this stuff is very magic.
10:16:50 <TrueBrain> `"display_opt": "SHOW_TOWN_NAMES|SHOW_STATION_NAMES|SHOW_SIGNS|WAYPOINTS|SHOW_COMPETITOR_SIGNS",`
10:16:51 <TrueBrain> hihi
10:46:18 *** Wolf01 has joined #openttd
10:47:10 <TrueBrain> `"graphics_set": "OpenGFX (7938)",`
10:47:33 <TrueBrain> `"blitter": "32bpp-optimized",`
10:47:43 <TrueBrain> turns out, crashlog is a good source for information 😄
11:10:22 *** WormnestAndroid has quit IRC (Remote host closed the connection)
11:10:28 *** WormnestAndroid has joined #openttd
11:18:38 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
11:18:46 *** WormnestAndroid has joined #openttd
11:31:23 *** WormnestAndroid has quit IRC (Remote host closed the connection)
11:31:52 *** WormnestAndroid has joined #openttd
11:38:39 *** Flygon has joined #openttd
11:41:43 <petern> Bug reports eh...
11:42:43 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #10574: [Bug]: Map generation fails when no town buildable roads are available https://github.com/OpenTTD/OpenTTD/issues/10574
12:11:30 <andythenorth> It was lunch
12:11:48 <LordAro> can confirm
12:12:47 <petern> Yes, I had a salad, and then the last bit of an easter egg, and a cinnamon bun thing and I'm still looking for more food. Hmm.
12:12:54 <petern> Maybe a coffee.
12:12:57 <pickpacket> I'll have to do some testing to confirm, but this morning it seemed helicopters wouldn't auto-replace on my map
12:13:21 <petern> Wasn't there something funky with helicopters searching for depots.
12:15:33 <pickpacket> could be that
12:17:40 <glx[d]> do they have only heliport in their orders ?
12:19:18 <petern> Why do I never have enough input or mixer channels?
12:19:42 <petern> Maybe I should try repairing my old 3-bus mixer that blew up.
12:22:08 <Eddi|zuHause> yeah, aircraft don't look for hangars outside their destinations
12:23:26 <petern> Damn, Doom's music is good.
12:24:05 <petern> (Obviously I knew that)
12:27:27 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #10716: Change: Set baseset description & video driver info size after initial sizing. https://github.com/OpenTTD/OpenTTD/pull/10716
12:30:50 <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #10678: Codechange: use std::string for base media filename/warning storage https://github.com/OpenTTD/OpenTTD/pull/10678#pullrequestreview-1399812175
12:39:27 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #10717: Codechange: Define Date/Year/Month/Day within TimerGameCalendar class https://github.com/OpenTTD/OpenTTD/pull/10717#pullrequestreview-1399825830
12:45:00 <pickpacket> glx[d]: they go between a heliport and a heli depot
12:46:50 <pickpacket> And when I told them to go to depot for servicin they did, and were replaced
13:01:12 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #10678: Codechange: use std::string for base media filename/warning storage https://github.com/OpenTTD/OpenTTD/pull/10678
13:31:27 *** nielsm has joined #openttd
13:50:42 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler updated pull request #10717: Codechange: Define Date/Year/Month/Day within TimerGameCalendar class https://github.com/OpenTTD/OpenTTD/pull/10717
14:01:20 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler updated pull request #10717: Codechange: Define Date/Year/Month/Day within TimerGameCalendar class https://github.com/OpenTTD/OpenTTD/pull/10717
14:01:44 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #10717: Codechange: Define Date/Year/Month/Day within TimerGameCalendar class https://github.com/OpenTTD/OpenTTD/pull/10717#issuecomment-1521844597
14:43:58 <TrueBrain> didn't Rubidium make something recently to convert md5sum to something readable in string format?
14:45:10 <TrueBrain> md5sumToString exists, but that is an annoying function 😛
14:55:10 <Rubidium_> I think I've not yet reached that point
15:00:26 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #10684: Codechange: replace error/usererror printf variant with fmt variant and rename https://github.com/OpenTTD/OpenTTD/pull/10684
15:01:11 <TrueBrain> meh; I was sure I saw something related, but I guess not 😄
15:10:25 <petern> I looked at md5sum but probably went with something less useful.
15:11:33 <petern> Oh I misread the last part, not converting to string.
15:11:56 <TrueBrain> https://gist.github.com/TrueBrain/1a87754b5d5cc8dedceabda826acf923?permalink_comment_id=4548020#gistcomment-4548020
15:11:56 <TrueBrain> Example JSON payload for the survey. Still missing some small things, but most info should be there now
15:12:12 <TrueBrain> (GRF parameters is mostly missing)
15:39:42 <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #10684: Codechange: replace error/usererror printf variant with fmt variant and rename https://github.com/OpenTTD/OpenTTD/pull/10684#pullrequestreview-1400218563
15:50:09 <Rubidium_> TrueBrain: did you maybe mean FormatArrayAsHex?
15:50:12 <petern> <https://www.nextptr.com/tutorial/ta1425731095/typesafe-integer-types-with-list-initialization-of-scoped-enums> suggests using enum class to have type-safe integers... huh
15:50:54 <petern> Looks pretty awkward though.
15:51:20 <glx[d]> https://github.com/OpenTTD/OpenTTD/compare/master...glx22:OpenTTD:nml <-- still many files to convert, but cmake side is working
15:51:54 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10655: Change: base autosaves intervals on real time (instead of game time) https://github.com/OpenTTD/OpenTTD/pull/10655#pullrequestreview-1400241403
15:54:11 <Rubidium_> petern: isn't that what strong_typedef_type essentially is?
15:55:14 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #10684: Codechange: replace error/usererror printf variant with fmt variant and rename https://github.com/OpenTTD/OpenTTD/pull/10684
16:01:29 <petern> Is that a thing?
16:01:32 <petern> <https://groups.google.com/a/isocpp.org/g/std-proposals/c/Y-7cdQcNDrk>
16:01:33 <petern> lol
16:19:13 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #10718: Cleanup: remnants of OTTD_PRINTF, str_fmt and vseprintf https://github.com/OpenTTD/OpenTTD/pull/10718
16:23:03 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #10718: Cleanup: remnants of OTTD_PRINTF, str_fmt and vseprintf https://github.com/OpenTTD/OpenTTD/pull/10718#pullrequestreview-1400298845
16:23:04 *** Wormnest has joined #openttd
16:25:57 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #10718: Cleanup: remnants of OTTD_PRINTF, str_fmt and vseprintf https://github.com/OpenTTD/OpenTTD/pull/10718#pullrequestreview-1400298436
16:29:35 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #10718: Cleanup: remnants of OTTD_PRINTF, str_fmt and vseprintf https://github.com/OpenTTD/OpenTTD/pull/10718#pullrequestreview-1400310665
16:30:35 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #10718: Cleanup: remnants of OTTD_PRINTF, str_fmt and vseprintf https://github.com/OpenTTD/OpenTTD/pull/10718
16:31:58 <Eddi|zuHause> petern: that reads like every grf spec discussion ever: "we have <hack>, if we now put <hack> on <hack> we can achieve <X> without having to implement it properly"
16:32:46 <Eddi|zuHause> say, requesting depot flipping to be a counter. or flipping graphics for driving backwards
16:34:52 <Eddi|zuHause> (admittedly, using var61 to implement longer vehicles falls into that same category)
16:37:58 *** HerzogDeXtEr has joined #openttd
16:55:08 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #10655: Change: base autosaves intervals on real time (instead of game time) https://github.com/OpenTTD/OpenTTD/pull/10655#pullrequestreview-1400350054
16:57:22 <TrueBrain> Rubidium_: yup, sadly, it doesn't work in this case, as md5sum is an `uint8*` array 😛
16:57:35 <TrueBrain> so doing it old-skool 😄
17:08:22 <Rubidium_> just refactor it, so it isn't a uint8* anymore?
17:08:29 <TrueBrain> no tnx 🙂
17:08:41 <TrueBrain> otherwise this PR never finishes 😛
17:08:58 <TrueBrain> almost refactored all of crashlog too ... such scary code, all those seprintfs 🙂
17:14:04 <petern> Hm.
17:15:57 <TrueBrain> yes dear? 🙂
17:18:03 <petern> Header hell.
17:19:28 <petern> script_instance.cpp depends on newgrf_config.h somewhere :p
17:19:39 <TrueBrain> somewhere, yes 😄
17:23:11 <TrueBrain> hmm .. for survey, I kinda need an unique id per savegame .. but all we have for uuid-like things are very old (read: not really an uuid :D)
17:24:43 <andythenorth> Eddi|zuHause: Fortunately we waited out the depot flip idea until we had something better 😛
17:25:33 <TrueBrain> and our random is only 32bit .. which is a bit .. too small for this .. hmm
17:25:38 <Eddi|zuHause> andythenorth: you mean that 12 year old proposal? :p
17:26:22 <andythenorth> Variants
17:26:43 <Eddi|zuHause> yes. that one
17:27:20 <Eddi|zuHause> i can assure you, that we've been discussing that even before you came up with the depot flip hack
17:39:38 *** argoneus has quit IRC (Quit: The Lounge - https://thelounge.chat)
17:46:30 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #10718: Cleanup: remnants of OTTD_PRINTF, str_fmt and vseprintf https://github.com/OpenTTD/OpenTTD/pull/10718
17:49:51 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #10718: Cleanup: remnants of OTTD_PRINTF, str_fmt and vseprintf https://github.com/OpenTTD/OpenTTD/pull/10718
17:50:59 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #10718: Cleanup: remnants of OTTD_PRINTF, str_fmt and vseprintf https://github.com/OpenTTD/OpenTTD/pull/10718#pullrequestreview-1400431176
17:52:01 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain opened pull request #10719: Feature: opt-in survey when exiting a game https://github.com/OpenTTD/OpenTTD/pull/10719
17:52:16 <TrueBrain> curious if I got the CI correct in one go
17:53:12 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain updated pull request #10719: Feature: opt-in survey when exiting a game https://github.com/OpenTTD/OpenTTD/pull/10719
17:53:12 <Eddi|zuHause> in my experience, that never works :p
17:55:51 *** gelignite has joined #openttd
18:04:14 <TrueBrain> yippie, CI works 😄 vcpkg is so incredibly useful 😄
18:06:01 <TrueBrain> oops ... using a `Random()` extra changes the regression ofc 😄 Haha 🙂 So silly
18:06:26 <TrueBrain> but, the important part was: can we use this JSON library; answer: yes 🙂
18:12:40 <Eddi|zuHause> all hail JSON
18:13:49 *** lobstarooo has joined #openttd
18:13:50 *** lobstarooo has quit IRC ()
18:27:46 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #10667: Codechange: Use more std::string in fios land https://github.com/OpenTTD/OpenTTD/pull/10667#pullrequestreview-1400400240
18:30:27 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #10708: Change: Hide all variants from UI when (display) parent is hidden. https://github.com/OpenTTD/OpenTTD/pull/10708#pullrequestreview-1400493248
18:41:24 <DorpsGek> [OpenTTD/OpenTTD] DorpsGek pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/5794590b361fafb297e7c770492b774a0e7c6e1b
18:41:25 <DorpsGek> - Update: Translations from eints (by translators)
18:53:35 *** HerzogDeXtEr has quit IRC (Read error: Connection reset by peer)
19:03:37 *** WormnestAndroid has quit IRC (Remote host closed the connection)
19:03:38 *** WormnestAndroid has joined #openttd
19:34:15 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #10708: Change: Hide all variants from UI when (display) parent is hidden. https://github.com/OpenTTD/OpenTTD/pull/10708
19:36:08 <dP> what happens if you buy already hidden vehicle and it becomes parent?
19:36:40 <Eddi|zuHause> this is inside the buy menu, i presume
19:37:50 <petern> If it's hidden, it stays hidden. If it's not hidden, it stays unhidden.
19:38:16 <TrueBrain> I think you are hiding something ..... 😛
19:39:09 <dP> so you can still get hidden vehicle to show up because it's a parent?
19:41:32 <Eddi|zuHause> what's hidden cannot be seen
19:45:24 <TrueBrain> this is why I don't like kiekaboe
20:07:09 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #10713: Codechange: Use vector instead of mallloc/free for Action 6 data. https://github.com/OpenTTD/OpenTTD/pull/10713#pullrequestreview-1400670800
20:18:05 *** gelignite has quit IRC (Quit: Stay safe!)
20:19:46 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #10695: Fix #8177: Ships with max speed overflow to near-zero speed https://github.com/OpenTTD/OpenTTD/pull/10695#pullrequestreview-1400687836
20:22:41 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #10669: Fix #10627: Houses subsitute specs should only be copied on first definition. https://github.com/OpenTTD/OpenTTD/pull/10669#pullrequestreview-1400694460
20:26:01 <glx[d]> oh we use the standard 1-2-3 in orig_extra
20:46:02 *** nielsm has quit IRC (Ping timeout: 480 seconds)
21:15:58 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
21:16:48 *** WormnestAndroid has joined #openttd
21:24:51 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
21:27:42 *** WormnestAndroid has joined #openttd
21:40:22 *** _aD has joined #openttd
21:46:51 *** keikoz has quit IRC (Ping timeout: 480 seconds)
21:51:04 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
22:12:26 *** debdog has quit IRC (Ping timeout: 480 seconds)
22:12:27 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
22:12:54 *** WormnestAndroid has joined #openttd
23:43:15 *** _aD has quit IRC (Quit: leaving)