IRC logs for #openttd on OFTC at 2025-04-30
            
00:18:11 *** WormnestAndroid has quit IRC (Remote host closed the connection)
00:18:20 *** WormnestAndroid has joined #openttd
01:43:32 *** WormnestAndroid has quit IRC (Remote host closed the connection)
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 (Ping timeout: 480 seconds)
02:42:01 *** Wormnest has quit IRC (Quit: Leaving)
03:55:48 *** WormnestAndroid has quit IRC (Remote host closed the connection)
03:55:53 *** WormnestAndroid has joined #openttd
04:08:07 *** 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 (Ping timeout: 480 seconds)
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:05 <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:05 <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:25 <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:30 <peter1138> Exactly.
08:09:34 <peter1138> That's the context for my statement at 08:32 BST :-)
08:35:32 *** mindlesstux has quit IRC (Quit: The Lounge - https://thelounge.chat)
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:53 <_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:04 <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:04 <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:53 <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 *** frosch123 has quit IRC (Remote host closed the connection)
11:21:19 *** andythenorth has quit IRC (Remote host closed the connection)
11:21:19 *** belajalilija has quit IRC (Remote host closed the connection)
11:21:19 *** DorpsGek_vi has quit IRC (Remote host closed the connection)
11:21:19 *** reldred has quit IRC (Remote host closed the connection)
11:21:19 *** jfkuayue has quit IRC (Remote host closed the connection)
11:21:19 *** talltyler has quit IRC (Remote host closed the connection)
11:21:19 *** alpapilus has quit IRC (Remote host closed the connection)
11:21:19 *** tabytac has quit IRC (Remote host closed the connection)
11:21:19 *** peter1138[d] has quit IRC (Remote host closed the connection)
11:21:19 *** honza_ has quit IRC (Remote host closed the connection)
11:21:19 *** _glx_ has quit IRC (Remote host closed the connection)
11:21:19 *** digitalfox has quit IRC (Remote host closed the connection)
11:21:19 *** _jgr_ has quit IRC (Remote host closed the connection)
11:21:19 *** _sara_08 has quit IRC (Remote host closed the connection)
11:21:19 *** exceptik has quit IRC (Remote host closed the connection)
11:21:19 *** _zephyris has quit IRC (Remote host closed the connection)
11:21:19 *** jfs has quit IRC (Remote host closed the connection)
11:21:19 *** sosmakaroni has quit IRC (Remote host closed the connection)
11:21:19 *** kuhnovic has quit IRC (Remote host closed the connection)
11:21:19 *** squirejames has quit IRC (Remote host closed the connection)
11:21:19 *** locosage has quit IRC (Remote host closed the connection)
11:21:19 *** brickblock19280 has quit IRC (Remote host closed the connection)
11:21:19 *** zanooda2000 has quit IRC (Remote host closed the connection)
11:21:19 *** truebrain has quit IRC (Remote host closed the connection)
11:21:19 *** goddess_ishtar has quit IRC (Remote host closed the connection)
11:21:19 *** xarick has quit IRC (Remote host closed the connection)
11:21:36 *** 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:21 *** reldred has joined #openttd
11:55:21 <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 (Remote host closed the connection)
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 (Remote host closed the connection)
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:30 <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 (Quit: http://quassel-irc.org - Chat comfortably. Anywhere.)
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:41 <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:48 *** gelignite has joined #openttd
18:35:07 <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:38 <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 (Ping timeout: 480 seconds)
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:53 <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:13 <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 (Ping timeout: 480 seconds)
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 (Ping timeout: 480 seconds)
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 (Ping timeout: 480 seconds)
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 (Read error: Connection reset by peer)
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 (Ping timeout: 480 seconds)
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:28 <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:01 <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 (Remote host closed the connection)
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 (Quit: Once again the world is quick to bury me.)
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 (Ping timeout: 480 seconds)
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:56 <_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 (Quit: Stay safe!)
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:19 <_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