IRC logs for #openttd on OFTC at 2024-12-13
            
00:00:55 <wensimehrp> andythenorth: PR created
00:00:55 <wensimehrp> https://github.com/andythenorth/firs/pull/37
00:02:09 <peter1138> You removed a new line after STR_GENERIC_NEWLINE.
00:03:57 <wensimehrp> Hmm
00:04:08 <wensimehrp> I didn't edit that line
00:05:08 <peter1138> Your editor did.
00:15:43 <peter1138> Hmm, advantage of StringParameterData being a variant is I can chunk almost anything in there...
00:29:01 *** orspeeder has joined #openttd
00:29:01 <orspeeder> would it be ok if I mad ea PR that add more ways of adding industries to the map on the scenario editor? my motivation is make my life easier to "hand place" industries to certain geographic areas when I am switching industry sets, instead of deleting everything and actually hand-placing one by one all over again...
02:43:25 *** Wormnest has joined #openttd
02:43:38 *** Wormnest has quit IRC ()
02:56:19 <talltyler> Absolutely! Scenario Editor can use some love. πŸ™‚
03:24:09 *** gnu_jj has joined #openttd
03:27:19 *** gnu_jj_ has quit IRC (Ping timeout: 480 seconds)
03:37:27 *** APTX has quit IRC (Quit: Farewell)
03:37:32 *** APTX has joined #openttd
03:44:49 *** godbed has joined #openttd
03:45:05 *** godbed_ has joined #openttd
03:48:11 *** D-HUND has quit IRC (Ping timeout: 480 seconds)
03:48:36 *** debdog has quit IRC (Ping timeout: 480 seconds)
04:26:17 <orspeeder> There was a fork of openttd some time ago called "reproducible generator", I modified that fork and added some python scripts to generate some cool maps, for example with river valleys with erosions... sadly I don't have the source of all that right now 😦
04:26:34 <orspeeder> only evidence I have of that stuff is a scenario I uploaded on the forums
04:45:36 <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/15059bc253aafa5999815a41a4ebc22c15d1694e
04:45:37 <DorpsGek> - Update: Translations from eints (by translators)
05:03:44 <talltyler> Have you seen this? It’s merged, but there hasn’t been a stable release since.
05:03:44 <talltyler> https://github.com/OpenTTD/OpenTTD/pull/10409
05:05:05 <talltyler> As far as reproducible goes, it would be easy to add a way to generate this town data from an existing game, so you could easily reuse it in a new game with a different set of GRFs, etc.
05:09:18 *** keikoz has joined #openttd
05:58:43 *** tokai|noir has joined #openttd
05:58:43 *** ChanServ sets mode: +v tokai|noir
06:05:43 *** tokai has quit IRC (Ping timeout: 480 seconds)
06:27:19 *** keikoz has quit IRC (Ping timeout: 480 seconds)
06:41:56 *** Flygon has joined #openttd
08:15:59 <truebrain> Rubidium: what was that name of that SAST that we sometimes used to check OpenTTD's code? That expensive one with thousands of suggestions?
08:16:20 <truebrain> (I am not referring to CodeQL; the one we used before that)
08:21:50 <kuhnovic> Coverity?
08:21:58 <truebrain> Yes! Tnx πŸ™‚
08:22:38 <kuhnovic> We use it at work. It's pretty amazing how much it finds.
08:24:49 <truebrain> Did they improve the UI?
08:24:58 <truebrain> It used to be a very shitty UI
08:25:18 <truebrain> The analysis itself is the best I have seen πŸ˜„
08:26:32 <kuhnovic> I've only used it with the command line haha
08:27:14 <kuhnovic> I mostly deal with the analysis reports, it's part of our CI/CD. And I don't have to set that up πŸ˜›
08:29:38 <truebrain> Ghehe
09:36:52 <DorpsGek> [OpenTTD/OpenTTD] v1993 commented on pull request #13061: Add: Set application name for SDL2 https://github.com/OpenTTD/OpenTTD/pull/13061#issuecomment-2540994593
09:39:10 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #13061: Add: Set application name for SDL2 https://github.com/OpenTTD/OpenTTD/pull/13061#issuecomment-2540999113
09:51:07 <andythenorth> what shall we do today brain?
09:53:10 <LordAro> trains?
09:54:20 <andythenorth> ok
09:54:39 <andythenorth> maybe I'll get GPT to write me a toml validator for grf lang
09:55:18 <andythenorth> might go better if I got vscode and got copilot to do it, maybe
09:55:36 <andythenorth> GPT is about 40:60 reliable in recent experiences
09:55:57 <andythenorth> I am about 20:80 reliable, so GPT is better than me
09:57:34 <LordAro> lol
10:02:53 <andythenorth> we could discuss dunning kruger though
10:03:04 <andythenorth> if I'm 20% reliable, am I competent to self-assess?
10:09:10 <LordAro> an excellent point
10:31:44 <DorpsGek> [OpenTTD/OpenTTD] v1993 opened pull request #13169: Implement 2D scrolling under SDL2 https://github.com/OpenTTD/OpenTTD/pull/13169
10:35:36 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #13169: Implement 2D scrolling under SDL2 https://github.com/OpenTTD/OpenTTD/pull/13169#issuecomment-2541128273
10:46:58 <DorpsGek> [OpenTTD/OpenTTD] v1993 commented on pull request #13169: Implement 2D scrolling under SDL2 https://github.com/OpenTTD/OpenTTD/pull/13169#issuecomment-2541154614
10:50:41 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #13169: Implement 2D scrolling under SDL2 https://github.com/OpenTTD/OpenTTD/pull/13169#issuecomment-2541168783
11:23:50 <peter1138> Who would like to take on backporting parts of <https://github.com/JGRennison/OpenTTD-patches/wiki/Signalling> to vanilla? Probably not the programmable pre-signals part...
11:43:06 <peter1138> I imagine a fair bit would need rewriting to it with vanilla's methods. Saveload and Commands are very different between vanilla and JGRPP.
12:13:40 <xarick> himself
12:14:23 <xarick> i thought poe2 early access was free to download
12:14:36 <xarick> turns out it's behind a 30 paywall
12:24:55 <kuhnovic> Guess you have to stick to openttd then πŸ˜‰
12:29:49 <xarick> kinda disappointing
12:53:34 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #13169: Implement 2D scrolling under SDL2 https://github.com/OpenTTD/OpenTTD/pull/13169#pullrequestreview-2502234316
12:54:27 <peter1138> If you want we can stick OpenTTD behind a paywall?
12:55:09 <_glx_> xarick: It's a way to support the Devs, final version will be free
12:58:14 <peter1138> So you buy "points"? What do "points" do... seems very pay-to-win to me.
12:58:19 *** nielsm has joined #openttd
13:01:05 <_glx_> If they do it like first version, you only pay for cosmetic
13:05:50 <xarick> it's pay to win even though the elitists say it's not
13:06:13 <xarick> only because they call it "convenience"
13:07:44 <DorpsGek> [OpenTTD/OpenTTD] v1993 commented on pull request #13169: Implement 2D scrolling under SDL2 https://github.com/OpenTTD/OpenTTD/pull/13169#pullrequestreview-2502260383
13:37:19 *** kuka_lie has joined #openttd
13:37:29 <peter1138> Should I ask them to change casts... `(float)x` to `static_cast<float>(x)` ?
13:38:12 <andythenorth> should I do pay-to-win Iron Horse?
13:39:40 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #13169: Implement 2D scrolling under SDL2 https://github.com/OpenTTD/OpenTTD/pull/13169#pullrequestreview-2502322841
14:39:25 <peter1138> Hmm, so changing a spec to add instead of replace...
14:45:47 <DorpsGek> [OpenTTD/OpenTTD] v1993 updated pull request #13169: Implement 2D scrolling under SDL2 https://github.com/OpenTTD/OpenTTD/pull/13169
14:48:07 <DorpsGek> [OpenTTD/OpenTTD] v1993 commented on pull request #13169: Implement 2D scrolling under SDL2 https://github.com/OpenTTD/OpenTTD/pull/13169#pullrequestreview-2502470738
15:14:54 <Rubidium> truebrain / kuhnovic: indeed coverity. It's quite a hassle to set up and I wonder whether the accounts still work. Maybe something to check in a week-and-a-half? If I don't forget
15:15:29 <truebrain> Nah, CodeQL is finnneee. I was just looking for the name :d
15:55:24 *** virtualrandomnumber has joined #openttd
15:56:16 *** virtualrandomnumber has quit IRC ()
16:00:37 <_jgr_> peter1138: Saveload and the command layer would be fairly simple mechanical changes for the most part, they're both fairly thin layers
16:00:48 <_glx_> youtube and its obsession to auto translate video titles and description is so annoying
16:01:28 <_jgr_> I expect it's what it actually does, the design, and the UI which would result in the objections
16:14:49 *** gelignite has joined #openttd
16:17:29 *** keikoz has joined #openttd
16:47:04 *** Wormnest has joined #openttd
17:14:33 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #13169: Implement 2D scrolling under SDL2 https://github.com/OpenTTD/OpenTTD/pull/13169#pullrequestreview-2502769235
17:17:30 <peter1138> Ah, those picky developers πŸ™‚
17:27:33 <DorpsGek> [OpenTTD/OpenTTD] v1993 updated pull request #13169: Implement 2D scrolling under SDL2 https://github.com/OpenTTD/OpenTTD/pull/13169
17:27:33 *** Wormnest has quit IRC (Read error: Connection reset by peer)
17:28:16 <DorpsGek> [OpenTTD/OpenTTD] v1993 updated pull request #13169: Implement 2D scrolling under SDL2 https://github.com/OpenTTD/OpenTTD/pull/13169
17:28:21 *** Wormnest has joined #openttd
17:28:53 <DorpsGek> [OpenTTD/OpenTTD] v1993 commented on pull request #13169: Implement 2D scrolling under SDL2 https://github.com/OpenTTD/OpenTTD/pull/13169#pullrequestreview-2502800990
17:58:05 <DorpsGek> [OpenTTD/OpenTTD] v1993 opened pull request #13170: Codechange: Use lowercase name for winnls.h https://github.com/OpenTTD/OpenTTD/pull/13170
18:01:31 <_glx_> ok new merge experience doesn't show approve and run
18:01:43 <peter1138> Heh, I always just commented that line out and it was fine. But that seems correct.
18:03:16 <peter1138> I don't have any track pad devices to test #13169 myself, but it seems reasonable.
18:07:06 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #13170: Codechange: Use lowercase name for winnls.h https://github.com/OpenTTD/OpenTTD/pull/13170#pullrequestreview-2502862644
18:47:19 <DorpsGek> [OpenTTD/OpenTTD] v1993 commented on pull request #13169: Implement 2D scrolling under SDL2 https://github.com/OpenTTD/OpenTTD/pull/13169#issuecomment-2542048998
18:55:57 <DorpsGek> [OpenTTD/OpenTTD] glx22 merged pull request #13170: Codechange: Use lowercase name for winnls.h https://github.com/OpenTTD/OpenTTD/pull/13170
20:07:54 <peter1138> https://cdn.discordapp.com/attachments/1008473233844097104/1317221481901592676/image.png?ex=675de59a&is=675c941a&hm=92e047d02fff8e29a3dc5f566e1cf740649d2d820bd25ce78a5adbd6edd9a64f&
20:07:54 <peter1138> Pom-te-pom
20:08:53 <_glx_> you are ready for the pitchforks ?
20:19:14 <belajalilija> <a:partyparrot:798530839839113246>
20:52:34 <wensimehrp> peter1138: Yes
20:53:20 <peter1138> https://cdn.discordapp.com/attachments/1008473233844097104/1317232915880611900/image.png?ex=675df040&is=675c9ec0&hm=154686a803c349a5927ff886bd7a76b89e07ae0eebf6f4e892d00ae9163cbd39&
20:53:20 <peter1138> Or maybe
20:53:48 <peter1138> https://cdn.discordapp.com/attachments/1008473233844097104/1317233033903869962/image.png?ex=675df05c&is=675c9edc&hm=c27d04ebf83ac4921dc802f46ec9d1a033c70888199661b86f0a9701da137ea3&
20:53:58 <peter1138> Requires tooltips to be visible and... ead.
20:54:09 <peter1138> My r key is being dodgy 😦
20:58:29 <wensimehrp> The name advanced is pretty confusing
20:59:44 <wensimehrp> They're advanced since... They are basic?
21:03:57 <andythenorth> advanced is more pro
21:11:12 <peter1138> Sausages
21:13:11 <DorpsGek> [OpenTTD/OpenTTD] Kuhnovic opened pull request #13171: Fix #13105, Fix #12333: Ship pathfinder always considers reversing https://github.com/OpenTTD/OpenTTD/pull/13171
21:14:22 <kuhnovic> xarick: I'm doing it again, making somewhat radical changes to the ship PF. So if you feel like torture testing it with your 70K savegames, then that would be very helpful.
21:14:39 <peter1138> But also kinda pointless :p
21:18:07 <kuhnovic> Not entirely. He found some major issues with my previous attempt at fixing these issues.
21:21:43 <DorpsGek> [OpenTTD/OpenTTD] Kuhnovic commented on issue #13164: [Bug]: Ship takes large detour when leaving dock https://github.com/OpenTTD/OpenTTD/issues/13164
21:24:18 <kuhnovic> Is there a way to load a savegame and ignore any NewGRFs?
21:24:46 <kuhnovic> Nevermind, that works out of the box...
21:31:17 *** kuka_lie has quit IRC (Quit: leaving)
21:36:49 <DorpsGek> [OpenTTD/OpenTTD] Kuhnovic commented on issue #12333: [Bug]: Ship pathfinder lost entering dead end after terraforming https://github.com/OpenTTD/OpenTTD/issues/12333
21:37:24 <kuhnovic> That's enough pathfinder shenanigans for today
21:40:33 <andythenorth> ` nmlc warning: "generated/lang/portuguese_BR.lng", line 220: Undefined command "SIGNED"`
21:40:36 <andythenorth> fricking translations
21:40:40 <andythenorth> yak-shaving crap
21:41:20 <andythenorth> wensimehrp: merged yours πŸ˜‰
21:41:44 <wensimehrp> hao hao
21:43:12 <andythenorth> FFS, fixing that `SIGNED WORD` to be `SIGNED_WORD` causes nmlc to find a bunch more unused strings
21:43:22 <andythenorth> this is lolz
21:53:16 <peter1138> Hmm, returning a pointer to a std::string to be filled in. Perhaps.
21:54:52 <peter1138> Passing the string to the function means formatting the string even if it's not needed.
22:03:38 <belajalilija> peter1138: No, bad, people don’t read
22:03:52 <peter1138> Quite.
22:17:32 <andythenorth> such naptimes
22:17:41 <peter1138> Well.
22:17:56 <peter1138> If I try will I actually sleep?
22:21:51 <talltyler> Worth a shot
22:26:00 *** Wormnest has quit IRC (Ping timeout: 480 seconds)
22:30:54 <_glx_> andythenorth: I really should look at outputting unused strings directly from nml
22:48:58 *** nielsm has quit IRC (Ping timeout: 480 seconds)
22:57:31 *** keikoz has quit IRC (Ping timeout: 480 seconds)
23:05:34 <DorpsGek> [OpenTTD/nml] glx22 opened pull request #348: Add: --list-unused-strings to list the unused strings https://github.com/OpenTTD/nml/pull/348