⏴ go to previous day
00:18:48 <_glx_> so using nuget for vcpkg caching (recommended way since <https://github.com/microsoft/vcpkg-tool/pull/1662>) seems to work with <https://github.com/glx22/OpenTTD/pull/35/files>
00:33:26 *** dh1 has quit IRC (Quit: My Mac has gone to sleep. ZZZzzz…)
01:00:01 *** dh1 has joined #openttd
02:03:04 *** herms has quit IRC (Quit: bye)
02:04:05 *** herms has joined #openttd
02:07:52 *** gnu_jj_ has joined #openttd
02:11:09 *** gnu_jj has quit IRC (Ping timeout: 480 seconds)
02:23:31 *** Wormnest has quit IRC (Quit: Leaving)
04:17:14 *** keikoz has joined #openttd
04:20:36 *** dh1 has quit IRC (Quit: My Mac has gone to sleep. ZZZzzz…)
04:45:35 <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/8de32c45094431f5f64f6eefbd4e3061a9a9a074
04:45:36 <DorpsGek> - Update: Translations from eints (by translators)
04:58:42 <locosage> truebrain: In a way, they're still not in the game.
04:59:11 <locosage> Cos the cost of having them in MP is a bit to much 😜
05:00:54 *** reldred has joined #openttd
05:00:54 <reldred> wait did someone seriously raise a PR to remove trees
05:05:11 *** dh1 has joined #openttd
05:10:08 <yiffgirl> yiffgirl: example number several trillion of "why you always commit if it gets you closer even if unfinished": after trying to clean up things before commiting, you can now place bridges over stations... but you can't place stations under bridges. and i didn't commit the version that let you place stations under bridges. and i don't remember what i changed. augh
05:12:58 <yiffgirl> i don't understand the vast majority of this code.
05:45:38 <peter1138[d]> That sounds promising 🙂
05:48:47 <peter1138[d]> Rubidium: Coord perhaps?
05:54:42 <peter1138[d]> Vector... just to confuse it with std::vector which is entirely not a vector 😄
06:02:03 <DorpsGek> [OpenTTD/OpenTTD] Moth-Tolias opened pull request #14425: Feature: [NewGRF] Stations under bridges https://github.com/OpenTTD/OpenTTD/pull/14425
06:03:05 <yiffgirl> peter1138[d]: i'm so sorry for the horrors.
06:04:16 <peter1138[d]> It's okay, have a custard cream.
06:07:28 <peter1138[d]> Rubidium: One of the things that I noticed was the use of "width of sprite" and "height of sprite" begin used to refer to values which are actually x and y sub-tile coordinates. Definitely not width nor height. 🙂
06:29:07 *** Flygon has joined #openttd
06:34:16 <LordAro> yiffgirl: i might suggest that opening a PR was premature
06:35:21 <LordAro> peter1138[d]: i like the subtile consistency thing. ship it
06:37:07 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14425: Feature: [NewGRF] Stations under bridges https://github.com/OpenTTD/OpenTTD/pull/14425#pullrequestreview-3000123751
06:37:39 <peter1138[d]> Sorry but I gave up reviewing for now 🙂
06:39:11 <peter1138[d]> LordAro: Views on naming of things like PointXy, PointXyz?
06:39:32 <peter1138[d]> (Or just comment on the PR itself ;))
06:40:25 <LordAro> peter1138[d]: a bit ugly, I'd probably PointXY it
06:40:36 <LordAro> but ultimately not an issue
06:40:49 <yiffgirl> peter1138[d]: thank you ever so much for taking a look!
06:42:15 <DorpsGek> [OpenTTD/OpenTTD] github-advanced-security[bot] commented on pull request #14425: Feature: [NewGRF] Stations under bridges https://github.com/OpenTTD/OpenTTD/pull/14425#pullrequestreview-3000237298
06:43:43 <yiffgirl> LordAro: what would you have done?
06:48:42 <peter1138[d]> I think it's probably okay, it's a draft PR and without it it would be difficult to discuss and provide feedback. Nobody is going to look at a link to a commit in a branch without a PR.
06:48:42 <peter1138[d]> And it doesn't involve a 4kx4k map with 14 AIs running everything maxed out... :p
06:51:02 <peter1138[d]> Now... BridgeSpec::flags, that's a lol.
06:58:18 <peter1138[d]> LordAro: Point2D? Coordinate2D? I dunno.
07:03:48 <peter1138[d]> (Typical to focus on naming instead of the rest of it :p)
07:04:33 <andythenorth> what colour was the paint?
07:06:52 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14416: Fix #14396: Industry production graph showed zero instead of N/A. https://github.com/OpenTTD/OpenTTD/pull/14416#pullrequestreview-3000309792
07:12:31 <peter1138[d]> Hmm, whoops, `SB()` doesn't clamp nor assert the input data.
07:30:36 *** michi_cc_ has quit IRC ()
07:30:56 *** michi_cc_ has joined #openttd
07:40:05 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14416: Fix #14396: Industry production graph showed zero instead of N/A. https://github.com/OpenTTD/OpenTTD/pull/14416
07:55:28 *** ialokin has quit IRC (Quit: ZNC 1.8.2+deb3.1+deb12u1 - https://znc.in)
07:55:29 *** ialokin has joined #openttd
07:56:08 *** michi_cc_ has quit IRC ()
07:56:13 *** michi_cc_ has joined #openttd
09:23:03 <peter1138[d]> Well.
09:58:11 *** WormnestAndroid has quit IRC (Remote host closed the connection)
09:58:16 *** WormnestAndroid has joined #openttd
10:05:12 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14423: Codechange: Remove ZOOM_LVL_GUI macro. https://github.com/OpenTTD/OpenTTD/pull/14423
10:27:30 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14407: Update: Changelog for 15.0-beta3 https://github.com/OpenTTD/OpenTTD/pull/14407#issuecomment-3052092652
10:38:23 *** WormnestAndroid has quit IRC (Remote host closed the connection)
10:38:36 *** WormnestAndroid has joined #openttd
11:04:55 *** dh1 has quit IRC (Quit: My Mac has gone to sleep. ZZZzzz…)
11:22:31 <talltyler> Two suggestions were marked resolved without any change or resolution, I unresolved them for visibility (should be easily visible in Files Changed tab)
11:22:43 <talltyler> Apart from those, I think we should do beta3 soon. 🙂
11:23:09 <talltyler> I will draft a website post PR today.
11:24:11 <peter1138[d]> Hmm
11:25:50 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14407: Update: Changelog for 15.0-beta3 https://github.com/OpenTTD/OpenTTD/pull/14407
11:29:09 <talltyler> Anything from the milestone we want to get into the beta?
11:29:38 <talltyler> I have to update a couple of my PRs 🙂
11:59:00 <peter1138[d]> Phew, managed to get this query down from 5+ seconds to ~100ms
11:59:06 <peter1138[d]> Still heavy but better.
12:07:23 *** WormnestAndroid has quit IRC (Remote host closed the connection)
12:07:25 *** WormnestAndroid has joined #openttd
12:52:05 <peter1138[d]> Oops, my attempt to lock this task so it only runs one at a time failed.
12:52:16 <peter1138[d]> Because it happens so quickly each instance is able to acquire the lock :p
12:58:27 <peter1138[d]> Oh well, in this case happening too many times is not harmful, and better than not happening at all.
13:11:15 <DorpsGek> [OpenTTD/website] 2TallTyler opened pull request #357: Add: OpenTTD 15.0-beta3 announcement https://github.com/OpenTTD/website/pull/357
13:32:29 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler updated pull request #14406: Fix #14377: Make house picker window remember house protection state when closed https://github.com/OpenTTD/OpenTTD/pull/14406
13:33:22 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #14406: Fix #14377: Make house picker window remember house protection state when closed https://github.com/OpenTTD/OpenTTD/pull/14406#issuecomment-3052696498
13:35:01 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler closed issue #14101: [Bug]: Multiplayer disconnections due to desynchronization https://github.com/OpenTTD/OpenTTD/issues/14101
13:35:04 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on issue #14101: [Bug]: Multiplayer disconnections due to desynchronization https://github.com/OpenTTD/OpenTTD/issues/14101
13:35:46 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14406: Fix #14377: Make house picker window remember house protection state when closed https://github.com/OpenTTD/OpenTTD/pull/14406#pullrequestreview-3001603644
13:38:41 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler updated pull request #14406: Fix #14377: Make house picker window remember house protection state when closed https://github.com/OpenTTD/OpenTTD/pull/14406
13:38:51 <talltyler> Fixed 🙂
13:42:18 <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #14406: Fix #14377: Make house picker window remember house protection state when closed https://github.com/OpenTTD/OpenTTD/pull/14406#pullrequestreview-3001630201
13:42:42 <peter1138[d]> Hmm, it's getting hot in here.
13:43:26 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #12131: Fix: Scale graph gridlines and axes with GUI scale https://github.com/OpenTTD/OpenTTD/pull/12131#pullrequestreview-3001634287
13:53:12 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #14018: Feature: Add setting to disable vehicle intro date randomisation https://github.com/OpenTTD/OpenTTD/pull/14018#pullrequestreview-3001645366
14:18:24 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler merged pull request #14406: Fix #14377: Make house picker window remember house protection state when closed https://github.com/OpenTTD/OpenTTD/pull/14406
14:18:24 *** kuka_lie has joined #openttd
14:18:27 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler closed issue #14377: House placer should remember "prevent upgrades" choice https://github.com/OpenTTD/OpenTTD/issues/14377
14:26:46 *** akimoto has joined #openttd
14:28:13 *** gelignite has joined #openttd
14:29:39 <peter1138[d]> Ohhhhh what a lucky a man. He was.
14:55:30 *** akimoto has quit IRC (Ping timeout: 480 seconds)
14:56:49 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler updated pull request #14405: Fix #14375: When loading config, validate timekeeping mode and minutes per year https://github.com/OpenTTD/OpenTTD/pull/14405
14:57:33 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #14405: Fix #14375: When loading config, validate timekeeping mode and minutes per year https://github.com/OpenTTD/OpenTTD/pull/14405#issuecomment-3052980724
15:47:17 *** Wormnest has joined #openttd
16:37:37 <peter1138[d]> Silly LordAro, doing "work" (looking at OpenTTD PRs) while abroad on holiday 🙂
18:14:49 <_zephyris> Terrifying efficiency, thanks talltyler !
19:56:32 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14417: Change: Make truncation ellipsis translatable. https://github.com/OpenTTD/OpenTTD/pull/14417#pullrequestreview-3002895430
19:58:43 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #14416: Fix #14396: Industry production graph showed zero instead of N/A. https://github.com/OpenTTD/OpenTTD/pull/14416#pullrequestreview-3002901856
20:00:09 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14416: Fix #14396: Industry production graph showed zero instead of N/A. https://github.com/OpenTTD/OpenTTD/pull/14416#pullrequestreview-3002908507
20:02:37 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14405: Fix #14375: When loading config, validate timekeeping mode and minutes per year https://github.com/OpenTTD/OpenTTD/pull/14405#pullrequestreview-3002918247
20:04:41 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler merged pull request #14405: Fix #14375: When loading config, validate timekeeping mode and minutes per year https://github.com/OpenTTD/OpenTTD/pull/14405
20:04:44 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler closed issue #14375: [Bug]: Timekeeping mode and Minutes Per Year not validated when starting game from manually-edited config https://github.com/OpenTTD/OpenTTD/issues/14375
20:25:35 *** kuka_lie_ has joined #openttd
20:35:57 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
20:35:59 *** WormnestAndroid has joined #openttd
20:36:04 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
20:36:19 *** WormnestAndroid has joined #openttd
20:44:51 *** dh1 has joined #openttd
21:06:34 <yiffgirl> how does `MBR` work? if my understanding is correct, then it should be putting raw values as the fields of a struct, but i really don't think that's how structs work, so i must be wrong
21:09:12 <_jgr_> It's a `#define`, see further up in the file
21:12:55 <yiffgirl> is... that not how `#define`s work?
21:27:23 *** gelignite has quit IRC ()
21:28:13 <DorpsGek> [OpenTTD/OpenTTD] glx22 opened pull request #14426: Codechange: [Actions] Use NuGet for vcpkg binary caching https://github.com/OpenTTD/OpenTTD/pull/14426
21:33:15 <_glx_> and of course it doesn't work
21:38:37 <DorpsGek> [OpenTTD/OpenTTD] glx22 updated pull request #14426: Codechange: [Actions] Use NuGet for vcpkg binary caching https://github.com/OpenTTD/OpenTTD/pull/14426
21:40:31 <dwfreed> yiffgirl: one of many ways you can initialize struct values is by listing the values in the order of the struct members, exactly how the MBR define is doing it
21:41:24 <yiffgirl> oh
21:41:24 <yiffgirl> ohhhhhhh
21:42:04 <yiffgirl> okay that's the piece of information i've been missing. i had no idea that was a thing. what a bizzare language
21:43:13 <dwfreed> if you wish to be more explicit, or only partially initialize the struct, you can specify the struct member in your initializer, like this: { .foo = "bar", .bar = "foo" }
21:45:31 <yiffgirl> oh i'm real small brain. i was confusing `const BridgeSpec _orig_bridge[] = {` for a definition
21:47:28 <dwfreed> it is an initializer
21:47:36 *** tokai|noir has joined #openttd
21:47:36 *** ChanServ sets mode: +v tokai|noir
21:49:37 <DorpsGek> [OpenTTD/OpenTTD] glx22 updated pull request #14426: Codechange: [Actions] Use NuGet for vcpkg binary caching https://github.com/OpenTTD/OpenTTD/pull/14426
21:54:35 <_glx_> (I know it's the token permissions, I was just trying to enable debug output)
21:54:36 *** tokai has quit IRC (Ping timeout: 480 seconds)
21:55:47 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14417: Change: Make truncation ellipsis translatable. https://github.com/OpenTTD/OpenTTD/pull/14417
22:00:12 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #14426: Codechange: [Actions] Use NuGet for vcpkg binary caching https://github.com/OpenTTD/OpenTTD/pull/14426#issuecomment-3054213750
22:03:02 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14416: Fix #14396: Industry production graph showed zero instead of N/A. https://github.com/OpenTTD/OpenTTD/pull/14416
22:03:05 <DorpsGek> [OpenTTD/OpenTTD] PeterN closed issue #14396: [Bug]: Industry cargo graphs are initialized to all visible zeroes instead of invisible/blank values. https://github.com/OpenTTD/OpenTTD/issues/14396
22:05:19 *** keikoz has quit IRC (Ping timeout: 480 seconds)
22:07:41 <peter1138[d]> <https://www.tt-forums.net/viewtopic.php?t=92363> Oh my.
22:08:02 <peter1138[d]> "My play style is using an exploit that no longer works" 🙂
22:09:57 <reldred> Kill him
22:10:33 <reldred> ‘My play style has been to blatantly cheat’
22:11:01 *** Flygon has quit IRC (Remote host closed the connection)
22:15:01 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14321: Add: Include history of cargo delivered and waiting to industry graphs. https://github.com/OpenTTD/OpenTTD/pull/14321
22:42:32 <peter1138[d]> reldred: A bit extreme.
22:59:15 <belajalilija> well
23:01:33 <belajalilija> maybe not
23:05:52 <DorpsGek> [OpenTTD/OpenTTD] zephyris merged pull request #12131: Fix: Scale graph gridlines and axes with GUI scale https://github.com/OpenTTD/OpenTTD/pull/12131
23:15:32 <talltyler> I think the reply says it best:
23:15:32 <talltyler> > By the way, if this has been your main mode of playing for 30 years, I'm afraid you've been missing out on all the fun...
23:19:53 <talltyler> In general, I think a lot of Civ designer Soren Johnson’s quote:
23:19:53 <talltyler> > Given the opportunity, players will optimize the fun out of a game.
23:19:53 <talltyler> The article it accompanies is quite good.
23:24:15 <belajalilija> yeah it takes the fun out of stuff
23:25:03 *** dh1 has quit IRC (Ping timeout: 480 seconds)
continue to next day ⏵