IRC logs for #openttd on OFTC at 2025-05-13
            
00:02:11 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #14258: Fix #14256, dfd9fbf873: Company finance windows not updated when paused. https://github.com/OpenTTD/OpenTTD/pull/14258#pullrequestreview-2834958048
00:02:12 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
00:02:47 *** WormnestAndroid has joined #openttd
00:03:42 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #14259: Fix 0829604b4f: Dragging of selected order was still not possible. https://github.com/OpenTTD/OpenTTD/pull/14259#pullrequestreview-2834959787
00:04:14 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #14261: Fix 0ce30d05c8: Replacing a dropdown list could reposition it partially off-screen. https://github.com/OpenTTD/OpenTTD/pull/14261#pullrequestreview-2834960261
00:06:02 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #14254: Change: Remove Apply button from NewGRF config window when unneeded. https://github.com/OpenTTD/OpenTTD/pull/14254#pullrequestreview-2834962048
00:10:38 *** akimoto has quit IRC (Remote host closed the connection)
00:17:27 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14262: Codechange: Make more use of std::byte for generic buffers. https://github.com/OpenTTD/OpenTTD/pull/14262
00:17:52 <peter1138[d]> Oof, thanks.
00:21:50 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14258: Fix #14256, dfd9fbf873: Company finance windows not updated when paused. https://github.com/OpenTTD/OpenTTD/pull/14258#pullrequestreview-2834975773
00:22:27 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14254: Change: Remove Apply button from NewGRF config window when unneeded. https://github.com/OpenTTD/OpenTTD/pull/14254
00:22:48 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14261: Fix 0ce30d05c8: Replacing a dropdown list could reposition it partially off-screen. https://github.com/OpenTTD/OpenTTD/pull/14261
00:23:02 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14259: Fix 0829604b4f: Dragging of selected order was still not possible. https://github.com/OpenTTD/OpenTTD/pull/14259
00:23:30 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14258: Fix #14256, dfd9fbf873: Company finance windows not updated when paused. https://github.com/OpenTTD/OpenTTD/pull/14258
00:23:33 <DorpsGek> [OpenTTD/OpenTTD] PeterN closed issue #14256: [Bug]: Burrow and Repay Loan buttons do not refresh https://github.com/OpenTTD/OpenTTD/issues/14256
00:23:43 <peter1138[d]> Spam, glorious spam.
01:36:27 *** Flygon has quit IRC (Read error: Connection reset by peer)
01:39:37 *** tokai|noir has joined #openttd
01:39:37 *** ChanServ sets mode: +v tokai|noir
01:46:30 *** tokai has quit IRC (Ping timeout: 480 seconds)
01:49:33 *** Wormnest has joined #openttd
01:55:49 <yiffgirl> peter1138[d]: ```
01:55:49 <yiffgirl> (gdb) backtrace
01:55:49 <yiffgirl> #0 0x000071839f8ade22 in ?? () from /usr/lib/libc.so.6
01:55:49 <yiffgirl> #1 0x000071839f8a1fda in ?? () from /usr/lib/libc.so.6
01:55:49 <yiffgirl> #2 0x000071839f8a2024 in ?? () from /usr/lib/libc.so.6
01:55:51 <yiffgirl> #3 0x000071839f91c05e in poll () from /usr/lib/libc.so.6
01:55:51 <yiffgirl> #4 0x000071839e9a0e77 in ?? () from /usr/lib/libSDL3.so.0
01:55:53 <yiffgirl> #5 0x000071839e98c876 in ?? () from /usr/lib/libSDL3.so.0
01:55:53 <yiffgirl> #6 0x00005d1fcee199f1 in VideoDriver_SDL_OpenGL::Paint() ()
01:55:55 <yiffgirl> #7 0x00005d1fcee1dd53 in VideoDriver::Tick() ()
01:55:55 <yiffgirl> #8 0x00005d1fcee18367 in VideoDriver_SDL_Base::MainLoop() ()
01:55:57 <yiffgirl> #9 0x00005d1fcefba18c in openttd_main(int, char**) ()
01:55:57 <yiffgirl> #10 0x000071839f8376b5 in ?? () from /usr/lib/libc.so.6
01:55:59 <yiffgirl> #11 0x000071839f837769 in __libc_start_main () from /usr/lib/libc.so.6
01:55:59 <yiffgirl> #12 0x00005d1fceab6875 in _start ()
01:56:01 <yiffgirl> (gdb)
01:59:52 <yiffgirl> so... i guess something is going funky in Paint()? maybe?
01:59:52 <yiffgirl> this doesn't seem all that helpful
02:11:59 *** gnu_jj_ has joined #openttd
02:15:35 *** gnu_jj has quit IRC (Ping timeout: 480 seconds)
02:27:23 *** Wormnest has quit IRC (Quit: Leaving)
02:28:10 <_glx_> well we don't support SDL3
02:29:10 <_glx_> and the trace shows it's something happening in SDL3
03:03:45 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
03:03:46 *** WormnestAndroid has joined #openttd
03:03:49 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
03:03:50 *** WormnestAndroid has joined #openttd
03:03:53 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
03:04:05 *** WormnestAndroid has joined #openttd
03:05:43 <yiffgirl> i think that's just the sdl 2 to 3 shim rather than it being compiled with sdl3
03:07:17 <yiffgirl> anyway. i assume that's all I can learn here?
03:07:49 <dwfreed> install libc and libsdl debug symbols so your backtrace is actually useful
04:08:23 *** keikoz has joined #openttd
04:30:17 <yiffgirl> guh
04:30:17 <yiffgirl> gdb only bothers to load the symbols from debuginfod when not running as sudo, but trying to connect from a non-sudo gdb gets denied by ptrace... and even with sudo i can't tell it not to do that
04:42:33 <yiffgirl> yiffgirl: update: `echo 0 | sudo tee /proc/sys/kernel/yama/ptrace_scope` worked, let's see what happens...
04:47:43 <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/c7188589996433dd3ef9533a1a9ab0636118268c
04:47:44 <DorpsGek> - Update: Translations from eints (by translators)
04:50:44 <yiffgirl> https://cdn.discordapp.com/attachments/1008473233844097104/1371711236256960522/message.txt?ex=68242123&is=6822cfa3&hm=4c5c3b2e61b40e964f9c8a73386bfa5156a82557e4338a8da6d414d9a5cbad7a&
04:50:44 <yiffgirl> hm
05:12:28 *** keikoz has quit IRC (Ping timeout: 480 seconds)
06:01:46 <peter1138[d]> Try a stack trace of a different thread. That one might just waiting for a locked one
06:03:02 *** keikoz has joined #openttd
06:05:40 <peter1138[d]> `thread apply all bt`
06:29:21 <pickpacket> Hmm. I've forgotten again; what's the upper limit of how big FIRS (or any other industry set) can become?
06:29:37 <pickpacket> Something about cargo id?
06:31:54 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14235: Fix #14234, 5aa4ad5135: Crash due to NUL chars in external strings. https://github.com/OpenTTD/OpenTTD/pull/14235
06:32:33 <andythenorth> pickpacket: 64 cargos
06:32:54 <andythenorth> it's "enough"
06:35:40 <peter1138[d]> Pom te pom.
06:38:06 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1371738254830862437/image.png?ex=68243a4d&is=6822e8cd&hm=b79757d9f15625ae641d01126d759a37f32fe7ca277aac6ebacdd73a908ef5de&
06:38:06 <andythenorth> raising the limit would lead to situations like this
06:38:15 <andythenorth> very bad for compile times ๐Ÿ˜›
06:40:33 <yiffgirl> https://cdn.discordapp.com/attachments/1008473233844097104/1371738873499090944/message.txt?ex=68243ae1&is=6822e961&hm=f7c59fc3ff57bfaae1917a4ac5a0b6211dd22a43fb13f98efb44e26960fb3560&
06:40:33 <yiffgirl> peter1138[d]: wow. that's a lot
06:42:37 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #11495: Change: Simplify sprite cache memory management https://github.com/OpenTTD/OpenTTD/pull/11495
06:47:33 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #11495: Change: Simplify sprite cache memory management https://github.com/OpenTTD/OpenTTD/pull/11495#issuecomment-2875247205
06:48:15 <pickpacket> andythenorth: I don't understand what's problematic about that output ๐Ÿค” How many cargo types are in FIRS now?
06:48:40 <andythenorth> nobody really needs 11388 trains in one grf ๐Ÿ˜›
06:48:49 <andythenorth> and the compile time is shockingly bad
06:49:09 <pickpacket> What has that got to do with cargo ids, though?
06:49:31 <pickpacket> The number of trains, I mean
06:50:01 <andythenorth> it's more about limits, and why they're good
06:50:10 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1371741291007709204/image.png?ex=68243d21&is=6822eba1&hm=ad6146323d2463f955bd2995bb7bbb5511fb0aa482a66a8a83ea50835139bf67&
06:50:10 <andythenorth> anyway
06:50:16 <andythenorth> FIRS
06:50:41 <peter1138[d]> yiffgirl: Mostly threads set up by other things, e.g. audio processing.
06:50:53 <peter1138[d]> However, yeah, doesn't seem to be ottd itself in a loop.
06:51:09 <pickpacket> The reason I ask is because someone in Hellish's last stream commented that "chickens and eggs should be part of FIRS" and Hellish suggested joining the FIRS dev team and adding it. And it got me wondering whether it's even possible to add anything more to it ๐Ÿ˜‚
06:51:38 <andythenorth> fork
06:51:53 <pickpacket> *is* it possible?
06:52:32 <peter1138[d]> I would suggest that that is more of a "that's a stupd idea so you'll have to do it yourself if you want to" answer :p
06:52:48 <pickpacket> Well, obv
06:53:23 <pickpacket> No sense in trying if it's not even technically possible though ๐Ÿ˜„
06:59:41 <andythenorth> it's entirely possible
06:59:51 <andythenorth> you'd have to figure out where chickens and eggs come from
06:59:53 <andythenorth> and which makes which
07:01:38 <yiffgirl> the "firs dev team" is just you andy, right?
07:18:44 <yiffgirl> peter1138[d]: i wonder if i tell it to continue it'll fix itself... nope. had to send it a stop signal again.
07:23:14 <yiffgirl> but i guess if it's not actually stuck in a loop then something must be wrong with the graphics on my end
07:23:14 <yiffgirl> arch doesn't even package actual sdl2 any more, just the 2-to-3 shim
07:23:34 <yiffgirl> is there anything more i can do from here?
07:23:39 <peter1138[d]> Yeah "it kinda works, let's ship it"
07:24:00 <peter1138[d]> Disable "hardware acceleration" in OpenTTD and hope it doesn't happen again.
07:24:26 <yiffgirl> will do.
07:29:03 *** reldred has joined #openttd
07:29:03 <reldred> Hardware acceleration is inconsistent even on Windows
07:29:19 <reldred> Even between Nvidia and AMD
07:29:31 <reldred> Worked really well on a mac I had for a while though
07:29:34 <reldred> Go figure
07:30:21 <andythenorth> yiffgirl: I get help, but yes
07:31:49 <reldred> but then tonnes of people have no issues with hardawre acceleration so yeah, strange
07:33:29 <pickpacket> andythenorth: I didn't understand the 1000+ trains bit. Does the number of cargoes affect that?
07:34:35 <andythenorth> not at all no
07:35:03 <andythenorth> it was more of an abstract point ๐Ÿ˜›
07:49:21 <pickpacket> ah
08:33:52 *** Smedles_ has joined #openttd
08:37:18 *** Smedles has quit IRC (Ping timeout: 480 seconds)
08:50:21 *** mindlesstux has quit IRC (Quit: The Lounge - https://thelounge.chat)
08:51:22 *** mindlesstux has joined #openttd
08:57:24 <xarick> hi
09:01:23 <xarick> std::byte is returning?
09:07:16 *** SigHunter has quit IRC ()
09:10:03 *** SigHunter has joined #openttd
10:08:12 *** kuhnovic has joined #openttd
10:08:12 <kuhnovic> Lunch and coffee, what a day
11:12:02 *** WormnestAndroid has quit IRC (Remote host closed the connection)
11:12:06 *** WormnestAndroid has joined #openttd
11:15:27 *** Flygon has joined #openttd
11:31:18 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1371812039525404733/image.png?ex=68247f05&is=68232d85&hm=fb117a810589da91c8e2a8fd18f0e75aaadd6558e242f5b144d13166e762da19&
11:31:18 <xarick> air stuff kind of working...
11:31:53 <xarick> air has a complete different build logic than the other transport modes...
11:32:46 <xarick> and I'm trying to fit it in
12:15:22 <peter1138[d]> Coffee eh?
12:17:07 <pickpacket> xarick: I'm trying to fit in, too
12:17:12 <pickpacket> Oh, wait...
12:44:26 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14262: Codechange: Make more use of std::byte for generic buffers. https://github.com/OpenTTD/OpenTTD/pull/14262#pullrequestreview-2836617339
12:49:40 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #14260: Change: [Script] ScriptVehicleList_Station accepts an optional VehicleType parameter https://github.com/OpenTTD/OpenTTD/pull/14260#pullrequestreview-2836635251
13:01:57 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14262: Codechange: Make more use of std::byte for generic buffers. https://github.com/OpenTTD/OpenTTD/pull/14262
13:13:14 <pickpacket> Would this count as a regression? https://www.tt-forums.net/viewtopic.php?t=92307&sid=418485b11c71cd7bd4a7365317e45a99
13:18:06 *** kuka_lie has joined #openttd
13:23:27 <peter1138[d]> No, it was a deliberate change.
13:28:43 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #14262: Codechange: Make more use of std::byte for generic buffers. https://github.com/OpenTTD/OpenTTD/pull/14262#pullrequestreview-2836791733
13:35:22 <LordAro> pickpacket: 1172
13:38:29 <pickpacket> LordAro: issue or PR?
13:39:26 <pickpacket> ... neither
13:39:36 <pickpacket> What am I looking for?
13:42:07 <peter1138[d]> xkcd probably
13:51:21 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14262: Codechange: Make more use of std::byte for generic buffers. https://github.com/OpenTTD/OpenTTD/pull/14262
13:53:26 <pickpacket> ohhh. I think I know which one in that case.
13:53:42 <pickpacket> Yup!
14:07:32 <LordAro> :)
14:19:52 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14235: Fix #14234, 5aa4ad5135: Crash due to NUL chars in external strings. https://github.com/OpenTTD/OpenTTD/pull/14235#pullrequestreview-2836989678
14:34:35 <xarick> ctrl - double click
14:34:38 <xarick> for service
14:36:03 <xarick> how does unbunch order works in that way anyway?
14:36:20 <xarick> it's not part of the orders
14:38:18 <xarick> oh nevermind, i am confused
14:39:06 <xarick> ah while creating orders
14:40:32 <xarick> > I very rarely use the unbunch functionality because I prefer to use timetables to spread the vehicles evenly
14:40:32 <xarick> uh...
14:40:53 <xarick> but isn't exactly that what unbunch does?
14:41:28 <DorpsGek> [OpenTTD/OpenTTD] glx22 merged pull request #14260: Change: [Script] ScriptVehicleList_Station accepts an optional VehicleType parameter https://github.com/OpenTTD/OpenTTD/pull/14260
14:41:54 <_glx_> unbunch doesn't use timetable at all
14:42:21 <_glx_> autoseparation (JGRPP feature) uses timetables
14:42:53 <talltyler> If you are using timetables, unbunching does set them "on time" when they leave the unbunching depot. So you can use timetables to choose how long vehicle stop at stations, if you want.
14:43:22 <talltyler> But the timetable has no effect on the unbunching behavior.
14:43:30 <talltyler> So you are correct, I am just being pedantic. ๐Ÿ˜„
14:43:35 <_glx_> I don't imagine an IA using timetables anyway
14:43:51 <talltyler> I don't think they can
14:44:09 <xarick> it's not implemented ๐Ÿ˜ฆ
14:44:29 <xarick> and I don't understand timetables to start coding on that
14:45:03 <_glx_> it's hard enough to use for human, so let it out of AI world
14:46:30 <_glx_> order handling is already a pain for AIs
14:50:07 <xarick> can we have the unbunch flag?
14:52:44 <xarick> 14260 was already merged
14:56:52 *** nielsm has joined #openttd
15:01:59 <xarick> regression.txt conflicts
15:02:13 <xarick> because it prints the line number
15:02:46 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #13476: Add: [Script] GetEconomyAge and GetWagonEconomyAge https://github.com/OpenTTD/OpenTTD/pull/13476
15:04:02 <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #13456: Add: [Script] ScriptVehicleList_Waypoint https://github.com/OpenTTD/OpenTTD/pull/13456#pullrequestreview-2837159733
15:06:55 <xarick> oh snap, conflicts with the new stuff
15:07:20 *** Wormnest has joined #openttd
15:08:45 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #13455: Change: [Script] ScriptVehicleList_Station no longer works for waypoints https://github.com/OpenTTD/OpenTTD/pull/13455#issuecomment-2876914999
15:09:44 <xarick> oh good point
15:13:08 <xarick> I actually remember checking if any AI was using the function for rail waypoints and I found none
15:13:30 <xarick> do i still create a compatibility thiingy
15:15:16 <xarick> how am I supposed to do this with separate PRs ๐Ÿ˜ฆ
15:15:25 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14235: Fix #14234, 5aa4ad5135: Crash due to NUL chars in external strings. https://github.com/OpenTTD/OpenTTD/pull/14235#pullrequestreview-2837196931
15:16:13 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14235: Fix #14234, 5aa4ad5135: Crash due to NUL chars in external strings. https://github.com/OpenTTD/OpenTTD/pull/14235
15:21:41 <_glx_> xarick: wait for #13456 merge, then you can add compat layer to #13455 (should not be too hard, something like calling the new ScriptVehicleList_Station() and add the list from ScriptVehicleList_Waypoint())
15:23:48 <xarick> scriptwaypointlist should also use the vehicle type treatment, gonna try copy paste
15:26:40 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #13455: Change: [Script] ScriptVehicleList_Station no longer works for waypoints https://github.com/OpenTTD/OpenTTD/pull/13455
15:31:40 <_glx_> waypoints are specific to one type
15:32:15 <_glx_> no need to add vehicle type filter
15:33:42 <xarick> oh, ok
15:34:06 <xarick> can't have a waypoint that is both rail and road nowadays?
15:34:47 <xarick> yeh, you're right, it can't
15:36:02 <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #13419: Fix: [Script] Make ScriptOrder functions aware of road waypoints https://github.com/OpenTTD/OpenTTD/pull/13419#pullrequestreview-2837275820
15:43:35 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #14235: Fix #14234, 5aa4ad5135: Crash due to NUL chars in external strings. https://github.com/OpenTTD/OpenTTD/pull/14235#pullrequestreview-2837299547
15:45:51 <DorpsGek> [OpenTTD/nml] frosch123 merged pull request #373: Add: Road/tram/railtype variable 0x45. https://github.com/OpenTTD/nml/pull/373
15:45:58 <DorpsGek> [OpenTTD/nml] frosch123 merged pull request #374: Add: Parent scope for airports and airport tiles. https://github.com/OpenTTD/nml/pull/374
15:47:20 <DorpsGek> [OpenTTD/nml] frosch123 updated pull request #375: Add: Constants for (new) station and roadstop animation and random triggers. https://github.com/OpenTTD/nml/pull/375
15:49:43 <DorpsGek> [OpenTTD/nml] frosch123 approved pull request #342: Codechange: Vendor in latest ply version https://github.com/OpenTTD/nml/pull/342#pullrequestreview-2837317425
16:00:31 *** Flygon has quit IRC (Remote host closed the connection)
16:01:41 <DorpsGek> [OpenTTD/OpenTTD] github-advanced-security[bot] commented on pull request #14235: Fix #14234, 5aa4ad5135: Crash due to NUL chars in external strings. https://github.com/OpenTTD/OpenTTD/pull/14235#pullrequestreview-2837356658
16:02:38 <peter1138[d]> Meh
16:03:40 <DorpsGek> [OpenTTD/nml] glx22 updated pull request #342: Codechange: Vendor in latest ply version https://github.com/OpenTTD/nml/pull/342
16:19:16 <DorpsGek> [OpenTTD/nml] frosch123 commented on pull request #359: Add: Support for NewGRF badges. https://github.com/OpenTTD/nml/pull/359#pullrequestreview-2837339825
16:19:54 <DorpsGek> [OpenTTD/nml] frosch123 merged pull request #375: Add: Constants for (new) station and roadstop animation and random triggers. https://github.com/OpenTTD/nml/pull/375
16:21:27 <DorpsGek> [OpenTTD/nml] glx22 closed issue #332: Bug: `const` keyword not working https://github.com/OpenTTD/nml/issues/332
16:21:30 <DorpsGek> [OpenTTD/nml] glx22 merged pull request #342: Change: Vendor in latest ply version https://github.com/OpenTTD/nml/pull/342
16:24:03 <xarick> 13409 needs a compatibility layer
16:24:09 <xarick> ? question
16:31:30 <_glx_> yes, for older AI if AIStation::GetStationID fails it should fallback to AIWaypoint::GetWaypointID to restore old behaviour
16:36:00 *** gelignite has joined #openttd
16:36:37 <xarick> hmm, that's a bit of a problem
16:37:02 <_glx_> no it's simple to do
16:37:37 <xarick> AITile.IsStationTile
16:38:58 <xarick> Unsure if it's safe to rely on AIStation::GetStationID
16:39:02 <xarick> to fail
16:39:14 *** Wolf01 has joined #openttd
16:40:01 <xarick> okay maybe it's fine
16:42:30 <_glx_> ```local id = AIStation::GetStationID(tile);
16:42:30 <_glx_> return (id == -1) ? AIWaypoint::GetWaypointID(tile) : id;```
16:42:46 <_glx_> or something like that
16:43:35 <xarick> I was looking for the official invalid station constant
16:43:40 <xarick> id
16:43:46 <_glx_> there is not
16:43:56 <_glx_> but typically should be -1
16:44:14 <xarick> StationID::Invalid()
16:45:05 <_glx_> other option is AIStation.IsValidStation(id)
16:45:10 <xarick> ah, found it!
16:45:18 <xarick> it's in BaseStation for reasons...
16:46:26 <_glx_> ah of course, since it's shared with waypoints
16:47:25 <xarick> ```AIStation.GetStationIDCompat14 <- AIStation.GetStationID;
16:47:25 <xarick> AIStation.GetStationID <- function(tile) {
16:47:25 <xarick> local id = AIStation.GetStationIDCompat14(tile);
16:47:25 <xarick> if (id == AIBaseStation.STATION_INVALID) return AIWaypoint.GetWaypointID(tile);
16:47:25 <xarick> return id;
16:47:27 <xarick> }```
16:49:31 <_glx_> looks good
16:50:08 <DorpsGek> [OpenTTD/nml] frosch123 closed issue #303: Prevent train overturning `misc_flags` function https://github.com/OpenTTD/nml/issues/303
16:50:11 <DorpsGek> [OpenTTD/nml] frosch123 commented on issue #303: Prevent train overturning `misc_flags` function https://github.com/OpenTTD/nml/issues/303
16:56:15 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #13409: Change: [Script] GetStationID can no longer return the StationID of waypoints https://github.com/OpenTTD/OpenTTD/pull/13409
17:04:14 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #11495: Change: Simplify sprite cache memory management https://github.com/OpenTTD/OpenTTD/pull/11495
17:08:16 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #13398: Add: [Script] Multiple road waypoints API functions https://github.com/OpenTTD/OpenTTD/pull/13398
17:13:17 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #12249: Add: [Script] Unbunch order flag https://github.com/OpenTTD/OpenTTD/pull/12249
17:18:03 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #12096: Add: [Script] ScriptAirportTypeList https://github.com/OpenTTD/OpenTTD/pull/12096
17:28:08 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #12093: Add: [Script] GetTileOfAirport https://github.com/OpenTTD/OpenTTD/pull/12093
17:37:19 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick commented on pull request #13409: Change: [Script] GetStationID can no longer return the StationID of waypoints https://github.com/OpenTTD/OpenTTD/pull/13409#issuecomment-2877425171
17:39:35 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #12088: Add: [Script] GetAirportPlaneCompatibility https://github.com/OpenTTD/OpenTTD/pull/12088
17:45:07 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #12087: Add: [Script] GetAirportNumTerminals https://github.com/OpenTTD/OpenTTD/pull/12087
17:55:37 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #12084: Add: [Script] GetAirportNumHangars, simplify GetNumHangars https://github.com/OpenTTD/OpenTTD/pull/12084
18:00:57 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #12083: Add: [Script] Hangar index parameter to ScriptAirport::GetHangarOfAirport https://github.com/OpenTTD/OpenTTD/pull/12083
18:18:51 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #11495: Change: Simplify sprite cache memory management https://github.com/OpenTTD/OpenTTD/pull/11495#issuecomment-2877529433
18:48:43 <peter1138[d]> Well,
18:49:01 <peter1138[d]> I lack the ability to test low memory ๐Ÿ˜ฎ
18:51:43 <peter1138[d]> Hmm, cgroups perhaps.
18:52:25 *** _jgr_ has joined #openttd
18:52:25 <_jgr_> I doubt that the old behaviour will work at all on non-Windows due to overcommit
19:05:19 *** Wormnest has quit IRC (Ping timeout: 480 seconds)
19:23:00 <xarick> when is darwin 300 available?=
19:23:14 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
19:24:28 *** WormnestAndroid has joined #openttd
19:26:47 <frosch123> It works with ulimit.
19:26:47 <frosch123> The intention is not low memory though. The test is about low address space in 32bit builds for windows and emscripten, where we do not know how much heap is occupied by external deps.
19:31:59 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14263: Codechange: Add commonly used BaseBitSet functions. https://github.com/OpenTTD/OpenTTD/pull/14263
19:32:09 <peter1138[d]> Which ulimit?
19:33:38 <peter1138[d]> `ulimit -m` hasn't worked since kernel 2.4.30...
19:37:23 <peter1138[d]> Okay well anyway, if it's desirable and the intention is to ensure that the sprite cache is pre-allocated to a certain size, this is not going to help.
19:40:06 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #14263: Codechange: Add commonly used BaseBitSet functions. https://github.com/OpenTTD/OpenTTD/pull/14263#pullrequestreview-2837915746
19:53:02 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14263: Codechange: Add commonly used BaseBitSet functions. https://github.com/OpenTTD/OpenTTD/pull/14263
19:53:03 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
19:54:38 <peter1138[d]> KOMs eh
19:56:20 *** WormnestAndroid has joined #openttd
19:56:37 <LordAro> peter1138[d]: pros of uploading first
19:58:46 *** akimoto has joined #openttd
20:09:06 *** WormnestAndroid has quit IRC (Read error: No route to host)
20:09:07 *** WormnestAndroid has joined #openttd
20:17:10 *** gelignite has quit IRC ()
20:21:32 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14263: Codechange: Add commonly used BaseBitSet functions. https://github.com/OpenTTD/OpenTTD/pull/14263#pullrequestreview-2838025993
20:36:34 <peter1138[d]> Alright, how would you implement an iterator that starts "in the middle" and alternately takes from the front and back?
20:36:59 <LordAro> "carefully"
20:37:11 <peter1138[d]> e.g. a range of `0 1 2 3 4 5 6 7 8 9` should go something like `5 4 6 3 7 2 8 1 9 0`.
20:37:25 <peter1138[d]> With added fun that "the middle" might be somewhere else.
20:37:40 <LordAro> "ask chatgpt"
20:37:40 <peter1138[d]> e.g. `3 2 4 1 5 0 6 7 8 9`
20:37:45 <peter1138[d]> lolz
20:41:47 <xarick> something distance maybe?
20:43:27 <xarick> I did a similar thing when iterating the tiles of a road path. I want to place the depot closest to the middle of the road.
20:44:12 <xarick> middle of the path*
20:46:02 <xarick> https://github.com/SamuXarick/LuDiAI-AfterFix/blob/7e14c8607f2d6aeae8f96da7002503b59b90b743/RoadBuildManager.nut#L1230-L1232
20:48:35 <Rubidium> peter1138[d]: is range in that context essentially a span? Or a continuous range of integers like iota?
20:49:35 *** nielsm has quit IRC (Ping timeout: 480 seconds)
20:51:19 *** WormnestAndroid has quit IRC (Read error: No route to host)
20:53:46 <peter1138[d]> span
20:54:28 <talltyler> I need to write code more often, I'm super rusty
20:55:16 *** WormnestAndroid has joined #openttd
21:00:58 <talltyler> What am I missing here? It buys the selected vehicle, never randomizing (even if I always randomize without checking the flag).
21:00:58 <talltyler> https://github.com/2TallTyler/OpenTTD/commit/0ef58cf2e20a312a26f0c460a2ffe230be497dec
21:06:05 <talltyler> Trying to resolve Andyโ€™s random horse wagon problem, but eight hours of Unreal Engine (real job) has cooked my brain ๐Ÿ˜›
21:10:05 <peter1138[d]> 1) If the variant isn't expanded, no children will be listed.
21:10:36 <peter1138[d]> 2) You only include direct children, no further descendants.
21:13:57 <talltyler> Mmm, is there a better list of available engines to iterate through?
21:14:19 <talltyler> Looking for grandchildren, etc., sounds like recursion will be needed ๐Ÿ™‚
21:14:37 <talltyler> (As youโ€™ve already done when drawing them)
21:20:13 <peter1138[d]> Well, there's the actual engine list...
21:24:10 <peter1138[d]> (And you should probably just fill a container of EngineID, not GUIEngineListItems)
21:24:32 <talltyler> That does sound easier ๐Ÿ™‚
21:24:51 <talltyler> Iโ€™ll have a crack at that later, thanks for the help ๐Ÿ˜„
21:33:22 <Rubidium> peter1138[d]: I'd think about something like https://godbolt.org/z/43WzG4fv1
21:33:41 <andythenorth> might be easier in the long run to invert, and have the grf handle a 'give me a random ID''
21:34:00 <andythenorth> variants are great, but there are facets....and authors use them for weird stuff in weird ways
21:34:07 <andythenorth> I'd consider decoupling ๐Ÿ™‚
21:35:20 <andythenorth> variable 10 cb 162 might be a thing https://newgrf-specs.tt-wiki.net/wiki/Callbacks#Build_Probability_.28162.29
21:35:36 <peter1138[d]> Hmm, that's a different kind of iterator ๐Ÿ˜ฎ
21:36:30 <peter1138[d]> 'generator' file not found.
21:36:32 <peter1138[d]> Hmm
21:36:42 <peter1138[d]> Oh, C++23.
21:37:05 *** keikoz has quit IRC (Ping timeout: 480 seconds)
21:44:24 *** michi_cc has joined #openttd
21:44:24 <michi_cc> Rubidium: Could also use ranges::zip_view instead of the generator I think, but then that is C++23, too.
21:45:30 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
21:50:44 <talltyler> andythenorth: A callback or a property containing a list of IDs?
21:53:38 <peter1138[d]> A callback can only return one value, so it would need to be repeatedly called.
21:53:52 <peter1138[d]> (Or registers, but they're still limited)
21:58:34 <talltyler> Right, I mean the callback would select randomly and provide the ID to be built
21:59:28 <talltyler> Could be a novel way for authors to do naughty things, who knows ๐Ÿ˜„
21:59:31 <peter1138[d]> Oh, right. Yeah.
21:59:54 <peter1138[d]> Conflating it with the other "need" of "build 3 random parts"
22:00:28 <talltyler> This is not โ€œfill to lengthโ€ ๐Ÿ™‚
22:00:46 <talltyler> That can be achieved by clicking the button multiple times, I donโ€™t see the need to automate that
22:01:42 <talltyler> Besides enabling Train Whack (which is a noble goal), whatโ€™s the usecase for a separate list of possible alternatives?
22:02:31 <talltyler> The main reason Iโ€™m using variants is because players can easily see the pool of possible options using an existing GUI theyโ€™re already looking at
22:03:06 <talltyler> I would hate to see the possible alternatives listed in extra text, that would be terrible design on my part to make that necessary ๐Ÿ™‚
22:05:17 <talltyler> I suppose one argument is โ€œan item can only be in one hierarchyโ€ โ€” so you couldnโ€™t have a milk tank in Food Wagons and Tank Wagons simultaneously
22:05:22 *** kuka_lie has quit IRC (Quit: Lost terminal)
22:19:04 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
22:20:00 *** WormnestAndroid has joined #openttd
22:21:13 <peter1138[d]> terminate called after throwing an instance of 'std::bad_alloc'
22:21:13 <peter1138[d]> what(): std::bad_alloc
22:21:14 <peter1138[d]> Hmm
22:22:31 <peter1138[d]> Used firejail to limit memory.
22:22:49 <peter1138[d]> [2025-05-13 23:22:41] dbg: [misc:0] Not enough memory to allocate 2048 MiB of spritecache. Spritecache was reduced to 8 MiB.
22:22:50 <peter1138[d]> Well.
23:17:14 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
23:26:43 *** Flygon has joined #openttd
23:27:21 *** WormnestAndroid has joined #openttd
23:29:16 <frosch123> talltyler: train alterations on build are complicated. You always have to also consider what clone vehicle, renew vehicle, and replace vehicle shall do. Random vehicle flipping is already slightly broken on that
23:29:44 <talltyler> Yes, I discovered that ๐Ÿ™‚
23:30:04 <talltyler> I have stopped writing code, design needs to happen first ๐Ÿ˜„
23:30:22 <frosch123> Consist templates or whatever they are called may fix all of that though
23:31:07 <talltyler> Template replace? I've used it in JGRPP, it's not a magic bullet ๐Ÿ™‚