Log for #openttd on 30th April 2025:
Times are UTC Toggle Colours

00:18:11  *** WormnestAndroid has quit IRC
00:18:20  *** WormnestAndroid has joined #openttd
01:43:32  *** WormnestAndroid has quit IRC
01:43:50  *** WormnestAndroid has joined #openttd
01:59:50  *** Wormnest has joined #openttd
02:28:02  *** gnu_jj_ has joined #openttd
02:31:20  *** gnu_jj has quit IRC
02:42:02  *** Wormnest has quit IRC
03:55:48  *** WormnestAndroid has quit IRC
03:55:53  *** WormnestAndroid has joined #openttd
04:08:08  *** keikoz has joined #openttd
04:46:57  <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/7bec2029fa6fc75be7edbd8e8a739710dc9a5660
04:46:58  <DorpsGek>   - Update: Translations from eints (by translators)
05:21:09  *** keikoz has quit IRC
05:45:33  <DorpsGek> [OpenTTD/OpenTTD] Release workflow was not successful https://github.com/OpenTTD/OpenTTD/actions/runs/14747334975
06:16:25  *** Wolf01 has joined #openttd
06:18:08  <DorpsGek> [OpenTTD/OpenTTD-TTF] zephyris commented on pull request #8: Feature: Hebrew Alphabet for OpenTTD Small and SmallCaps https://github.com/OpenTTD/OpenTTD-TTF/pull/8#issuecomment-2840918839
06:18:57  <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #13948: Codechange: Move ownership of Orders to OrderList. https://github.com/OpenTTD/OpenTTD/pull/13948
06:25:06  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14167: Fix 261e09a549: Compilation failure with Fluidsynth enabled. https://github.com/OpenTTD/OpenTTD/pull/14167#pullrequestreview-2805876925
06:28:01  <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14167: Fix 261e09a549: Compilation failure with Fluidsynth enabled. https://github.com/OpenTTD/OpenTTD/pull/14167
06:30:29  <DorpsGek> [OpenTTD/OpenTTD-TTF] PeterN approved pull request #8: Feature: Hebrew Alphabet for OpenTTD Small and SmallCaps https://github.com/OpenTTD/OpenTTD-TTF/pull/8#pullrequestreview-2805887920
06:33:06  <DorpsGek> [OpenTTD/OpenTTD-TTF] zephyris commented on pull request #8: Feature: Hebrew Alphabet for OpenTTD Small and SmallCaps https://github.com/OpenTTD/OpenTTD-TTF/pull/8#issuecomment-2840945595
06:33:15  <DorpsGek> [OpenTTD/OpenTTD-TTF] zephyris merged pull request #8: Feature: Hebrew Alphabet for OpenTTD Small and SmallCaps https://github.com/OpenTTD/OpenTTD-TTF/pull/8
06:42:27  <DorpsGek> [OpenTTD/OpenTTD-TTF] zephyris opened issue #9: The new shekel currency symbol is too tall https://github.com/OpenTTD/OpenTTD-TTF/issues/9
06:51:55  <truebrain> Okay the nightly this time is an actual build failure 🙂
06:53:25  <DorpsGek> [OpenTTD/OpenTTD] Iris-Persephone updated pull request #14158: Add: "Many random towns" button in scenario editor now prompts for the number of towns, with defaults based on the current new game settings https://github.com/OpenTTD/OpenTTD/pull/14158
07:21:40  <DorpsGek> [OpenTTD/OpenTTD-TTF] zephyris opened pull request #10: Feature: Hebrew Alphabet for OpenTTD Serif https://github.com/OpenTTD/OpenTTD-TTF/pull/10
07:23:36  <peter1138> _zephyris, just in case you hadn't had enough of fonts... https://en.wikipedia.org/wiki/Vietnamese_alphabet
07:25:26  <_zephyris> vietnamese
07:25:26  <_zephyris> OpenTTD-Sans missing 61: (ƠơƯựẠạẢảẤấẦầẨẩẫậắằẳẵẶặẹẻẽếềểễệỉịọỏốồổỗỘộớờỞởỡợụỦủỨứỪừửữỰựỳỷỹ)
07:25:26  <_zephyris> OpenTTD-Serif missing 61: (ƠơƯựẠạẢảẤấẦầẨẩẫậắằẳẵẶặẹẻẽếềểễệỉịọỏốồổỗỘộớờỞởỡợụỦủỨứỪừửữỰựỳỷỹ)
07:25:26  <_zephyris> OpenTTD-Mono missing 61: (ƠơƯựẠạẢảẤấẦầẨẩẫậắằẳẵẶặẹẻẽếềểễệỉịọỏốồổỗỘộớờỞởỡợụỦủỨứỪừửữỰựỳỷỹ)
07:25:26  <_zephyris> OpenTTD-Small missing 61: (ƠơƯựẠạẢảẤấẦầẨẩẫậắằẳẵẶặẹẻẽếềểễệỉịọỏốồổỗỘộớờỞởỡợụỦủỨứỪừửữỰựỳỷỹ)
07:25:28  <_zephyris> OpenTTD-SmallCaps missing 61: (ƠơƯựẠạẢảẤấẦầẨẩẫậắằẳẵẶặẹẻẽếềểễệỉịọỏốồổỗỘộớờỞởỡợụỦủỨứỪừửữỰựỳỷỹ)
07:25:36  <_zephyris> The problem is the double-stacked diacritics
07:25:44  <peter1138> Yup.
07:26:21  <_zephyris> Hmm, though they are mostly lower case double stacked...
07:29:04  <_zephyris> Not sure how possible `Ấ` is in 10px 😉
07:29:17  <Heiki> Ắ Ấ Ằ Ầ Ẳ Ẩ Ẵ Ẫ are all distinct characters
07:31:14  <_zephyris> Yeah, the list above is just the ones used in OTTD translations
07:32:13  <peter1138> My side-by-side fonts branch probably needs some logic to determine if OpenTTD Sans should come before or after the fallback autodetected font.
07:34:22  <_zephyris> I think Vietnamese is the only Latin alphabet language not supported now
07:34:22  <peter1138> For non-latin script, before works reasonably well.
07:34:56  <_zephyris> How's it decided now?
07:35:10  <peter1138> In master? It's not, you only get one font.
07:36:54  <peter1138> And, still only experimenting. I didn't get as far as making this branch work on Windows or MacOS.
07:37:26  <_zephyris> What does the side-by-side fonts do? Mixing characters from different fonts?
07:38:40  <peter1138> Yes, there's some bits missing that make it properly useful though.
07:39:05  <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1367042561276907520/image.png?ex=68132519&is=6811d399&hm=027314cf2fe227131763dc4e243fd8ab1ad560dbca78307ddfa5df48f20b2f1e&
07:39:25  <peter1138> This is with `default;ChillRoundF`
07:43:27  <peter1138> The fallback font system has a few issues, mainly it only checks for characters in the compiled language files.
07:43:48  <peter1138> Anything in a savegame, online content, or servers is ignored.
07:45:26  <peter1138> So it would be nice if the text system could discover missing glyphs and load fonts "on demand".
07:46:01  <peter1138> But doing so shouldn't replace the current font for things that can be rendered.
07:48:28  <peter1138> Not sure how feasible it is to actually scan and load for fonts on demand.
07:49:26  <peter1138> What I am sure if is that changing the font configuration during text layouting (which is where unicode is mapped to glyphs) is not a great idea.
07:49:35  <peter1138> But something deferred might work.
07:51:40  <peter1138> Make a list of missing characters. Layout with usual missing glyph glyph. Then periodically check the list of missing characters and search for a font.
07:52:55  <peter1138> Our text rendering flow is somewhat different to everywhere else, I think :)
08:06:49  <_zephyris> I see the aim 🙂
08:07:44  <_zephyris> Does look nice, but you would want to avoid things like Vietnamese where one character per word might fall back to a different font...
08:08:31  <peter1138> Exactly.
08:09:35  <peter1138> That's the context for my statement at 08:32 BST :-)
08:35:32  *** mindlesstux has quit IRC
08:36:20  *** mindlesstux has joined #openttd
08:40:56  <xarick> hi
08:43:35  <_zephyris> Hmm, need to learn how to do multiple combining diacritic anchor points...
08:49:53  <_zephyris> https://cdn.discordapp.com/attachments/1008473233844097104/1367060381163589662/image.png?ex=681335b1&is=6811e431&hm=cec796082fd926fcc3c63e5a768be6cad0b2dfd9c7245aa1af0b9dc263f7371d&
08:49:54  <_zephyris> 😉
08:51:24  <peter1138> With these, I would aim to make the scalable font look good at 2x interface scale, rather than trying to squash it to fit into 1x.
08:51:55  <peter1138> (But I'm not drawing it :))
08:52:50  <_zephyris> Trust in anti-aliasing
08:55:07  <_zephyris> Based on the translations only `ẤẦẨ`  are the problematic ones.
08:55:10  <_zephyris> https://cdn.discordapp.com/attachments/1008473233844097104/1367061708987826288/image.png?ex=681336ee&is=6811e56e&hm=63c534e1f1487d8e815347653681ebe0cc13f906e57c74ef470479b3662a76a1&
08:56:36  <_zephyris> Unconventional to push the second diacritic to the side, but maybe it's readable?
08:56:55  <_zephyris> `Ắ` etc. would need a very different strategy...
09:08:26  <DorpsGek> [OpenTTD/OpenTTD] frosch123 opened pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169
09:15:45  <Heiki> “When a tone mark is used together with another diacritic, offsetting the tone mark to the right preserves consistency and avoids slowing down saccades.” https://en.wikipedia.org/wiki/Vietnamese_language_and_computers#Vietnamese_alphabet (and there’s also an example)
09:15:51  <peter1138> Yes, I have some fonts that just go up & up (Arial), and others that place adjacent (ChillRoundF)
09:21:20  <kuhnovic> Must be frustrating sometimes, working on fonts of languages you don't know / can't verify.
09:32:00  <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #14165: Codechange: replace char* with C++ style strings https://github.com/OpenTTD/OpenTTD/pull/14165#pullrequestreview-2806436199
09:34:23  <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169
09:48:34  <_zephyris> 1000%
09:52:33  <_zephyris> The offset is interesting, but I'd want to put it even more to the right than that example - ie. both diacritics directly above the letter
09:53:00  <DorpsGek> [OpenTTD/OpenTTD] hpiirai opened issue #14170: [Bug]: Missing space in timetable window https://github.com/OpenTTD/OpenTTD/issues/14170
09:54:50  <LordAro> "Muffinpooland"
09:54:56  <LordAro> hilarous.
09:55:01  <LordAro> +i
09:55:55  <kuhnovic> I had to really double check, I didn't see the issue at first 😛
09:56:06  <Heiki> haha
09:56:42  <Heiki> (16 seconds from GitHub to DorpsGek, not bad)
09:58:08  <xarick> I noticed that in the portuguese translation too, but it was fine in the english one, i thought it was just a bad translator job
09:59:01  <xarick> let me dig the screenshot
10:01:12  <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1367078327252156426/image.png?ex=68134668&is=6811f4e8&hm=38694eb724a830fa0e043036f202a0b11a76b521d716f6af8599708363b59cc3&
10:01:12  <xarick> oh, actually...
10:01:26  <xarick> didn't even notice
10:03:04  <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1367078797777829959/image.png?ex=681346d8&is=6811f558&hm=6b9bd6ba863bdfcd515b7c7eff198e68d99942ac5c5b5261a2e682fd2e0bff8d&
10:03:05  <xarick> yeh, this is what I noticed first, the portuguese affected. Should be "Munington e "
10:03:18  <xarick> "Letown e"
10:04:36  <xarick> "ficar" isn't quite the ideal word, but it's okay
10:05:10  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #14165: Codechange: replace char* with C++ style strings https://github.com/OpenTTD/OpenTTD/pull/14165
10:12:20  <peter1138> 3Yeah, the non-estimated strings are missing {SPACE}, because I presumably had a malfunction.
10:12:49  <peter1138> Not entirely keen on this bit of string concatenation though.
10:15:45  <peter1138> The strings which are in brackets are (probably) fine.
10:24:33  <peter1138> Nice, frosch123 removed the fetch_metadata macro on the sly, I've been failing to do that for ages :p
10:44:57  <reldred> Load average: 23.69 15.68 6.50
10:44:59  <reldred> hmmmmm
10:49:05  <DorpsGek> [OpenTTD/OpenTTD] frosch123 opened pull request #14171: Codechange: Use more std::string_view https://github.com/OpenTTD/OpenTTD/pull/14171
10:52:27  <peter1138> Oh right, I have to actually switch to the PR branch to compile it, not just fetch it :D
10:53:57  <goddess_ishtar> mood
10:55:25  <LordAro> reldred: depends how many cores you have :p
10:59:20  <_zephyris> https://cdn.discordapp.com/attachments/1008473233844097104/1367092955638140938/kerning-on-the-popes-tombstone-v0-rpdfp1tnvcxe1.png?ex=68135407&is=68120287&hm=09aecb54162f028abfe282590b7f92cc140764a837c13ae0125aa345d2f886d8&
10:59:20  <_zephyris> kuhnovic: But, at least it's a small audience.... Not like messing up the kerning on the pope's tombstone.
11:00:20  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169#issuecomment-2841606418
11:03:11  <reldred> LordAro: twelve, but they're on someone elses probably very oversubscribed xeon silver
11:03:50  <_zephyris> On the subject of basesets and versions - I've been poking the workflow to make ogfx2 releases happen
11:04:03  <_zephyris> But, can't find where/how the workflow knows the version, so how it can find the build file
11:04:18  * peter1138 attempts to debug the unit test... :o
11:05:03  <LordAro> reldred: alas
11:05:15  <LordAro> reldred: have you tried downloading more?
11:05:41  <reldred> LordAro: yeah but it costs more money and I haven't gotten around to downloading more of that either
11:08:18  <LordAro> reldred: try downloading some time
11:09:54  <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14171: Codechange: Use more std::string_view https://github.com/OpenTTD/OpenTTD/pull/14171
11:10:16  *** keikoz has joined #openttd
11:14:48  <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169
11:15:45  <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169#issuecomment-2841652121
11:17:42  <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169#pullrequestreview-2806717245
11:21:19  *** alpapilus has quit IRC
11:21:19  *** tabytac has quit IRC
11:21:19  *** peter1138[d] has quit IRC
11:21:19  *** honza_ has quit IRC
11:21:19  *** _glx_ has quit IRC
11:21:19  *** digitalfox has quit IRC
11:21:19  *** frosch123 has quit IRC
11:21:19  *** andythenorth has quit IRC
11:21:19  *** belajalilija has quit IRC
11:21:19  *** DorpsGek_vi has quit IRC
11:21:19  *** reldred has quit IRC
11:21:19  *** jfkuayue has quit IRC
11:21:19  *** talltyler has quit IRC
11:21:19  *** _jgr_ has quit IRC
11:21:19  *** _sara_08 has quit IRC
11:21:19  *** exceptik has quit IRC
11:21:19  *** _zephyris has quit IRC
11:21:19  *** jfs has quit IRC
11:21:19  *** sosmakaroni has quit IRC
11:21:19  *** kuhnovic has quit IRC
11:21:19  *** squirejames has quit IRC
11:21:19  *** locosage has quit IRC
11:21:19  *** brickblock19280 has quit IRC
11:21:19  *** zanooda2000 has quit IRC
11:21:19  *** truebrain has quit IRC
11:21:19  *** goddess_ishtar has quit IRC
11:21:19  *** xarick has quit IRC
11:21:37  *** DorpsGek_vi has joined #openttd
11:36:50  <peter1138> Well.
11:37:24  <LordAro> who killed the bridge this time?
11:37:32  <LordAro> did andy try to post unicode again?
11:43:22  <peter1138> ASCII only now, no unicode, we're Bri'ish.
11:46:12  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14158: Add: "Many random towns" button in scenario editor now prompts for the number of towns, with defaults based on the current new game settings https://github.com/OpenTTD/OpenTTD/pull/14158#pullrequestreview-2806779617
11:49:05  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169#issuecomment-2841723744
11:52:43  <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14171: Codechange: Use more std::string_view https://github.com/OpenTTD/OpenTTD/pull/14171
11:55:22  *** reldred has joined #openttd
11:55:22  <reldred> peter1138: proper brexit geezers
11:55:51  <peter1138> Brexit Benefits.
11:56:28  <peter1138> UKP 350,000,000 to the NHS every week.
11:56:44  <peter1138> Although maybe we should go back to L/S/D.
11:56:58  <LordAro> "UKP"
11:57:34  <peter1138> The symbol is non-ASCII :)
11:57:54  <LordAro> yeah, but GBP isn't :p
11:58:13  <peter1138> Normally.
11:58:18  <peter1138> But frankly I had a moment.
11:59:22  <peter1138> GREAT BREXIT POUNDS.
12:03:08  <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14171: Codechange: Use more std::string_view https://github.com/OpenTTD/OpenTTD/pull/14171
12:11:10  <DorpsGek> [OpenTTD/OpenTTD] frosch123 closed pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169
12:11:13  <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169#issuecomment-2841774980
12:11:22  <DorpsGek> [OpenTTD/OpenTTD] frosch123 reopened pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169
12:11:43  <peter1138> Uh...
12:11:44  *** frosch123 has joined #openttd
12:11:44  <frosch123> Wrong button :)
12:12:21  <peter1138> It's got to CI again ;(
12:12:47  <frosch123> Is hard-coding the baseset short-names ok?
12:26:33  <peter1138> Dunno, however as this now matches existing behaviour, any potential change can be done separately later.
12:33:10  <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169
12:33:13  <DorpsGek> [OpenTTD/OpenTTD] frosch123 dismissed a review for pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169#pullrequestreview-2806717245
12:33:26  <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169#issuecomment-2841828973
13:00:08  <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169#pullrequestreview-2806974841
13:23:31  <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #14169: Add: [BaseSet] Allow basesets to set minor and patch versions in obg/obs/obm files. https://github.com/OpenTTD/OpenTTD/pull/14169
13:25:32  *** WormnestAndroid has quit IRC
13:25:44  *** WormnestAndroid has joined #openttd
13:26:54  <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14171: Codechange: Use more std::string_view https://github.com/OpenTTD/OpenTTD/pull/14171
13:31:13  <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14171: Codechange: Use more std::string_view https://github.com/OpenTTD/OpenTTD/pull/14171
13:36:42  *** talltyler has joined #openttd
13:36:42  <talltyler> Infinite water could use a review if anyone is feeling inspired 🙂
13:37:14  *** xarick has joined #openttd
13:37:14  <xarick> there is a vehicle-details-tooltip branch on OpenTTD
13:37:50  <xarick> and a dependabot etc...
13:37:57  <xarick> https://github.com/OpenTTD/OpenTTD/tree/dependabot/github_actions/actions-c710d2a3e3
13:38:28  <xarick> <https://github.com/OpenTTD/OpenTTD/tree/vehicle-details-tooltip>
13:38:41  <xarick> is that a mistake
13:40:42  <LordAro> the latter definitely is
13:41:44  *** _glx_ has joined #openttd
13:41:44  <_glx_> Dependabot branch is correct
13:44:35  <peter1138> Easy to do.
13:44:41  <LordAro> mm
13:45:18  <LordAro> git remote set-url --push origin not-a-url :)
13:46:10  <xarick> strange, industry generation is now fast again
13:48:45  <xarick> one of these recent merges must have fixed something
13:52:17  *** Flygon has quit IRC
14:25:08  *** kuhnovic has joined #openttd
14:25:08  <kuhnovic> Must be the new power
14:30:13  *** nielsm has joined #openttd
14:30:58  <peter1138> Overclocked power supply.
15:17:57  *** Wormnest has joined #openttd
15:54:31  <xarick> enum still doesn't work, just tried
15:55:20  <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1367167448771596308/image.png?ex=68139968&is=681247e8&hm=3a73ab584cf0c2bc36c170af5a5c414a3be0682b80eeb38a37f3510e7bdf6eff&
15:55:20  <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1367167449056940092/image.png?ex=68139968&is=681247e8&hm=c5a6c86db9b9954f3f5ed6dbba0b40a62e2f6b4e7ce1e3e66308dba941dc7889&
15:56:04  <xarick> am I doing something wrong?
16:05:30  <_glx_> Maybe
16:06:20  <LordAro> probably
16:06:40  <_glx_> Syntax seems correct according to http://squirrel-lang.org/doc/squirrel2.html#constants
16:07:18  <LordAro> there's no commas in that link
16:07:41  <LordAro> oh, seems optional
16:07:47  <LordAro> trailing comma/trailing semicolon, maybe?
16:17:11  <xarick> reverted back to class
16:18:49  <xarick> <https://github.com/SamuXarick/LuDiAI-AfterFix/blob/master/main.nut#L222>
16:32:33  *** Smedles has quit IRC
16:32:48  *** Smedles has joined #openttd
16:49:15  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14171: Codechange: Use more std::string_view https://github.com/OpenTTD/OpenTTD/pull/14171#pullrequestreview-2807764557
16:49:19  <_glx_> according to doc it should work, maybe there's an issue with `require()`
16:59:57  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14121: Add: [NewGRF] Add purchase list Action3 CID for houses, industries, industry tiles, airports and airport tiles. https://github.com/OpenTTD/OpenTTD/pull/14121#pullrequestreview-2807806377
17:04:50  <_glx_> I remembered something like that happening before, and it's supposed to be fixed since <https://github.com/OpenTTD/OpenTTD/pull/10573>
17:13:12  <_glx_> ok I can confirm it's broken, need to verify it was actually fixed
17:19:57  <_glx_> well seems the fix isn't working
17:23:37  <_glx_> hmm garbage from other scripts detection is not helping
17:26:36  <_glx_> ok without the garbage it's easy to see it doesn't work at all
17:33:00  <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #13993: Add: Player configuration of badge display. https://github.com/OpenTTD/OpenTTD/pull/13993#pullrequestreview-2807780898
17:34:00  <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #14171: Codechange: Use more std::string_view https://github.com/OpenTTD/OpenTTD/pull/14171
17:51:02  *** peter1138[d] has joined #openttd
17:51:02  <peter1138[d]> Hmm.
17:51:35  <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1367196704025219102/image.png?ex=6813b4a7&is=68126327&hm=b235b9d267ad2ca32979cb74896e3a35156877b8da0ddaf1ea0be215230a91b9&
17:51:35  <peter1138[d]> Lots of slight string variations.
17:52:03  <peter1138[d]> And it's missing the error bit.
17:53:00  <peter1138[d]> And it assumes anything in brackets is still fine to be concatenated.
17:53:25  <peter1138[d]> (There's a lot of combinations)
17:54:24  <talltyler> Yeah, it’s a mess 🙂
17:54:42  <peter1138[d]> These strings are new...
17:55:34  <peter1138[d]> When I split it out, most of the concatenation is strings in brackets, so it's "okay".
17:55:42  <talltyler> The need to display all these things is a mess, I guess is what I mean 🙂
17:56:58  <frosch123> I have plans to redesign the order list, but after release
17:57:08  <peter1138[d]> Hmm.
17:57:19  <peter1138[d]> Should I just add two missing {SPACE} for now then?
17:57:26  <peter1138[d]> Leave it concatenated, as it's always been.
17:57:44  <frosch123> Should be fine for 15
18:01:21  <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14172: Fix #14170, 7fd0e6c27d: Missing spaces for timetabled order strings. https://github.com/OpenTTD/OpenTTD/pull/14172
18:03:08  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14172: Fix #14170, 7fd0e6c27d: Missing spaces for timetabled order strings. https://github.com/OpenTTD/OpenTTD/pull/14172#pullrequestreview-2807993766
18:10:01  <peter1138[d]> Hmm, is there space for a small left arrow.
18:10:37  <Heiki> →
18:10:38  <Heiki> oops, ←
18:15:13  <peter1138> Sprite 145/269 (and 593 but that isn't defined)
18:15:19  <peter1138> That's the small right arrow.
18:15:36  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #14172: Fix #14170, 7fd0e6c27d: Missing spaces for timetabled order strings. https://github.com/OpenTTD/OpenTTD/pull/14172#pullrequestreview-2808020953
18:16:10  <peter1138> Er, 369, not 269.
18:17:23  <peter1138> 128/352 seems to be free.
18:17:53  <peter1138> Hmm, don't we add some symbols...
18:19:19  <peter1138> Might just be the monospace font.
18:26:28  <peter1138> Oh, chars. D'oh.
18:31:49  *** gelignite has joined #openttd
18:35:08  <peter1138> Hmm, why can I no longer type into an editbox.
18:35:44  <frosch123> Many glyphs are added as font chars only, via action12.
18:35:44  <frosch123> Do you need a spriteid? Or is a string code enough?
18:36:22  <frosch123> Did I break edit boxes?
18:37:10  <peter1138> Yeah, I'm adding to chars.png and using Action 12. I just wanted to find a control code that is not otherwise occupied :)
18:43:57  <frosch123> Ah, it picked the wrong insert overload
18:48:50  <DorpsGek> [OpenTTD/OpenTTD] frosch123 opened pull request #14173: Fix 66733e2a: Textbuf was broken, and did not insert any input. https://github.com/OpenTTD/OpenTTD/pull/14173
18:49:27  <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1367211265558184008/image.png?ex=6813c237&is=681270b7&hm=26169247563195719ca66a3f6326dd8f77548f4472107b9ba6698783749fdfc9&
18:49:34  <peter1138[d]> Such a small detail.
18:50:06  <frosch123> I thought I can't test ime stuff, and did not expect to break the boring case
18:50:19  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14173: Fix 66733e2a: Textbuf was broken, and did not insert any input. https://github.com/OpenTTD/OpenTTD/pull/14173#pullrequestreview-2808097651
18:51:25  *** alpapilus has joined #openttd
18:51:25  <alpapilus> peter1138[d]: Outdated Concorde.
18:52:05  <alpapilus> peter1138[d]: Oh yeah will that be for trains and cars?
18:52:07  <alpapilus> Well
18:52:13  <alpapilus> Idk will that work.
18:52:39  <peter1138[d]> I'm not adding anything new except making the tiny arrow direction point the correct way in RTL languages.
18:54:11  <frosch123> Looks good
18:54:58  <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14172: Fix #14170, 7fd0e6c27d: Missing spaces for timetabled order strings. https://github.com/OpenTTD/OpenTTD/pull/14172
18:55:01  <DorpsGek> [OpenTTD/OpenTTD] PeterN closed issue #14170: [Bug]: Missing space in timetable window https://github.com/OpenTTD/OpenTTD/issues/14170
18:56:14  <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1367212972832981072/image.png?ex=6813c3ce&is=6812724e&hm=fd837f00e177fa1b99ab8e77f84e1c128f20ab4301d69504f3c799720b10e2f1&
18:56:14  <xarick> strange
18:57:33  <xarick> probably makes sense somewhere
19:00:01  <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14174: Fix: Small order list arrow was wrong direction for RTL text. https://github.com/OpenTTD/OpenTTD/pull/14174
19:05:38  <xarick> can we create a 3rd class of aircraft?
19:06:02  <xarick> erm 4th
19:06:39  <xarick> small, large and intercontinental or another apropriate name
19:07:30  <xarick> requires the "hub" class airports to land properly
19:10:03  *** andythenorth has joined #openttd
19:10:03  <andythenorth> probably not
19:10:26  <andythenorth> I have seen 747s land on quite short, ill equipped runways
19:10:48  <xarick> oh, real life...
19:11:28  <xarick> these airports would have an advantage
19:11:38  <xarick> they would unload and load fast
19:11:49  <alpapilus> andythenorth: Kaitak.
19:12:51  *** _zephyris has joined #openttd
19:12:51  <_zephyris> Why? Would it be fun?
19:13:04  <_zephyris> People already complain that planes are broken and crashes are annoying.
19:13:21  <alpapilus> Just don’t play planes.
19:13:30  <xarick> oh that's due to a feature killed by... you know it
19:13:45  <xarick> no plane crashes was a setting before
19:15:27  <xarick> i mean... these hub like airports should have a more prominent purpose
19:18:05  *** D-HUND is now known as debdog
19:18:44  <xarick> no newgrfs , i rather have it in the core game
19:18:54  <xarick> newgrfs just ruin the rules too much
19:30:58  *** nielsm has quit IRC
19:31:57  <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #14173: Fix 66733e2a: Textbuf was broken, and did not insert any input. https://github.com/OpenTTD/OpenTTD/pull/14173
19:33:30  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/14175
19:35:23  <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #13665: Codechange: Code style changes to AI/GS GUI files https://github.com/OpenTTD/OpenTTD/pull/13665
19:44:52  <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #13565: Add #13519, e4c511d: [Script] Saveload and config file support for handpicked configs https://github.com/OpenTTD/OpenTTD/pull/13565
19:48:37  <_glx_> so it really looks like I assumed things without actually testing in #10573
19:54:35  *** kuka_lie has joined #openttd
19:56:13  <xarick> displaying general company information taxes a bit in graphics rendering
19:59:24  <xarick> when my ai is building rail pieces, the thing goes to 10 ms graphics rendering time
20:02:05  <xarick> hmm
20:10:54  <xarick> is the vehicle tile hash faster now?
20:17:23  <peter1138[d]> xarick: Because it refreshes the company value now.
20:20:44  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/14175
20:23:33  <xarick> oh
20:23:54  <xarick> calculating company value is expensive
20:26:24  <peter1138> No, but drawing windows can be.
20:30:30  <_zephyris> Maybe I should charge for drawing windows
20:35:26  <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14174: Fix: Small order list arrow was wrong direction for RTL text. https://github.com/OpenTTD/OpenTTD/pull/14174#issuecomment-2843218666
20:46:56  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/14175
20:48:12  <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/14175#pullrequestreview-2808385209
20:51:05  <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/14175#pullrequestreview-2808406163
20:58:07  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/14175
21:03:19  <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/14175#pullrequestreview-2808440428
21:06:06  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/14175
21:11:37  <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1367247041885114489/image.png?ex=6813e389&is=68129209&hm=2aa5eba4579421ab7f90ec09b2404f4d28c5702d9967b22719f438993dc4e050&
21:11:37  <peter1138[d]> Hmm
21:11:55  <andythenorth> when are you level 41+?
21:13:14  <xarick> what is SA_FORCE?
21:14:04  <DorpsGek> [OpenTTD/OpenTTD] glx22 opened pull request #14176: Fix: [Script] Access to enum/consts defined outside of main.nut https://github.com/OpenTTD/OpenTTD/pull/14176
21:14:40  <_glx_> so #10573 actually partially worked
21:16:38  <xarick> so many changes in a short amount of time
21:16:50  <xarick> rebasing is complicated
21:17:42  <_glx_> helps to understand the changes 🙂
21:19:56  *** Wormnest has quit IRC
21:21:29  <peter1138[d]> Well.
21:21:34  <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1367249545196474499/image.png?ex=6813e5dd&is=6812945d&hm=dbfca90e191c534255f7c17bf6b6d2ed7b73a2d0b21381ef07463f25c5bc76f6&
21:21:34  <peter1138[d]> With the wrong font...
21:22:35  <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1367249801711849613/image.png?ex=6813e61b&is=6812949b&hm=5f0771a9bf33d23e13c30abce1dc964746b88cf4b5a87c1eb91ffdd67afbe9d8&
21:22:35  <peter1138[d]> Huh, worse even.
21:23:26  <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/14175#pullrequestreview-2808484633
21:23:27  <xarick> tile hash changes are massive...
21:23:34  <xarick> i can never rebase that
21:25:54  <peter1138[d]> Hmm, wonder how much more memory this uses.
21:26:04  <peter1138[d]> Probably "nothing" compared to the rest of the game state.
21:26:59  <andythenorth> does it make Horse compile slower or faster? 😛
21:27:13  <andythenorth> hmm I may have skipped lunch
21:27:42  <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1367251091091230750/image.png?ex=6813e74e&is=681295ce&hm=9490868f0096c6679646abf1e4d6c1afa064b81709941b6fdf221f70f3816a5e&
21:27:44  <peter1138[d]> Oh
21:29:14  *** tokai has joined #openttd
21:29:14  *** ChanServ sets mode: +v tokai
21:29:41  <xarick> openttd is multithreaded
21:29:44  <xarick> 100%
21:29:58  <_glx_> it is, but only outside gameloop
21:34:04  *** WormnestAndroid has quit IRC
21:34:35  <xarick> i screwed up... what is that frosch commit doing there? <https://github.com/SamuXarick/OpenTTD/pull/19/commits>
21:34:52  *** WormnestAndroid has joined #openttd
21:35:14  <xarick> how do i fix this
21:35:54  <LordAro> updating your local master, probably
21:36:09  *** tokai|noir has quit IRC
21:36:11  <LordAro> the master on your fork, specifically
21:37:47  <xarick> i did just that... must I wait?
21:38:19  <LordAro> git repos don't generally have things magically happen to them, no
21:38:45  <LordAro> although you're not wrong actually
21:38:46  *** WormnestAndroid has quit IRC
21:38:52  <LordAro> just a github display issue
21:39:24  *** WormnestAndroid has joined #openttd
21:40:13  <xarick> i have a bad feeling about this
21:45:19  *** keikoz has quit IRC
21:47:15  <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14177: Codechange: Cache layouted text for the last used width. https://github.com/OpenTTD/OpenTTD/pull/14177
21:49:10  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #14175: Codechange: more string view changes https://github.com/OpenTTD/OpenTTD/pull/14175
21:51:29  <peter1138[d]> Well.
21:52:00  <peter1138[d]> So this was the other result of trying to improve performance of some text rendering.
21:54:02  <peter1138[d]> Hmm, that may have been older than I remember.
21:55:14  <peter1138[d]> Right, what was andy's ridiculous GS doing?
21:57:47  <peter1138[d]> Thought I had a savegame for it. Hmm.
22:16:23  <peter1138[d]> Ah, that was it, #11226.
22:18:23  *** kuka_lie has quit IRC
22:25:30  <_glx_> debugger in story page ?
22:26:29  <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1367265881574604870/image.png?ex=6813f514&is=6812a394&hm=6d828a04d90a5490df58df8006bac2e773f14cfc47bd48648cbedd6bb46037b9&
22:26:29  <peter1138[d]> Uh...
22:26:30  <peter1138[d]> Yeah, that one.
22:26:50  <peter1138[d]> Oh
22:27:09  <xarick> dear _glx_ <https://github.com/SamuXarick/OpenTTD/pull/23/commits/045d18bb209ec26ccf25a11817440e5c564d13c7> I really need this
22:27:41  <xarick> it's painful to have to exit to main menu, to change a setting and then start the game again
22:28:25  <xarick> can you do something about it, in your style
22:29:25  <_glx_> very specific use case
22:30:31  <peter1138[d]> Hmm, well, I'm sure it used to work, but I can't get this test save/GS going now.
22:37:22  <_glx_> https://cdn.discordapp.com/attachments/1008473233844097104/1367268620375429231/image.png?ex=6813f7a1&is=6812a621&hm=1855529ae58a390925cbe79122ccfab151f4835fb6e5164c6b1ca9592786ed8f&
22:37:22  <_glx_> not better here
22:38:44  <_glx_> and `(undefined string)` everywhere in story book
22:42:52  <_glx_> and in 14.0 I get a crash when opening story book
22:44:20  *** Wolf01 has quit IRC
22:44:40  <peter1138[d]> Heh
22:44:55  <peter1138[d]> Is this an accidental regression discovery?
22:45:24  <peter1138[d]> Partly the issue is that zip file isn't laid out correctly.
22:47:07  <_glx_> dunno
22:47:22  <_glx_> https://cdn.discordapp.com/attachments/1008473233844097104/1367271140476719134/image.png?ex=6813f9fa&is=6812a87a&hm=5bfeb9b822d7c63d67cf2584953d897ebb50d73f3f024bf77db38306388752a4&
22:47:22  <_glx_> in crash dump I can see
22:47:52  <peter1138[d]> Not too worried about crashes in 14.0, given all the string changes since then.
22:48:19  <_glx_> in master it was fine, just showing (undefined string)
22:51:19  <_glx_> well 14.1 crashes too with the same savegame
22:52:04  <_glx_> beta shows (undefined string)
22:52:31  <_glx_> but yeag the zip is a pain 🙂
22:53:04  *** D-HUND has joined #openttd
22:55:44  <_glx_> https://cdn.discordapp.com/attachments/1008473233844097104/1367273242368151646/image.png?ex=6813fbef&is=6812aa6f&hm=999e111650e69b72a5091d4e1167a38e5ae8c8cd97a995ef3b5fe2eb2f098178&
22:55:44  <_glx_> tried new game with the script and firs from zip
22:56:09  <_glx_> https://cdn.discordapp.com/attachments/1008473233844097104/1367273350472269824/image.png?ex=6813fc09&is=6812aa89&hm=71bf712a4c1bff7dc7083afaaa3e51ea15c7a147601fa688d5dad194fcb5ff0c&
22:56:15  <_glx_> that could be an issue
22:56:29  *** debdog has quit IRC
22:58:47  <_glx_> https://cdn.discordapp.com/attachments/1008473233844097104/1367274012480110622/image.png?ex=6813fca7&is=6812ab27&hm=d2916bb7c7c765002ccd678bb22a0569c0040b52f2663b98a2b0a71d43b4aac0&
22:58:47  <_glx_> little better without this file
22:59:57  <_glx_> indeed the lang file doesn't have this string
23:03:21  <_glx_> not sure it's using the correct script as I don't see any settings
23:06:33  *** gelignite has quit IRC
23:09:34  <_glx_> ok it's loading the script I expected, but it's supposed to show settings and doesn't
23:22:05  <_glx_> ok it was a hidden setting I always forgot (ai_developer_tools)
23:36:18  <_glx_> https://cdn.discordapp.com/attachments/1008473233844097104/1367283455632605315/image.png?ex=68140572&is=6812b3f2&hm=08a0d39deb1355cdc4bdf46bc7ab03ab6133dc92e7e41723537a4f57b16a3f71&
23:36:19  <_glx_> now I have a version with the string defined
23:36:42  <_glx_> https://cdn.discordapp.com/attachments/1008473233844097104/1367283552219168788/image.png?ex=68140589&is=6812b409&hm=bd85814a783f0fc7ae6c99823dcf25b1b685e5267f73d2197efe4e4ffa4461a9&
23:36:42  <_glx_> but it still fails with
23:42:03  <_glx_> ok now I can load the save in beta and everything works fine
23:42:21  <_glx_> so we have a regression in game string stuff
23:54:47  *** Flygon has joined #openttd