Times are UTC Toggle Colours
00:33:16 *** wallabra has joined #openttd 02:08:43 *** Wormnest has quit IRC 02:17:20 *** Flygon has quit IRC 02:58:42 *** D-HUND has joined #openttd 03:02:05 *** debdog has quit IRC 03:10:16 *** NGC3982 has quit IRC 03:13:26 *** NGC3982 has joined #openttd 03:19:30 *** D-HUND is now known as debdog 03:37:32 *** keikoz has joined #openttd 03:47:31 *** Deep3D has quit IRC 03:51:52 *** Deep3D has joined #openttd 04:09:39 *** bryjen has joined #openttd 04:18:55 *** tokai has joined #openttd 04:18:55 *** ChanServ sets mode: +v tokai 04:25:53 *** tokai|noir has quit IRC 05:43:38 *** bryjen has quit IRC 05:58:35 *** keikoz has quit IRC 06:30:08 <truebrain> "Oops" 06:30:22 <truebrain> At least there was room to expand π 06:42:46 *** keikoz has joined #openttd 07:03:37 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain updated pull request #11232: Change: store crash logs in JSON format https://github.com/OpenTTD/OpenTTD/pull/11232 07:03:51 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain updated pull request #11251: Codechange: [Script] use nlohmann for Squirrel <-> JSON conversion https://github.com/OpenTTD/OpenTTD/pull/11251 07:07:23 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #11259: Codechange: Use begin/end of nwidget parts instead of begin/length. https://github.com/OpenTTD/OpenTTD/pull/11259#pullrequestreview-1608888958 07:09:00 <peter1139> Oops 07:11:50 <truebrain> hmmm ... I am getting a bit annoyed by the performance of Python in our backend lately .. can Cloudflare accept random TCP connections already? No .. darn .. π 07:13:16 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #11258: Change: Replace fixed length _grf_text array with vector. https://github.com/OpenTTD/OpenTTD/pull/11258 07:13:18 <truebrain> also, my statistics of the backend is funny .. we have a negative amount of active turn connections, it says. For some reason, connections can drop without them going via disconnect functions .. 07:14:03 <truebrain> peter1139: can I nitpick about sentences not ending with dots, or shall I just leave that? π 07:14:10 <truebrain> (it is such a nitpick ....) 07:15:24 <peter1139> Oops. 07:15:47 <peter1139> (Can you be more specific, sorry...) 07:16:11 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #11258: Change: Replace fixed length _grf_text array with vector. https://github.com/OpenTTD/OpenTTD/pull/11258#pullrequestreview-1608902406 07:16:15 <truebrain> sure; I just never know if that is worth complaining about, honestly π 07:17:24 <truebrain> performance wise, this array is terrible π All operations are O(n) .. guess as long as it is small enough, it is not worth the overhead of a more complex structure. Also not for this PR btw. 07:18:19 <truebrain> the key seems to be (grfid, stringid), and the value (textholder, def_string) 07:18:44 <truebrain> definitially not for this PR π 07:21:32 <peter1139> Yes, I suppose so. One detail is that the index is looked up and passed around, so using some other keyed container must also support indexed access. 07:22:01 <truebrain> so another lookup from index to that pair .. yeah, it gets complicated really quick π 07:24:35 <peter1139> We could additionally store a map, per NewGRF, of stringid to index. Nearly every place (but not all) that uses GetGRFStringID also has access to GRFFile *. 07:25:35 <truebrain> I guess that is only worth it if profiling shows this is a performance issue; it was more the pedanticness in me that went: wait .. a `find_if` on a vector to find a pair .. that smells like a `std::map` π 07:25:38 <peter1139> Or, they are allocated sequentially, presumably one NewGRF at a time. So it may be possible to limit the search by remembering the first and last IDs used by a NewGRF. 07:26:09 <truebrain> but I doubt this show up in any profile; NewGRF does a lot more stuff that takes more time π 07:27:37 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #11259: Codechange: Use begin/end of nwidget parts instead of begin/length. https://github.com/OpenTTD/OpenTTD/pull/11259 07:27:57 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #11259: Codechange: Use begin/end of nwidget parts instead of begin/length. https://github.com/OpenTTD/OpenTTD/pull/11259#pullrequestreview-1608922561 07:28:04 <truebrain> I assume you tested it; was too lazy to validate π 07:28:58 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #11258: Change: Replace fixed length _grf_text array with vector. https://github.com/OpenTTD/OpenTTD/pull/11258 07:29:01 <peter1139> It wouldn't be the first time! 07:29:14 <peter1139> (Although I do tend to say if I've not tested it...) 07:29:18 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #11258: Change: Replace fixed length _grf_text array with vector. https://github.com/OpenTTD/OpenTTD/pull/11258#pullrequestreview-1608924635 07:30:14 <peter1139> It took me a little parsing to understand what the returned widget counters were being used for. 07:30:59 <truebrain> I just went: I am sure he thought about if this was the right thing to return π 07:31:01 <truebrain> π 07:35:22 <peter1139> My car tax increased from Β£30 to Β£35. Definitely paying for all the damage it does there... :/ 07:35:50 <truebrain> percentage wise, that is a pretty big increase 07:37:28 <peter1139> So that 20MB static array doesn't appear to show up in RES in top. 07:37:57 <truebrain> as frosch mentioned, most likely only when a page is being used, it is mapped in 07:43:03 <peter1139> Ah yes, comparing a blank savegame with FIRS loaded, shows about 20MB less used. 07:43:51 <truebrain> tomorrow I will run my performance benchmark, see how many games changes in memory-usage because of it π 07:46:18 <peter1139> Weird though, I can't remember what I was looking at when I spotted that and the array size jumped out that me. 07:52:51 <truebrain> as that goes π 07:56:08 <truebrain> now the question .. do I keep trying to fix the backend applications, to get more performance / less issues out of them 07:56:21 <andythenorth> peter1139: don't you have to complain about cyclists not paying for roads or something? 07:56:29 <andythenorth> isn't that expected tradition? 07:56:30 <truebrain> or do I accept that sooner or later things like Cloudflare allow WASM on custom TCP ports, and already go build it in WASM .. 07:56:31 <truebrain> hmmm 07:57:07 <peter1139> andythenorth, obviously I'm too busy jumping red lights and breaking the speed limit to do that. 08:49:14 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #11258: Change: Replace fixed length _grf_text array with vector. https://github.com/OpenTTD/OpenTTD/pull/11258 08:54:02 <andythenorth> does this make the game snappier? 09:19:51 <peter1139> No, it is unlikely to affect performance at all, but does prevent 20MB being allocated for no reason. 09:20:50 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #11259: Codechange: Use begin/end of nwidget parts instead of begin/length. https://github.com/OpenTTD/OpenTTD/pull/11259 09:28:09 <peter1139> Apparently I didn't press the Enable auto-merge button hard enough. 09:50:33 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #11232: Change: store crash logs in JSON format https://github.com/OpenTTD/OpenTTD/pull/11232#issuecomment-1704960210 10:05:09 *** virtualrandomnumber has joined #openttd 10:05:37 *** virtualrandomnumber has quit IRC 10:13:36 <truebrain> lol, whether a setting is its default value or not is a lot more tricky then I would like π 10:13:44 <truebrain> for the GUI it is simple, as it only deals with integer-based settings 10:13:53 <truebrain> but internally, we also have more complicated settings, like strings and lists π 10:17:28 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #11232: Change: store crash logs in JSON format https://github.com/OpenTTD/OpenTTD/pull/11232#issuecomment-1704999409 10:18:31 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain updated pull request #11232: Change: store crash logs in JSON format https://github.com/OpenTTD/OpenTTD/pull/11232 10:20:06 <truebrain> https://cdn.discordapp.com/attachments/1008473233844097104/1148200811919310929/image.png 10:20:06 <truebrain> that is one weird setting ... 10:21:19 <truebrain> `autosave_interval` ... didn't we remove that? 10:21:43 <truebrain> ah ... he forgot something essential .. π 10:22:09 <peter1139> No, it was just changed from a list to a value. 10:22:33 <truebrain> no, it was "fixed" to not be in the list 10:22:36 <truebrain> but that commit has a mistake 10:23:37 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain opened pull request #11260: Fix 8c9ecde9: actually remove autosave_interval from setting window https://github.com/OpenTTD/OpenTTD/pull/11260 10:23:57 <truebrain> so that makes it a fix on a fix, we are so good in this software development stuff π 10:24:37 <peter1139> Always blame the reviewer ;-) 10:25:04 *** gelignite has joined #openttd 10:25:10 <truebrain> and the reviewer not actually testing what he is approving .. and the commiter not actually testing .. and owh owh owh, so many people to blame here π 10:25:23 <peter1139> I have to say, I find the manual `interface->Add()` lines a bit of a weird way to do it :-) 10:25:44 <truebrain> I agree 10:25:47 <truebrain> but I aint touching that! 10:25:57 <peter1139> Indeed. Definitely not in this PR. :) 10:26:09 <truebrain> already had to look into how the GUI actually shows what settings are non-default .. which is some weird code, now we made the settings into a variant 10:26:30 <truebrain> I also got annoyed that it is `small_font` and `medium_font`, instead of `font.small` and `font.medium` .. 10:26:44 <truebrain> but ... I aint touching that either! π 10:26:57 <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #11260: Fix 8c9ecde9: actually remove autosave_interval from setting window https://github.com/OpenTTD/OpenTTD/pull/11260#pullrequestreview-1609241116 10:27:43 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #11260: Fix 8c9ecde9: actually remove autosave_interval from setting window https://github.com/OpenTTD/OpenTTD/pull/11260#issuecomment-1705013696 10:28:19 <truebrain> `"pf.forbid_90_deg": "false",` ... wait, why do I have this setting at false ... 10:28:25 <truebrain> I guess I was testing shit when we talked about it π 10:28:47 <truebrain> my configuration is such a mess of "things I wanted to test", making it nearly impossible to start a proper game without going: what the actual fuck 10:30:16 <_glx_> Old default ? 10:30:20 <truebrain> no 10:30:28 <truebrain> there was a thing with that setting a while ago 10:30:35 <truebrain> so I was flipping it around to test the change for that 10:30:43 <truebrain> clearly ... it was left in an unwanted state π 10:31:26 <_glx_> I once had `select_goods` in the wrong state 10:36:43 <truebrain> okay, in #11232 I took no effort to see if a list was different from default .. turns out, only resolution is using a list in modern games 10:36:45 <truebrain> so that is fine π 10:36:58 <truebrain> also fine to not put any effort in it π 10:37:26 <truebrain> owh, I am wrong .. `music.custom_1` and friend are also lists 10:37:28 <truebrain> still 10:48:30 <_glx_> Not important settings 11:16:07 <peter1139> Why is cake? 11:16:28 <andythenorth> why is coffee? (see also) 11:17:38 <peter1139> I've gone from "hmm, probably need to eat a little bit more" to "oh shit, I definitely need to eat less" pretty quickly :( 11:18:51 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain merged pull request #11260: Fix 8c9ecde9: actually remove autosave_interval from setting window https://github.com/OpenTTD/OpenTTD/pull/11260 11:19:27 <peter1139> Although to be fair yesterday was the club BBQ so bound to be temporarily a bit higher after that... 11:22:26 *** Flygon has joined #openttd 12:38:46 <peter1139> https://github.com/microsoft/vscode/issues/146422 Hmm, known but no fix. 12:57:35 <truebrain> that is annoying ... lol 13:05:40 *** gelignite has quit IRC 13:12:57 <truebrain> also interesting: `micro 1708 3.0 0.4 62915453608 67924 pts/1 Sl+ 14:59 0:00 python test.py` 13:13:05 <truebrain> yes, you read that right .. 62 TiB of VSZ .. 13:21:03 <truebrain> never have I ever had an application where I had to worry about VSZ .. but this is reaching a limit π 13:21:29 <peter1139> Weird. 13:21:43 <truebrain> very curious what they are doing to cause that .... 13:22:28 <truebrain> it is a Rust Python module .. let's try in native Rust, see if that shows the same issue .. 13:30:15 <truebrain> in native Rust it does exactly the same .. oh-oh .. that isn't good π 13:35:01 <peter1139> Rust: ensuring it won't corrupt memory by never reusing it... 13:36:46 <truebrain> this isn't Rust 13:36:53 <truebrain> it is the application I am working with .. 13:36:57 <truebrain> (WASM stuff) 13:38:02 <truebrain> after drilling down .. the WASM application is requesting memory, and the engine I am testing happily does so ... but for some reason a bit more than one would assume π 13:38:11 <truebrain> in WASM you allocate memory in pages of 64 KiB 13:38:29 <truebrain> so I request 1 page ... and ... it allocates like .... much much much more π 13:39:01 <truebrain> 6GB, instead of 64 KiB .. 13:39:05 <truebrain> lolz 13:39:44 <peter1139> Sorry, I guess I was confused by you mentioning rust three times in two lines. 13:40:01 <truebrain> yeah, I wasn't really clear in what I was doing .. I was just flabbergasted π 13:58:19 <truebrain> okay, it is this wasm engine that is the issue ... lol. What a weird issue .. 14:19:59 <truebrain> haha, each instance of the WASM I was spinning up creates a virtual allocation of 4GB 14:20:03 <truebrain> so that explains π 14:22:15 <peter1139> Yikes. 14:22:25 *** gelignite has joined #openttd 14:22:45 <peter1139> Urgh, everything on Linux seems to have its own different looking widget toolkit :p 14:23:26 <peter1139> Some of them have (fairly prominent) settings to make them look like other widget toolkits, but don't actually do that, they just look like something else again. 14:51:11 <alfagamma7> Had a busy day 14:51:34 <alfagamma7> Well changing ISP is a hassle that I don't like but nvm 14:56:37 *** urdh has quit IRC 14:56:48 *** urdh has joined #openttd 15:07:02 *** Kitrana has joined #openttd 15:12:17 *** Kitrana2 has quit IRC 15:12:51 <peter1139> 29Β°C :O 15:35:39 <andythenorth> nice eh 15:54:19 <peter1139> Delicious. 16:00:31 *** HerzogDeXtEr has joined #openttd 16:14:22 <andythenorth> had a coke 16:25:21 <peter1139> Is it? 16:28:48 <andythenorth> it was, I finished it 16:37:40 *** Smedles has joined #openttd 16:52:06 <alfagamma7> Diet coke? 16:52:43 <andythenorth> oof no 16:52:54 *** Wolf01 has joined #openttd 17:25:36 <peter1139> Zero? 17:51:55 <Eddi|zuHause> doesn't that cause cancer? 17:52:33 <kamnet> If you drink 100 cans of it a day, maybe. 17:54:06 <kamnet> Or, if you're a lab rat. But lots of human food you give to lab rats gives them cancer. That's why they're good for testing stuff, scientists are able to accurately predict stuff with them. 17:56:54 <Eddi|zuHause> there probably are people that drink that much :p 17:59:05 *** DorpsGek_vi[1] has joined #openttd 17:59:59 *** kamnet[d] has joined #openttd 17:59:59 <kamnet[d]> That's at least 1200 oz of fluid a day. Most people are absolutely sick of drinking any liquid when they hit about 200 oz in a day. Most people only consume 32-64 oz a day of any liquid, including liquid in your solid foods. 18:00:40 <peter1139> Spoken like someone who isn't addicted to Coke Zero / Pepsi Max... 18:01:40 <kamnet[d]> Oh I definitely was for a time. I could polish off 4 liters of soda in a day at one point in my life. 18:03:27 <kamnet[d]> 200 oz would be around 6 liters. 1200 oz is seventeen 2-liter bottles. 18:06:01 *** ketsuban[d] has quit IRC 18:06:01 *** _glx_ has quit IRC 18:06:01 *** michi_cc[d] has quit IRC 18:06:01 *** _zephyris has quit IRC 18:06:03 *** frosch123 has quit IRC 18:06:04 *** emperorjake has quit IRC 18:06:04 *** johnfranklin has quit IRC 18:06:04 *** talltyler has quit IRC 18:06:05 *** ahyangyi has quit IRC 18:06:05 *** andythenorth has quit IRC 18:06:05 *** truebrain has quit IRC 18:06:05 *** DorpsGek_vi has quit IRC 18:06:13 <peter1139> Uh oh 18:07:30 <kamnet[d]> Hm, just to check my math on that, human body can retain between 8 and 15 lbs of excess fluid before you start hitting serious health complications and organ shutdowns, which converts to around 120-240 oz of fluid. Your kidneys would be in overdrive at that level and probably at risk of entering renal failure. 18:09:37 <kamnet[d]> So basically if you tried taking on more than seven 2-liters of soda in a day, your organs would fail before the cancer even gets a chance to set in. 18:13:09 <Eddi|zuHause> so 2 liters is 6 cans, 6*7 is 42? 18:15:36 <kamnet[d]> Seems about right. 18:20:47 *** andythenorth has joined #openttd 18:20:47 <andythenorth> I knew someone who had kidney issues due to mainlining Coke all day 18:20:55 <andythenorth> probably around 2 litres 18:20:58 <andythenorth> per day 18:21:30 *** alfagamma7 has joined #openttd 18:21:30 <alfagamma7> Oof 18:38:15 <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/9d2920e9c5bcaaef8a61b184ef5b959f9c9aaa33 18:38:16 <DorpsGek> - Update: Translations from eints (by translators) 19:13:08 <peter1139> Well 19:14:35 *** talltyler has joined #openttd 19:14:35 <talltyler> Is it? 19:15:28 <andythenorth> why don't we have GSCheat? 19:15:43 <andythenorth> and specifically GSCheat.SetDate() ? 19:15:55 <andythenorth> I could solve daylength in one simple move π 19:20:41 <talltyler> Or just wait until Christmas when I do it properly π 19:21:23 <talltyler> My mouth might be writing checks that my ass canβt cash, but eh 19:22:41 <talltyler> The goal is clearly in sight, thereβs just a few more steps to get there π 19:26:38 <peter1139> Oh, cheques. 19:27:21 <andythenorth> like deques 19:27:44 <peter1139> std::cheque 19:30:34 <peter1139> I have had cheese. No wonder I'm putting on weight too quickly :/ 19:31:52 <talltyler> Huh, havenβt seen that British spelling before 19:34:44 *** ketsuban has joined #openttd 19:34:44 <ketsuban> "Deque" is **d**ouble-**e**nded **que**ue. 19:35:23 *** ketsuban is now known as Guest1755 19:35:24 *** Guest1755 is now known as ketsuban[d] 19:36:34 <ketsuban[d]> The homophony with "deck" is mainly just to be cute. It's not how we spell "deck". 19:40:58 <peter1139> ... What are you talking about? 19:41:32 <peter1139> Nobody pronounces deque as 'deck' 19:42:17 <ketsuban[d]> How else would you pronounce it? 19:42:44 <peter1139> d'quezz 19:44:28 <andythenorth> can we discuss how pronounce mySQL? 19:44:33 <andythenorth> or is that over? 19:45:08 <peter1139> It's pronounced "owned by Oracle" 19:50:07 *** truebrain has joined #openttd 19:50:07 <truebrain> Owh, not "garbage"? Must have been misprouncing it ... 19:50:18 <andythenorth> lol we made a Flash game for Oracle once 19:50:26 <DorpsGek> [OpenTTD/OpenTTD] PikkaBird opened issue #11261: [Bug]: Airport menu selectability after closing window on a class with no available airports https://github.com/OpenTTD/OpenTTD/issues/11261 19:50:29 <andythenorth> we had an existing tech stack for high scores 19:50:48 <andythenorth> which used a not-Oracle DB 19:51:07 <andythenorth> the contract required us to rewrite the high score table to use Oracle 19:51:11 <andythenorth> we got paid extra 19:51:55 <andythenorth> cool story eh? π 19:58:25 <peter1139> "GPL Roadtype Collection, Pt. 1/2/3/4/4/5/6". What? 19:59:05 <peter1139> Oh it's that guy. 20:00:55 *** brickblock19280 has joined #openttd 20:00:55 <brickblock19280> yeah 20:02:52 <andythenorth> how is reddit today though? 20:10:33 *** wallaby2 has joined #openttd 20:12:52 <andythenorth> such things https://www.reddit.com/r/openttd/comments/169ok6d/what_is_the_purpose_of_multiplayer/ 20:13:02 *** asymptotically2 has quit IRC 20:13:02 *** jlx__ has quit IRC 20:13:02 *** pickpacket has quit IRC 20:13:02 *** Hobbyboy|BNC has joined #openttd 20:13:02 *** wallabra has quit IRC 20:13:04 *** Hobbyboy has quit IRC 20:13:05 *** pickpacket has joined #openttd 20:13:07 *** asymptotically2 has joined #openttd 20:13:23 *** jlx__ has joined #openttd 20:56:48 *** keikoz has quit IRC 21:44:25 *** Wolf01 has quit IRC 21:59:19 <DorpsGek> [OpenTTD/OpenTTD] Kuhnovic opened pull request #11262: Change: Allow vehicles in the build-vehicle-window to be filtered based on their cargo type https://github.com/OpenTTD/OpenTTD/pull/11262 22:06:58 *** _glx_ has joined #openttd 22:06:58 <_glx_> isn't there already a filter for that ? 22:25:29 <talltyler> Yes, thereβs a drop-down for cargos. I intentionally did not look at cargo types in the text filter because of that. π 22:26:03 <talltyler> I will write a comment accordingly in the PR later 22:26:48 <andythenorth> if we add enough text searches, the results will become unusable : 22:26:58 <andythenorth> "probably fine" 22:28:31 *** HerzogDeXtEr has quit IRC 22:35:44 *** eirc has quit IRC 22:36:01 *** eirc has joined #openttd 23:33:03 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #11262: Change: Allow vehicles in the build-vehicle-window to be filtered based on their cargo type https://github.com/OpenTTD/OpenTTD/pull/11262#issuecomment-1705766368