IRC logs for #openttd on OFTC at 2023-05-27
            
00:04:22 <zirkon> I have an issue with music, I can't manage to get pipewire to set up the midi port to fluidsynth. Version is 13.1
00:04:31 <zirkon> Any help is appreciated
01:22:14 *** Wormnest_ has joined #openttd
01:22:14 *** Wormnest has quit IRC (Read error: Connection reset by peer)
02:06:24 *** Wormnest_ has quit IRC (Quit: Leaving)
02:42:43 *** D-HUND has joined #openttd
02:46:03 *** debdog has quit IRC (Ping timeout: 480 seconds)
03:10:56 <DorpsGek> [OpenTTD/OpenTTD] fmang commented on pull request #10849: Add: Ctrl+click Rename on VehicleView to rename the group https://github.com/OpenTTD/OpenTTD/pull/10849#issuecomment-1565179561
03:44:24 *** keikoz has joined #openttd
04:32:18 *** Flygon has joined #openttd
04:41:55 *** axet has joined #openttd
04:51:24 <pickpacket> glx[d]: what was added to nml?
05:13:43 *** keikoz has quit IRC ()
05:14:34 *** D-HUND is now known as debdog
05:31:01 *** zirkon has quit IRC (Ping timeout: 480 seconds)
05:41:40 *** keikoz has joined #openttd
06:16:37 *** axet has quit IRC (Remote host closed the connection)
06:32:53 <DorpsGek> [OpenTTD/OpenTTD] askondro commented on issue #10882: [Bug]: Cannot destroy train building station https://github.com/OpenTTD/OpenTTD/issues/10882
06:33:08 <DorpsGek> [OpenTTD/OpenTTD] askondro closed issue #10882: [Bug]: Cannot destroy train building station https://github.com/OpenTTD/OpenTTD/issues/10882
06:57:17 <DorpsGek> [OpenTTD/bananas-api] ahyangyi opened pull request #373: Fix: bring the image size limit of PIL in line https://github.com/OpenTTD/bananas-api/pull/373
07:23:25 *** nielsm has joined #openttd
07:38:43 <DorpsGek> [OpenTTD/OpenTTD] btzy commented on pull request #10849: Add: Ctrl+click Rename on VehicleView to rename the group https://github.com/OpenTTD/OpenTTD/pull/10849#issuecomment-1565253671
07:39:33 *** _aD has joined #openttd
08:05:58 <andythenorth> is cat?
08:09:16 <DorpsGek> [OpenTTD/bananas-api] TrueBrain commented on pull request #373: Fix: bring the image size limit of PIL in line https://github.com/OpenTTD/bananas-api/pull/373#pullrequestreview-1447214654
08:18:06 *** GeorgeVB has joined #openttd
08:18:06 <GeorgeVB> Any ideas on https://github.com/OpenTTD/OpenTTD/issues/10763 ?
08:18:06 <GeorgeVB> May be I should provide more informaton (wich one?) to get this fixed?
08:21:09 <andythenorth> Horse 97%
08:27:52 <andythenorth> GeorgeVB: the zip attached to 10763 doesn't use variant groups?
08:28:00 <andythenorth> 97
08:28:20 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1111933900705497088/image.png
08:28:51 <andythenorth> nvm, needed to rescan grfs
08:29:30 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1111934193627316304/image.png
08:29:30 <andythenorth> ok so the parent vehicle isn't buildable?
08:29:32 <andythenorth> how is that done?
08:31:28 <DorpsGek> [OpenTTD/bananas-api] ahyangyi updated pull request #373: Fix: bring the image size limit of PIL in line https://github.com/OpenTTD/bananas-api/pull/373
08:32:23 <TrueBrain> that exception name .. lol .. `DecompressionBombError` ... just perfect ๐Ÿ˜„
08:32:26 <DorpsGek> [OpenTTD/bananas-api] ahyangyi commented on pull request #373: Fix: bring the image size limit of PIL in line https://github.com/OpenTTD/bananas-api/pull/373#pullrequestreview-1447221845
08:33:15 <DorpsGek> [OpenTTD/bananas-api] TrueBrain approved pull request #373: Fix: bring the image size limit of PIL in line https://github.com/OpenTTD/bananas-api/pull/373#pullrequestreview-1447221925
08:33:35 <GeorgeVB> andythenorth: unbuildable vehicle with a separate id
08:34:24 <DorpsGek> [OpenTTD/bananas-api] TrueBrain commented on pull request #373: Fix: bring the image size limit of PIL in line https://github.com/OpenTTD/bananas-api/pull/373#pullrequestreview-1447222541
08:34:33 <andythenorth> unbuildable = unavailable for climate?
08:34:36 <TrueBrain> grrr, I should run the CI before I approve stuff ๐Ÿ˜›
08:35:24 <andythenorth> ok yeah Horse has a specific variants problem also
08:35:32 <andythenorth> didn't notice until I tried xUSSR
08:36:41 *** Ahyangyi has joined #openttd
08:36:41 <Ahyangyi> TrueBrain: Perhaps I should squash my commits? Since the commit message checker checks each commit message...
08:36:50 <TrueBrain> you can; I can do that on merge too
08:37:10 <TrueBrain> honestly, it is becoming more and more annoying that we are this strict during PRs about commit messages .. I wonder if there isn't a better way .. but okay
08:37:21 <TrueBrain> OpenTTD's policy says you should squash and force-push, yes
08:37:38 <GeorgeVB> andythenorth: Yes, for all of them
08:37:41 <Ahyangyi> Oh, `bananas_api/new_upload/readers/heightmap.py:9:1: E402 module level import not at top of file`
08:37:42 <Ahyangyi> I see
08:38:27 <DorpsGek> [OpenTTD/bananas-api] ahyangyi dismissed a review for pull request #373: Fix: bring the image size limit of PIL in line https://github.com/OpenTTD/bananas-api/pull/373#pullrequestreview-1447221925
08:38:30 <DorpsGek> [OpenTTD/bananas-api] ahyangyi updated pull request #373: Fix: bring the image size limit of PIL in line https://github.com/OpenTTD/bananas-api/pull/373
08:38:57 <andythenorth> GeorgeVB: my guess is that autoreplace menu handles 'unavailable' differently to purchase menu
08:40:51 <GeorgeVB> It's a bug, difenitly
08:40:53 <Ahyangyi> "Dismissing reviews" seems to be the side effect of force pushing ๐Ÿ˜›
08:41:08 <DorpsGek> [OpenTTD/bananas-api] TrueBrain approved pull request #373: Fix: bring the image size limit of PIL in line https://github.com/OpenTTD/bananas-api/pull/373#pullrequestreview-1447234447
08:41:09 <TrueBrain> no, of any change ๐Ÿ™‚
08:41:17 <TrueBrain> we like an approval on the actual PR, not on some former version of it ๐Ÿ˜›
08:42:36 <Ahyangyi> Indeed; just get flake8 running locally as well, it should make further PRs* smoother
08:42:40 <Ahyangyi> *: if any
08:43:04 <TrueBrain> would be nice if someone (hint hint) could add the ability to update heightmaps, instead of creating a new one ๐Ÿ˜›
08:53:48 <jfs-> so re. the meetup, is it finally confirmed to happen on the 10th in Brussels?
09:13:44 *** _aD has quit IRC (Quit: leaving)
09:15:03 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1111945655116644382/image.png
09:15:03 <andythenorth> hmm it's almost like we might want to expose variant level context to purchase menu sprite chain
09:15:29 <andythenorth> it makes sense to have compound groups with engines and coaches
09:15:54 <andythenorth> but when the group is closed, it's harder to spot the engine
09:16:00 <andythenorth> coaches just look like coaches
09:16:12 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1111945945450565742/image.png
09:16:19 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1111945975955734528/image.png
09:16:30 * andythenorth bbl
09:38:12 <TrueBrain> pff, I think I finally see the light at the end of the tunnel for the infra changes .. this was far more complex than I was hoping for. But, also a lot simpler, meaning less weird GitHub Actions scripts ๐Ÿ˜„
09:45:16 <LordAro> woo
10:02:55 <glx[d]> pickpacket: Switch optimisations, a long time ago
10:10:41 *** Wolf01 has joined #openttd
10:56:24 <TallTyler> jfs-: Yes. I need to post something official and make an event here on Discord too. No use restricting it to only people who read GitHub discussionsโ€ฆ
11:12:48 *** michi_cc[d] has joined #openttd
11:12:48 <michi_cc[d]> So, this poses the question on who would be in Brussels in the evening, too. I canโ€™t stay all Sunday, so I either have to whatever the latest flight on Saturday is back or stay the night and then return in the morning. Obviously the cost of a hotel night only make sense if enough people would be around in the evening.
11:35:31 <jfs-> I will probably take a bus arriving at 11:30 and then fly home at night
11:35:44 <jfs-> I think 21:30 flight or there about
11:46:07 <DorpsGek> [OpenTTD/OpenTTD] Transportation-master-kcl commented on issue #10870: [Crash]: Game crashed when CivilAI just started construction https://github.com/OpenTTD/OpenTTD/issues/10870
12:20:33 <TallTyler> I could also stay the night, but will more likely take Thalys back to the Cologne area after dinner
12:23:25 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1111993058960093245/image.png
12:23:25 <andythenorth> petern: shades of Doom 1?
12:34:55 <petern> Shades of lunch?
12:37:22 <andythenorth> I had a cripss
12:37:31 <andythenorth> and a coffee
12:42:02 <andythenorth> rust grf compiler when?
12:42:09 <andythenorth> (but it ingests nml) ๐Ÿ˜›
12:44:35 <andythenorth> ugh, maybe I can just improve the Horse compile
12:44:45 <andythenorth> it's spending 5s just rendering sprites
12:44:48 <andythenorth> that's horrific
12:45:06 <andythenorth> 6.5s templating nml is even worse
12:48:32 <andythenorth> I can't trivially multi-process the nml because there'll be contention writing to a single nml file
12:48:42 <andythenorth> could fix that, but it's probably not worth the overhead
12:54:24 <andythenorth> well it's not IO
12:54:36 <andythenorth> looks like the first time a template is used there is a huge overhead compiling it to bytecode
12:54:42 <andythenorth> as much as 0.1s
12:54:58 <TrueBrain> TallTyler: I will be driving home after dinner too; and I guess that means 2 other dutchies will too (given I am driving them :D)
12:58:14 <TrueBrain> I didn't even ask for their opinion ๐Ÿ˜ฎ that is evil .. I should first ๐Ÿ™‚
12:58:21 <TrueBrain> so let me rephrase: I plan to drive home after dinner too ๐Ÿ˜›
13:13:49 <Eddi|zuHause> so there's a direct flixbus trip, 9:30 each direction over night...
13:14:28 <Eddi|zuHause> i'd be arriving 8:55 and leaving 20:55
13:16:11 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on discussion #10002: Meetup 2022 / 2023 / 10k event https://github.com/OpenTTD/OpenTTD/discussions/10002
13:17:40 <Eddi|zuHause> i'm unsure whether i want to commit to that
13:22:15 <Eddi|zuHause> alternatively, someone could pick me up at a place somewhere near the german border
13:27:15 <Eddi|zuHause> all train journeys in that direction are completely weird
13:29:04 <Eddi|zuHause> and triple the price
13:45:13 <petern> andythenorth: on OSX when using Truetype fonts, does the `font` console command show the font style or just the font name?
13:45:29 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1112013710978994196/image.png
13:45:32 <andythenorth> nah, wait
13:45:41 <andythenorth> truetype eh?
13:46:19 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1112013920257970287/image.png
13:46:53 <petern> That seems to be included. Is "Lato Bold" an option you could test?
13:46:53 <andythenorth> lol I never tried non-pixel font
13:47:11 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1112014140672847922/image.png
13:47:18 <petern> Okay, thanks ๐Ÿ˜„
13:47:42 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10879: Change: Include style when getting font name. https://github.com/OpenTTD/OpenTTD/pull/10879#issuecomment-1565426275
14:04:16 <michi_cc[d]> TrueBrain: Okay, looks like staying doesnโ€™t make much sense. Last flight for me would be 20:50 (thanks to the BER night curfew). No idea how busy/how much time you need to have for Brussels airport, but that would be 18:30 to 19:00 Iโ€™d guess.
14:11:42 <TrueBrain> so early dinner, you say?
14:15:31 <michi_cc[d]> Somewhat. Iโ€™m usually cutting it close on flights, but I have never flown from Brussels, so I donโ€™t know if one can risk just one hour beforehand.
14:15:36 <Eddi|zuHause> flying seems to make even less sense than train
14:17:07 <TrueBrain> michi_cc[d]: never flows from Brussels myself too, so can't help you there. Seen many airports, but that is not one of them ๐Ÿ˜„
14:18:43 <TrueBrain> but given you also have to travel to the airport, might be a bit tricky .. maybe better for you to dinner at the airport; just a bit sad that you can't join us in that case ...
14:19:25 <TrueBrain> either way, I wouldn't have any issues with an early dinner, where people who have to go, go ๐Ÿ™‚
14:19:37 <TrueBrain> no clue what TallTyler has in mind here ๐Ÿ™‚
14:26:53 <Eddi|zuHause> is anyone travelling by car from anywhere within germany?
14:28:40 <michi_cc[d]> TrueBrain: City train station to airport is like 20 minutes according to Google.
14:29:16 <TrueBrain> dinner at 1600, you leave at 1800, should be fine? ๐Ÿ˜„
14:29:22 <michi_cc[d]> And deal is done. Not really a cheap day in Brussels, but YOLO and stuff ๐Ÿ˜Ž
14:29:50 <TrueBrain> these meetups are never cheap ๐Ÿ˜„
14:30:06 <Eddi|zuHause> the 25k party didn't cost me anything...
14:30:12 <glx[d]> hehe
14:30:29 <TrueBrain> although this time it shouldn't cost me that much, given I have an electric car I can charge up with my solar panels ...
14:31:44 <TrueBrain> I could even drive back and forth without recharging .. but I think I will pick up some people, making that less likely ๐Ÿ˜›
14:32:13 <michi_cc[d]> I have no problem cutting it close. Iโ€™ll do some more research, but 1900 at the train station might be fine. So dinner not too far outside and it should be good.
14:32:58 <TrueBrain> well, Train World itself is next to .. a station. Shocking, I know right ๐Ÿ˜›
14:33:40 <michi_cc[d]> Just apparently not the one with the airport train.
14:33:46 <TrueBrain> that sucks
14:34:33 <Eddi|zuHause> probably easier to get someone to drive you to the airport
14:41:53 <andythenorth> Horse 99%
14:44:02 <Eddi|zuHause> does the world end if it reaches 100%?
14:45:18 <andythenorth> didn't the last 50 times or so
14:56:10 <Wolf01> Funny, this morning I saw a train with only the engine and the tail coach... shortest train ever
14:56:35 <Eddi|zuHause> we used to have this kind of train regularly
14:56:44 <glx[d]> common here too
14:57:05 <Eddi|zuHause> quite a rare sight now since the switch to mainly multiple units
14:57:41 <glx[d]> mostly on low capacity lines
15:08:46 <jfs-> that sounds like exactly the situation where a railcar or MU makes more sense
15:11:40 <Eddi|zuHause> it was mostly a case of "we have this ancient rolling stock anyway". it's mostly gone now since contracts require new or almost new rolling stock now
15:11:55 <glx[d]> ok it seems to be "painful" to do GHSA stuff, the workflow is not very compatible with our usual workflow
15:13:01 <glx[d]> the PR doesn't run the checks
15:15:48 <TrueBrain> So we learn ๐Ÿ™‚
15:16:15 <TrueBrain> I would say, in this case, just PR it against master like normal
15:17:32 <glx[d]> I also feel the green "merge pull request" button in the advisory report won't squash nor rebase
15:18:09 <TrueBrain> It is very .. unclear indeed
15:18:31 <TrueBrain> It tries to keep all this code very private
15:19:03 <glx[d]> which is nice, but also complicated
15:26:46 <DorpsGek> [OpenTTD/OpenTTD] glx22 opened pull request #10883: Fix: Check max member count in squirrel classes https://github.com/OpenTTD/OpenTTD/pull/10883
15:29:04 <glx[d]> and of course now the hidden PR checks are started (it's the same run because same commit)
15:30:08 <petern> Whatever it is, this bug is so severe I don't get to see it :p
15:31:46 <glx[d]> you don't have access to security tab ?
15:32:06 <TrueBrain> I don't understand anything about that code or the fix .. so yolo? ๐Ÿ˜„
15:32:17 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #10883: Fix: Check max member count in squirrel classes https://github.com/OpenTTD/OpenTTD/pull/10883#pullrequestreview-1447435530
15:32:42 <glx[d]> basically squirrel use the upper bits as flag, and overflowing into the flags is bad
15:32:57 <andythenorth> yolo security fixes ๐Ÿ˜›
15:32:58 <andythenorth> oof
15:33:17 <petern> I can see the tab, but only Code scanning or Dependabot stuff.
15:33:46 <glx[d]> ah maybe only admins can see it without adding extra people
15:34:04 <glx[d]> well and andy as he reported it ๐Ÿ™‚
15:35:15 <glx[d]> you should see it now
15:35:22 <TrueBrain> Do we need to check uploads to see if anyone misused it? I doubt it, but...
15:35:53 <glx[d]> they might have trigger memory limit first ๐Ÿ˜‰
15:36:00 <TrueBrain> And realise that this means we should push 13.2 soon ๐Ÿ™‚
15:36:32 <andythenorth> do I get a cookie for guessing that squirrel would be our most likely RCE? ๐Ÿ˜›
15:36:37 <andythenorth> I mean...it's not hard to guess
15:36:38 <TrueBrain> Yeah, last time Squirrel had something like this, our memory management made it impossible to abuse ๐Ÿ™‚
15:38:12 <glx[d]> enabled auto-squash
15:44:42 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #10884: Remove: obsolete NewGRF text unprinting. https://github.com/OpenTTD/OpenTTD/pull/10884
15:47:25 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #10884: Remove: obsolete NewGRF text unprinting. https://github.com/OpenTTD/OpenTTD/pull/10884#pullrequestreview-1447446097
16:00:38 <LordAro> that was quick
16:02:29 *** Wormnest has joined #openttd
16:03:49 <TrueBrain> We talked enough about it already ๐Ÿ˜›
16:07:29 <glx[d]> but you detected some PushWord usages
16:09:25 <DorpsGek> [OpenTTD/OpenTTD] glx22 merged pull request #10883: Fix: Check max member count in squirrel classes https://github.com/OpenTTD/OpenTTD/pull/10883
16:10:56 <TrueBrain> 3!
16:11:14 <TrueBrain> Out of the ~5000 ๐Ÿ™‚
16:11:38 <TrueBrain> Didn't check if it is an active grf or anything btw; just wanted to know if the script worked ๐Ÿ™‚
16:11:55 <glx[d]> wondering if you could extract the strings to see what they do
16:12:17 <glx[d]> because I don't really see how to use pushword
16:13:52 <TrueBrain> I will see what I can do for you
16:16:04 <petern> Remove push word too did you say? :p
16:36:11 <TrueBrain> right, so I have 4 hits on pushword .. all 4 in dutch transets
16:36:28 <TrueBrain> `544d0201:f3e710b4`, `544d0201:85886ca3`, `5f5f0400:c2b9a6f5` and `fbfb0601:e167e847`
16:37:32 <TrueBrain> all 3 GRFs are active on BaNaNaS, so it is even activelly used ๐Ÿ™‚
16:41:59 <TrueBrain> ugh, I forgot how terrible NFO was to read ๐Ÿ˜›
16:42:51 <TrueBrain> best I can do for you glx[d] is tell you it is a string like this: `\xc3\x9e\xee\x82\x98Peso por Eje: \xee\x82\x90\xee\x82\x80\r\xee\x82\x98Tiempo de carga: \xee\x82\x90\xee\x82\x9a\x03 \xee\x82\x86\xee\x82\x86\xee\x81\xbb\xee\x82\x85 d\xc3\xada\xee\x82\x9a\x15\x82\xee\x82\x9a\x10\x01\xee\x82\x9a\x11s\xee\x82\x9a\x12\r\xee\x82\x98\xee\x82\x80`
16:43:06 <TrueBrain> I am sure it makes sense to someone ๐Ÿ™‚
16:43:58 <glx[d]> well you can blame utf8 on this one ๐Ÿ™‚
16:44:46 <TrueBrain> weirdly enough, it only seems to be in translated strings ๐Ÿ™‚
16:45:11 <TrueBrain> `๎‚˜๎‚€emps de chargement : ๎‚๎‚š ๎‚†๎‚†๎ป๎‚… jour๎‚š๏ฟฝ๎‚š๎‚šs๎‚š` if you want to see it decoded ๐Ÿ˜›
16:45:42 <TrueBrain> all 3 GRFs are a copy of these strings
16:46:15 <TrueBrain> so that is the story of ext-ctrl-codes 0x03 ๐Ÿ™‚
16:47:19 <TrueBrain> the English string it seems to translate is `Loading Time: `
16:47:39 <glx[d]> yes translation seems fine
16:48:02 <TrueBrain> but where that is followed by 09 9A 03, these strings are followed by `"รฎ" 82 90 EE 82 9A 03`
16:48:05 <TrueBrain> I have no clue what this all means ๐Ÿ˜›
16:48:46 <TrueBrain> the german translation is fine
16:49:09 <TrueBrain> I have absolutely no clue what happened here, but it seems this is not intended to actually use pushword ๐Ÿ™‚
16:50:03 <TrueBrain> and make no mistake, this could be an error in our decoding ๐Ÿ™‚
16:50:43 <TrueBrain> `EE 82 98 "Temps de chargementร‚ : รฎ" 82 90 EE 82 9A 03 20 20 EE 82 86 EE 82 86 EE 81 BB EE 82 85 " jourรฎ" 82 9A 15 82 EE 82 9A 10 01 EE 82 9A 11 73 EE 82 9A 12`
16:50:45 <TrueBrain> is the NFO string
16:51:03 <TrueBrain> `98 "Loading " "Time: " 90 9A 03 20 20 86 86 7B 85 " day" 9A 15 82 9A 10 01 9A 11 73 9A 12` is the english variant
16:51:32 <TrueBrain> so yeah, encoding issue, it seems ๐Ÿ™‚
16:51:34 <TrueBrain> what is EE82?
16:52:00 <TrueBrain> Private Use Character
16:52:01 <TrueBrain> lol
16:53:04 <petern> We use E000 to E2FF but not EExx.
16:53:15 <TrueBrain> no, it seems someone saved these strings weirdly ๐Ÿ™‚
16:53:28 <TrueBrain> and the UTF-8 encoding took our lovely codepoints and made them in shit ๐Ÿ˜„
16:53:33 <TrueBrain> 98 -> EE 82 98
16:53:36 <TrueBrain> etc ๐Ÿ˜›
16:54:13 <TrueBrain> and as the strings are not marked as unicode, they are decoded as ascii
16:54:17 <TrueBrain> that is why the detection triggered ๐Ÿ™‚
16:54:34 <TrueBrain> so I would guess the translations also don't actually work ... ๐Ÿ˜›
16:54:44 <TrueBrain> so yeah, remove push-word? ๐Ÿ˜„ At least, there are no valid uses on BaNaNaS, it seems ๐Ÿ™‚
16:55:15 <glx[d]> oh that's why online utf8 decoder didn't like \xee
16:56:02 <TrueBrain> it is also UTF-16 .. so it is just weird
16:56:02 <glx[d]> BTW 98 is {BLACK} in newgrf world
16:56:20 <TrueBrain> but even grfcodec doesn't really know what to do with it
16:56:49 <glx[d]> grfcodec has an option to not try to print ascii
16:57:00 <TrueBrain> the whole French translation is like this btw
16:57:34 <TrueBrain> completely FUBAR (nice new Netflix serie btw :P)
16:57:46 <glx[d]> yeah I intend to watch it
16:57:53 <glx[d]> I like the pitch
16:57:56 <TrueBrain> it is surprisingly better than I expected
16:58:36 <TrueBrain> yeah, the whole translation is littered in EE 82 ๐Ÿ™‚
16:58:55 <TrueBrain> over a 1000 hits .. lol
16:59:05 <TrueBrain> 2660 hits, to be more exact
16:59:05 <glx[d]> hmm do they have a public repo ?
16:59:21 <TrueBrain> seems to hit 4 languages ๐Ÿ™‚
16:59:48 <glx[d]> let's see ingame, just for the fun
17:00:12 <glx[d]> dutch trainset you said ?
17:00:33 <TrueBrain> lol, made by FooBar, that is funny, sorry ๐Ÿ™‚
17:00:45 <TrueBrain> Dutch Trainset 2, is an example, yes
17:00:57 <TrueBrain> or Dutch Trainset 3XL
17:01:34 <TrueBrain> it is hosted on devzone .. so yeah ....
17:01:52 <glx[d]> so it most likely used eints
17:02:01 <TrueBrain> maybe frosch has the source ๐Ÿ˜›
17:02:07 <glx[d]> could be an nml bug too
17:02:32 <TrueBrain> it appears to be an UTF-16 encoding; I don't see a world that can be blamed to either tool honestly
17:03:13 <glx[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1112063475469008987/image.png
17:03:26 <glx[d]> for some reason it seems to be fine
17:04:19 <TrueBrain> lovely ๐Ÿ™‚
17:04:47 <glx[d]> let's try in debug with a breakpoint on 9A 03
17:04:50 <TrueBrain> on one hand could be how the Python decodes the strings .. on the other hand, could be luck ๐Ÿ˜„ I don't know enough about this stuff ๐Ÿ˜›
17:04:50 <frosch> i have some dutch trainsets, not sure which version
17:05:40 <petern> Weird that the French translation has a space before the :
17:06:02 <TrueBrain> the French are weird ๐Ÿ˜›
17:06:25 <glx[d]> because typo rules
17:06:50 <glx[d]> space before and after stuff with 2 elements
17:07:11 <petern> > In French, exclamation points, question marks, colons, and semi-colons (all forms of "high" punctuation(opens in a new tab)) should always have a space preceding them.
17:07:38 <frosch> TrueBrain: sorry, you have to give me a grfid, i have no idea which of the 5 dutch trainsets you mean
17:07:41 <glx[d]> I use {NBSP} in ths case ๐Ÿ™‚
17:07:55 <glx[d]> else it can have weird effecrs
17:07:57 <TrueBrain> TrueBrain: you used to read up frosch .. getting lazy? ๐Ÿ˜›
17:08:07 <TrueBrain> ๐Ÿ˜„ ๐Ÿ˜„ ๐Ÿ˜„
17:08:42 <petern> Hmm, station classes on a dropdown is a bit awkward ๐Ÿ˜ฆ
17:08:49 <TrueBrain> basically we wonder who introduced those EE 82 .. was that already from the source?
17:08:53 <TrueBrain> does NML? Does eints?
17:09:04 <glx[d]> in theory we also need to use ยซ and ยป
17:09:07 <petern> As it's two clicks to select instead of one. But it does take up less room.
17:09:11 <glx[d]> but that's annoying
17:10:05 <TrueBrain> I am kinda surprised it renders fine in-game honestly ๐Ÿ™‚
17:10:29 <glx[d]> that's why I'm starting the debugger
17:10:46 <TrueBrain> which one did you end up trying?
17:10:46 <glx[d]> to make sure it's not a false positive on 9A 03
17:11:17 <frosch> glx[d]: TrueBrain https://github.com/Transportman/dutchtrains/tree/main/lang
17:11:26 <frosch> the newest one of them
17:11:48 <glx[d]> breakpoint reached
17:11:55 <glx[d]> it's really using 9A 03
17:12:05 <TrueBrain> so it renders it properly purely by luck ๐Ÿ™‚
17:13:57 <TrueBrain> frosch: nothing weird in those source files at least ๐Ÿ™‚
17:14:14 <frosch> nml's grfstrings.py inserts push_word for alignment
17:14:46 <frosch> when translations reorder parameters, nml has to do some weird rotate-parameter struff
17:15:33 <TrueBrain> but why all those EE 82? ๐Ÿ™‚
17:15:40 <glx[d]> https://github.com/Transportman/dutchtrains/blob/main/lang/french.lng#L519-L523
17:15:40 <frosch> https://github.com/OpenTTD/nml/blob/master/nml/grfstrings.py#L471
17:16:05 <glx[d]> so it's not really using 9A 03 based on the strings
17:17:35 <glx[d]> oh nasty nml stuff
17:18:14 <TrueBrain> at least that is what is inserted here, so NML is to blame for it ๐Ÿ™‚
17:18:45 <TrueBrain> and 2x 0x86 too
17:18:52 <glx[d]> grf limitations most likely, forcing nml to do nasty stuff
17:18:54 <TrueBrain> still no clue what those EE 82 are doing in front of all those entries ๐Ÿ˜„
17:19:21 <TrueBrain> well, once again, I am convinced that NewGRFs are pure evil and just exists to mess with us ๐Ÿ˜„
17:19:57 <TrueBrain> anyway, the detection script frosch wrote works as advertised ๐Ÿ™‚
17:20:33 <glx[d]> openttd just puts the index before/after the control code, as it should be in normal world
17:21:06 <glx[d]> but as newgrf is abusing registers as an array of variable size values
17:24:43 *** Kitrana1 has quit IRC (Quit: Leaving.)
17:25:06 *** Kitrana has joined #openttd
17:25:09 <glx[d]> "BLACK": {"unicode": r"\UE098", "ascii": r"\98"}, <-- that's for 98
17:25:15 <frosch> TrueBrain: ee 82 is the unicode user range
17:25:32 <glx[d]> "PUSH_WORD": {"unicode": r"\UE09A\03", "ascii": r"\9A\03", "deprecated": True},
17:25:47 <frosch> U+E0xx is a user-defineable range in unicode, and that's where the string code live
17:26:17 <frosch> the push_word stuff is needed by nml to do things like "{1:NUM}{0:NUM}", i.e. reorder parameters
17:26:47 <frosch> newgrf have only one command for reordering parameters, and that is a rotation of 8 bytes (fixed value)
17:26:59 <glx[d]> yeah we need to upgrade the spec to be simpler for users
17:27:17 <glx[d]> a real SetDValue()
17:27:37 <glx[d]> and proper index marking in the strings
17:27:40 <frosch> that means (1) you cannot rotate more, so there is a limit on how far parameters can be moved, and (2) if you want to remove less, you have to PUSH_WORD junk on the stack, rotate, use want you want, and drop the junk again
17:29:17 <frosch> a SetDValue does not help here to remove the push_word. the push_word is used for reordering, so you need different NUM/FORCE/STRING codes, which accept offsets
17:29:53 <frosch> but at that point, you replace basically everything :p
17:30:15 <glx[d]> with SetDValue and proper index in the string you don't need to reorder ๐Ÿ™‚
17:30:34 <glx[d]> it's more like openttd does it
17:30:58 <frosch> yes, the index stuff ๐Ÿ™‚
17:32:03 <frosch> you could probably add a new string prefix (like the unicode prefix), and then just write the lang-file string with all the {} into the newgrf
17:32:13 <frosch> so the string is compiled inside ottd, like with gs
17:34:31 <glx[d]> authors would like that, more commands available for strings in openttd
17:36:37 <frosch> the "text stack" at register 100+x is mostly already a setdparam, if you declare all new string-codes to use 32bit
17:37:07 <frosch> but the excel people usually want more than 15 string parameters, and register 10F is already used for other struff
17:37:26 <glx[d]> might be simpler to add a \2 operator for that
17:37:41 <frosch> or use register 200+x ๐Ÿ™‚
17:37:57 <frosch> it's write-only in both cases
17:38:18 <frosch> and newgrf only can do 32bit, so no 64bit currencies and refitmasks
17:38:24 <petern> Does using tooltips to show details about the thing being hovered over break our "UI guidelines"?
17:39:16 <frosch> i used that in the depot window, to show loaded cargo and capacity
17:39:26 <frosch> even with ctrl :p
17:39:50 <petern> Ah ha, precedent!
17:44:22 <andythenorth> oof
17:44:33 <andythenorth> what shall I set the search strings to for Horse vehicles?
17:44:41 <andythenorth> name handling is all via callback
17:44:49 <andythenorth> but I need to set the action 0 prop
17:45:00 <andythenorth> shall I use keywords for SEO optimisation ๐Ÿ˜›
17:45:03 <petern> For searching?
17:45:14 <petern> Is it a bug that searching doesn't use the callback?
17:45:19 <andythenorth> dunno
17:45:29 <andythenorth> possibly it should be a specific callback context?
17:45:51 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1112074203475230792/image.png
17:47:35 <glx[d]> Nice
17:51:51 <Eddi|zuHause> i'm still contemplating the bus option... but i'll be dead...
18:40:46 <DorpsGek> [OpenTTD/OpenTTD] DorpsGek pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/90fdf17e3944b8334f0864eec0742b7ec1d62807
18:40:47 <DorpsGek> - Update: Translations from eints (by translators)
18:56:24 <petern> Hmm
19:08:40 <Eddi|zuHause> say i spend the night in brussels at the cheapest place i can find... what to do the next day?
19:09:53 <andythenorth> eat waffles
19:11:22 <jfs-> spend the entire day getting home via 6 different trains
19:11:44 <Eddi|zuHause> probably more like 8 :)
19:12:15 <Eddi|zuHause> or pay a fortune
19:14:20 <Eddi|zuHause> if i'm willing to spend 100โ‚ฌ+ i can probably do it in 2
19:15:06 <Eddi|zuHause> ignoring some local bits at the beginning and end
19:15:15 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1112096699163345027/image.png
19:15:15 <petern> Maybe that works
19:15:45 <petern> Maybe it's not important information anyway.
19:16:08 <Eddi|zuHause> how does that look in smaller gui zoom?
19:19:07 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1112097674695553154/image.png
19:19:07 <petern> Pretty much the same?
19:37:40 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10879: Change: Include style when getting font name. https://github.com/OpenTTD/OpenTTD/pull/10879#issuecomment-1565659234
19:39:56 <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #10879: Change: Include style when getting font name. https://github.com/OpenTTD/OpenTTD/pull/10879#pullrequestreview-1447553151
19:40:29 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #10879: Change: Include style when getting font name. https://github.com/OpenTTD/OpenTTD/pull/10879
19:46:10 <TallTyler> TrueBrain: I have nothing in mind. An early dinner sounds great to me. Iโ€™m not officially planning dinner because I donโ€™t know how to internationally (canโ€™t exactly call a restaurant to make a reservation) and because I donโ€™t want to be responsible for choosing a venue.
19:46:44 <TrueBrain> Well, I am sure we can manage ๐Ÿ˜„
19:48:32 <TrueBrain> Trying to figure out if and where I can charge my car there is an interesting problem ๐Ÿ˜„
19:48:52 <TrueBrain> We have much more infra for that over here ๐Ÿ™‚
19:52:19 <Eddi|zuHause> would surprise me if they don't have that
19:53:33 <Eddi|zuHause> different question... why does cities:skylines crash during startup? (probably a mod)
19:53:53 <petern> You probably don't have enough memory.
19:54:17 <Eddi|zuHause> that may also be true... but it used to work(-ish)
19:54:59 <Eddi|zuHause> and that would probably show during game load, not on launch
20:30:28 *** Flygon has quit IRC (Remote host closed the connection)
20:37:28 *** zirkon has joined #openttd
20:45:21 <Eddi|zuHause> recently i have a lot of totally weird errors where things don't work which worked before
20:45:37 <Eddi|zuHause> like this one game in dosbox, where i can load savegames, but not start new games
20:52:25 <glx[d]> memtest ?
21:02:27 *** nielsm has quit IRC (Ping timeout: 480 seconds)
21:43:08 <DorpsGek> [OpenTTD/OpenTTD] anatolyeltsov updated pull request #10541: Feature: Industry production graph https://github.com/OpenTTD/OpenTTD/pull/10541
21:44:54 <DorpsGek> [OpenTTD/OpenTTD] anatolyeltsov commented on pull request #10541: Feature: Industry production graph https://github.com/OpenTTD/OpenTTD/pull/10541#issuecomment-1565697357
22:31:47 *** keikoz has quit IRC (Ping timeout: 480 seconds)
22:41:09 <DorpsGek> [OpenTTD/team] lastata opened issue #421: [ru_RU] Translator access request https://github.com/OpenTTD/team/issues/421
22:44:30 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
23:52:22 *** HerzogDeXtEr has quit IRC (Read error: Connection reset by peer)
23:59:13 <DorpsGek> [OpenTTD/team] glx22 commented on issue #421: [ru_RU] Translator access request https://github.com/OpenTTD/team/issues/421