⏴ go to previous day
00:36:34 *** WormnestAndroid has quit IRC (Remote host closed the connection)
00:36:50 *** WormnestAndroid has joined #openttd
00:37:07 *** WormnestAndroid has quit IRC (Remote host closed the connection)
00:37:26 *** WormnestAndroid has joined #openttd
00:51:19 *** MinchinWeb[m] has quit IRC (Ping timeout: 480 seconds)
00:51:53 *** MinchinWeb[m] has joined #openttd
01:28:17 *** Flygon_ has quit IRC (Read error: Connection reset by peer)
01:42:51 *** willowj549 has joined #openttd
01:46:53 *** MinchinWeb[m] has quit IRC (Read error: Connection reset by peer)
01:48:52 <willowj549> using the radeon driver but openttd is looking for radeonsi, which is making hardware acceleration not work. OPENTTD 15.1
01:49:51 *** MinchinWeb[m] has joined #openttd
02:00:30 *** Wormnest has quit IRC (Quit: Leaving)
02:03:57 *** herms2 has quit IRC (Quit: bye)
02:04:51 *** herms2 has joined #openttd
02:40:03 *** tokai|noir has joined #openttd
02:40:03 *** ChanServ sets mode: +v tokai|noir
02:47:05 *** tokai has quit IRC (Ping timeout: 480 seconds)
03:01:36 *** Phileman has joined #openttd
03:09:15 *** Philemon has quit IRC (Ping timeout: 480 seconds)
03:10:55 *** MinchinWeb[m] has quit IRC (Ping timeout: 480 seconds)
03:11:45 *** MinchinWeb[m] has joined #openttd
03:52:26 *** willowj549 has quit IRC (Quit: Konversation terminated!)
03:59:09 *** Philemon has joined #openttd
04:00:14 *** MinchinWeb[m] has quit IRC (Read error: Connection reset by peer)
04:01:41 *** MinchinWeb[m] has joined #openttd
04:06:45 *** Phileman has quit IRC (Ping timeout: 480 seconds)
04:06:56 *** willowj549 has joined #openttd
04:35:07 *** willowj549 has quit IRC (Quit: Konversation terminated!)
05:18:45 <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/3f4984ebaae6dda97b7780dc66291ce543b7bfd3
05:18:46 <DorpsGek> - Update: Translations from eints (by translators)
05:37:51 *** rutuple has joined #openttd
05:44:03 *** rutuple_ has joined #openttd
05:45:55 *** rutuple has quit IRC (Ping timeout: 480 seconds)
05:57:48 *** urdh has quit IRC (Quit: Boom!)
05:58:15 *** urdh has joined #openttd
06:08:05 <DorpsGek> [OpenTTD/catcodec] PeterN merged pull request #7: Fix b417099: sample data is allowed to be empty https://github.com/OpenTTD/catcodec/pull/7
06:57:47 <peter1138> Hmm, now, how to make it compatible with the DOS version of sample.cat
07:00:30 <peter1138> Oh, the existing new_format parameter should suffice.
07:17:46 <peter1138> No, it doesn't.
07:26:24 <DorpsGek> [OpenTTD/catcodec] PeterN opened pull request #8: Codechange: remove unnecessary sample_size member https://github.com/OpenTTD/catcodec/pull/8
07:40:46 <peter1138> Hmm, maybe I should not remove the RIFF header handling. Without that the data from original sample.cats is kinda broken :/
07:51:44 <DorpsGek> [OpenTTD/catcodec] LordAro approved pull request #8: Codechange: remove unnecessary sample_size member https://github.com/OpenTTD/catcodec/pull/8#pullrequestreview-4111738221
07:56:48 <DorpsGek> [OpenTTD/catcodec] PeterN merged pull request #8: Codechange: remove unnecessary sample_size member https://github.com/OpenTTD/catcodec/pull/8
07:58:25 <peter1138> #8 :o
08:03:22 <DorpsGek> [OpenTTD/OpenTTD] zephyris commented on pull request #15391: Add: Setting to disallow train magic flip, and reverse at reduced speed https://github.com/OpenTTD/OpenTTD/pull/15391#issuecomment-4250283469
08:31:05 <DorpsGek> [OpenTTD/OpenTTD] orudge updated pull request #15483: Add: Support automatic detection of data from Atari's Transport Tycoon Deluxe re-release https://github.com/OpenTTD/OpenTTD/pull/15483
08:39:01 * peter1138 watches trains crawling backwards.
08:49:55 <reldred> It’s fun, I’m finding myself being really challenged again with network design
08:50:33 <reldred> Now early game passengers isn’t just an easy money maker it requires some genuine stuffing around until you unlock some MU’s
08:56:30 <_zephyris> Haven't managed to break that yet
09:06:00 <peter1138> HSTs carrying cargo though...
09:08:57 <peter1138> Can we have a conditional order to test the reversing flag?
09:09:10 <peter1138> That would allow sending to a waypoint to reverse, only if it's reversing.
09:11:44 <peter1138> Though I think the conditional order is probably evaluated before leaving a station, so the flag will be unknown.
09:13:47 <peter1138> Maximum Speed could almost handle it, but not quite (it reads the real maximum)
09:37:00 <reldred> Conditionals would be great
09:37:18 <reldred> Could avoid some footguns in scenarios with depots etc
09:44:52 *** rutuple_ has quit IRC (Quit: rutuple_)
10:04:25 <talltyler> I’ve been thinking about conditional orders. But in my tests of the new pathfinder penalty, having a station that lets trains leave forwards in either direction cleans up any mistakes quite nicely.
10:07:23 <peter1138> :)
10:07:43 <peter1138> I've got a PoC here anyway, so you don't need to implement it.
10:08:36 <peter1138> (It is fiddly to use)
10:11:49 <reldred> https://cdn.discordapp.com/attachments/1008473233844097104/1493916749085675644/image.png?ex=69e0b5e4&is=69df6464&hm=3d5da7cd57a549f770bcd456fd8d8bc4e0a504ca396ecfe578d0a1e74df9cfae&
10:11:49 <reldred> I have this janky little contraption I wanted it for earlier today, but, that being said, jgrpp drive through depots that were magically flipping the train, and that was an old save with an incredibly cursed terminus station, my idea being that I would only use the conditional to flip the train if it hasn't gone through the depot and gotten magic flipped.
10:28:45 *** WormnestAndroid has quit IRC (Remote host closed the connection)
10:28:58 *** WormnestAndroid has joined #openttd
10:29:02 *** WormnestAndroid has quit IRC (Remote host closed the connection)
10:29:14 *** WormnestAndroid has joined #openttd
10:29:16 *** WormnestAndroid has quit IRC (Remote host closed the connection)
10:29:39 *** WormnestAndroid has joined #openttd
10:56:16 *** Flygon has joined #openttd
11:08:34 <DorpsGek> [OpenTTD/OpenTTD] zephyris requested changes for pull request #15391: Add: Setting to disallow train magic flip, and reverse at reduced speed https://github.com/OpenTTD/OpenTTD/pull/15391#pullrequestreview-4112975439
11:09:54 <_zephyris> Conditionals would be nice for handling a mix of single and double headed trains which have shared orders - eg. only single headed travelling in reverse would go to the reversing siding.
11:10:06 <_zephyris> But, future wishes...
11:10:46 <_zephyris> I'm very close to a yolo approve... It's fun to play with, and probly just need to unleash it on players for people to find bugs/corner cases.
11:13:00 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #15391: Add: Setting to disallow train magic flip, and reverse at reduced speed https://github.com/OpenTTD/OpenTTD/pull/15391#pullrequestreview-4113007135
11:25:12 <DorpsGek> [OpenTTD/OpenGFX2] zephyris approved pull request #249: Feature: Add bell sound effect to trams https://github.com/OpenTTD/OpenGFX2/pull/249#pullrequestreview-4113073074
11:25:32 <DorpsGek> [OpenTTD/OpenGFX2] zephyris merged pull request #249: Feature: Add bell sound effect to trams https://github.com/OpenTTD/OpenGFX2/pull/249
11:25:35 <DorpsGek> [OpenTTD/OpenGFX2] zephyris closed issue #248: Sound effect for the trams NewGRF https://github.com/OpenTTD/OpenGFX2/issues/248
11:29:05 <DorpsGek> [OpenTTD/OpenGFX2] zephyris dismissed a review for pull request #232: Feature: 4x zoom coal mine https://github.com/OpenTTD/OpenGFX2/pull/232#pullrequestreview-3814620419
11:29:08 <DorpsGek> [OpenTTD/OpenGFX2] zephyris updated pull request #232: Feature: 4x zoom coal mine https://github.com/OpenTTD/OpenGFX2/pull/232
11:31:02 <talltyler> Thanks, will update that helptext later today.
11:32:19 <talltyler> “Terminus only” doesn’t make sense because it’s not just about stations. Thoughts on “Dead end only” versus “End of line only”? (Or other suggestions)
11:32:42 <peter1138> I don't think the wording suggestion makes sense. It assumes the setting is only about reversing at stations, but it's not.
11:32:54 <peter1138> Yeah.
11:33:08 <peter1138> End-of-line is more clear to me.
11:34:13 <peter1138> And "back up" could just be "reverse"
11:35:08 <reldred> yeah the setting on none does allow trains to reverse out of a non terminating station. fun to put a dmu on a shorter stretch of a subset of through stations on a larger line
11:35:36 <andythenorth> _should I read the reversing PRs_ 😛
11:35:41 <andythenorth> or just make assumptions?
11:35:43 <reldred> its one of those ones where at first I set it wrong every time but the more I sat and looked at it I realised that it did infact make sense if I stopped and looked at it
11:36:03 <reldred> andythenorth: it just works
11:36:09 <reldred> https://tenor.com/view/todd-howard-it-just-works-bethesda-this-all-just-works-gif-20598651
11:36:27 <andythenorth> wonder what reversing does to logic train possibilities
11:36:30 <reldred> thanks ~~todd~~ tyler
11:36:44 <reldred> andythenorth: i imagine those players just wouldn't turn it on
11:36:45 <andythenorth> it means trains could have different timing depending on direction
11:36:55 <andythenorth> which could have semantics
11:37:19 <andythenorth> it's basically an extra frequency control
11:37:34 <reldred> the only instance where the no flipping is mandatory are instances where there's no penalty to it. the penalty only comes about if you use the extra setting in 15391
11:37:39 <andythenorth> like the Turbo switchon 486
11:39:37 <andythenorth> did anyone try Backwards Horse yet?
11:39:41 <andythenorth> posted it here yesterday
11:40:47 <peter1138> My 386 had a turbo switch labelled 8 and 16. I don't actually know if the non-turbo speed was actually 8 MHz though.
11:41:09 <peter1138> 386sx that is, we're not that rich.
11:44:18 <talltyler> “Reverse” means things already, I will change “back up” to “drive backwards” to match the code.
11:45:36 <talltyler> andythenorth: I built the setup in your crash screenshot but couldn’t repro
11:46:58 <andythenorth> just change grfs mid-game between 2 branches 😛
11:47:07 <andythenorth> "surprised we don't support that"
11:56:20 <talltyler> Specifying that the rear vehicle needs a driver's cab seems to include putting an engine on the rear, whether single or double-headed, no?
11:56:43 <talltyler> Players can figure out if a vehicle has a driver's cab. It's not referring only to the NewGRF flag.
12:00:13 <andythenorth> badge? 😛
12:00:20 <andythenorth> that is a serious suggestion
12:01:23 <_glx_> Do scripts get the info ?
12:01:44 <peter1138> talltyler, the NewGRF flag is a technical detail that we should avoid mentioned in the setting text if reasonable.
12:01:51 <peter1138> -ed+ing
12:02:07 <peter1138> "driver's cab" or "driving cab"
12:04:38 <talltyler> Oh definitely, I mean I am using "driver's cab" in a generic sense, not referring to the NewGRF implementation.
12:04:53 <talltyler> No opinion on "driver's cab" versus "driving cab", open to suggestions
12:08:28 <DorpsGek> [OpenTTD/OpenGFX2] zephyris opened issue #250: Toyland town level crossings use normal climate graphics https://github.com/OpenTTD/OpenGFX2/issues/250
12:17:06 <andythenorth> IRL the limitation is actually a brake control valve usually
12:17:12 <andythenorth> and might not be 'the driver'
12:17:21 <andythenorth> conductor or guard
12:17:50 <andythenorth> not sure that helps, but leading our playerbase into 'it definitely has a cab' might be confusing
12:25:00 <talltyler> The intended use is a driver in a full cab. A conductor/guard/shunter riding the back of the train with a brake valve won't be able to travel at full speed, and cabooses and brake vans should not have the NewGRF flag set.
12:26:10 <talltyler> The advantage of having a brake valve, at least in the US, is that you don't have to provide constant communication to the driver, not any speed advantage. You're still going restricted speed.
12:27:13 <talltyler> Reversing with brake vans and cabooses is intended to be exactly the same as any other wagon: don't set the flag, and if the player disables magic flip the train will drive backwards at 20 mph.
12:30:08 <peter1138> Did ban authors from trying to implement the hacky image-replacement push-pull thing?
12:30:11 <peter1138> +we
12:30:57 <emperorjake> I don't see how that's enforceable
12:31:33 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler updated pull request #15391: Add: Setting to disallow train magic flip, and reverse at reduced speed https://github.com/OpenTTD/OpenTTD/pull/15391
12:32:05 <talltyler> We didn't ban it, but I did just break it 🙂
12:32:49 <talltyler> Now train sets which didn't try to implement it have push-pull, and the fake push-pull train sets appear to magic flip, essentially a double negative
12:33:26 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #15391: Add: Setting to disallow train magic flip, and reverse at reduced speed https://github.com/OpenTTD/OpenTTD/pull/15391#pullrequestreview-4113547873
12:34:06 <DorpsGek> [OpenTTD/nml] PeterN approved pull request #420: Add: Flag for allow unpowered wagons to lead a train when backing up (OpenTTD #15379) https://github.com/OpenTTD/nml/pull/420#pullrequestreview-4113552210
12:35:04 <DorpsGek> [OpenTTD/nml] 2TallTyler merged pull request #420: Add: Flag for allow unpowered wagons to lead a train when backing up (OpenTTD #15379) https://github.com/OpenTTD/nml/pull/420
12:37:04 <emperorjake> Well, that means we can remove a lot of code from JP+ MU
12:37:14 <emperorjake> Too bad about older, unmaintained sets though
12:38:31 <peter1138> Do you have the word "ban" on highlight or something? :)
12:39:18 <peter1138> I'm pretty sure it's obvious that such a comment is not really suggesting bans.
12:41:34 <talltyler> Might be worth also mentioning the variable (not yet merged) for GRFs to read if the train is currently driving backwards, for anyone who wants to get fancy with headlights/tail lights/pantographs: https://github.com/OpenTTD/nml/pull/421
12:41:59 <DorpsGek> [OpenTTD/nml] PeterN commented on pull request #420: Add: Flag for allow unpowered wagons to lead a train when backing up (OpenTTD #15379) https://github.com/OpenTTD/nml/pull/420#issuecomment-4252062313
12:42:28 <DorpsGek> [OpenTTD/nml] PeterN approved pull request #421: Add: Variable for when a train is driving backwards https://github.com/OpenTTD/nml/pull/421#pullrequestreview-4113605571
12:42:42 <peter1138>
12:44:30 <peter1138> In before "we'll keep doing it the old way because we prefer it" and "the devs have forced this on us"
12:44:46 <LordAro> :o
12:44:49 <peter1138> talltyler, now mentions 421 :)
12:46:04 <andythenorth> well
12:46:16 <andythenorth> I could remove the flag from Horse brake vans etc
12:46:28 <peter1138> So processing the order list before a train starts moving is sensible, and required, but means that the "is driving backards" conditional order is tricky to use.
12:46:42 <DorpsGek> [OpenTTD/nml] 2TallTyler merged pull request #421: Add: Variable for when a train is driving backwards https://github.com/OpenTTD/nml/pull/421
12:47:04 <peter1138> Do you want (any) trains to be able to drive backwards at full speed with those brake vans attached?
12:47:26 <andythenorth> I don't have strong opinions
12:48:00 <talltyler> I have a strong opinion that I don’t want to be sitting in a brake van at the front of a train doing 60 mph 😛
12:48:11 <LordAro> coward
12:48:45 <talltyler> I will update GRF specs today with the new additions. Thanks for the reviews. 🙂
12:50:09 <talltyler> I see the “driving backwards” conditional orders as most helpful when you get to a waypoint order. But I will probably fix rogue backwards trains in my own games using loops and the pathfinder penalty to let them sort themselves out.
12:50:23 <andythenorth> Horse No-Change Edition
12:50:32 <andythenorth> well. lunch?
12:53:56 <DorpsGek> [OpenTTD/OpenTTD] orudge updated pull request #15483: Add: Support automatic detection of data from Atari's Transport Tycoon Deluxe re-release https://github.com/OpenTTD/OpenTTD/pull/15483
12:57:27 <DorpsGek> [OpenTTD/OpenTTD] orudge updated pull request #15483: Add: Support automatic detection of data from Atari's Transport Tycoon Deluxe re-release https://github.com/OpenTTD/OpenTTD/pull/15483
12:58:51 <DorpsGek> [OpenTTD/nml] glx22 updated pull request #414: Update: changelog for 0.9.0 https://github.com/OpenTTD/nml/pull/414
13:03:00 <DorpsGek> [OpenTTD/OpenTTD] orudge commented on pull request #15483: Add: Support automatic detection of data from Atari's Transport Tycoon Deluxe re-release https://github.com/OpenTTD/OpenTTD/pull/15483#issuecomment-4252285803
13:05:26 <DorpsGek> [OpenTTD/nml] glx22 updated pull request #414: Update: changelog for 0.9.0 https://github.com/OpenTTD/nml/pull/414
13:21:00 <will_marshall_> everybody hates NML PR 419 💔
13:24:46 <DorpsGek> [OpenTTD/nml] PeterN commented on pull request #414: Update: changelog for 0.9.0 https://github.com/OpenTTD/nml/pull/414#pullrequestreview-4113932115
13:25:59 <DorpsGek> [OpenTTD/nml] PeterN approved pull request #422: Change: distinguish perimeter from area in station distributed cargo flag. https://github.com/OpenTTD/nml/pull/422#pullrequestreview-4113941343
13:31:00 <peter1138> I think the issue with 419 might be exposing something (more easily) that might be desyncy, not really checked yet.
13:41:25 *** Wormnest has joined #openttd
13:43:10 <will_marshall_> Right, I follow. I can audit OpenTTD at least to see how those variables are used.
13:43:42 <LordAro> i'm not sure the words 'audit' and 'OpenTTD' have ever been used together before
13:44:24 <will_marshall_> I will go and sniff at all the places `gcache.cached_power` is modified and apply my big brain to see if they're cool or screwy.
13:45:29 <will_marshall_> I suspect that anywhere that cached_power was desyncing presently would cause a complete desync though - given that var is used in the train movement calculation?
13:45:55 *** MinchinWeb[m] has quit IRC (Ping timeout: 480 seconds)
13:46:16 <DorpsGek> [OpenTTD/nml] 2TallTyler commented on pull request #420: Add: Flag for allow unpowered wagons to lead a train when backing up (OpenTTD #15379) https://github.com/OpenTTD/nml/pull/420#issuecomment-4252591968
13:46:21 <will_marshall_> Unless the calc for acceleration not using the cached value, which I'll find out later on.
13:47:04 *** MinchinWeb[m] has joined #openttd
13:49:52 <peter1138> It's not so much "using the var" that's the issue, rather "using the var to change some other property", given all the shinanigans that NewGRF allows.
13:53:06 <DorpsGek> [OpenTTD/nml] 2TallTyler commented on pull request #421: Add: Variable for when a train is driving backwards https://github.com/OpenTTD/nml/pull/421#issuecomment-4252639450
13:54:13 <will_marshall_> Does NML conventionally provide a guarantee that using it in abnormal ways will only produce non-breaking/non-desyncing NewGRFs? Not trying to be smart here.
13:55:30 <will_marshall_> Admittedly making an engine whose power depends on the power of the rest of the consist would be peak BAD FEATURE but would it break things?
13:56:32 <andythenorth> nmlc doesn't enforce much
13:57:30 <andythenorth> cb 36 is only available for certain contexts / properties
13:57:54 <andythenorth> varying power by consist power is a non-starter though
13:57:55 <talltyler> You can create infinite loops and use invalid cargos in industry production code, but OpenTTD warns you pretty loudly about that.
13:58:18 <talltyler> Desyncs are less likely to be caught by GRF authors during testing.
13:59:55 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #15379: Feature: Trains with an engine on the rear drive backwards when reversing https://github.com/OpenTTD/OpenTTD/pull/15379#issuecomment-4252688382
14:15:05 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler updated pull request #13265: Change: Convert Magic Bulldozer to settings https://github.com/OpenTTD/OpenTTD/pull/13265
14:18:49 <DorpsGek> [OpenTTD/nml] 2TallTyler merged pull request #422: Change: distinguish perimeter from area in station distributed cargo flag. https://github.com/OpenTTD/nml/pull/422
14:19:26 <DorpsGek> [OpenTTD/nml] 2TallTyler commented on pull request #422: Change: distinguish perimeter from area in station distributed cargo flag. https://github.com/OpenTTD/nml/pull/422#issuecomment-4252843050
14:19:47 <DorpsGek> [OpenTTD/nml] 2TallTyler commented on pull request #414: Update: changelog for 0.9.0 https://github.com/OpenTTD/nml/pull/414#issuecomment-4252845595
14:40:17 <rito12_13[d]> Do you get emailed when someone change nml docs?
14:42:30 <talltyler> Me? Nope.
14:42:51 <andythenorth> I do for some pages
14:46:10 <peter1138> If I've edited it in the past, yeah.
14:51:56 <DorpsGek> [OpenTTD/nml] Rito13 commented on pull request #422: Change: distinguish perimeter from area in station distributed cargo flag. https://github.com/OpenTTD/nml/pull/422#issuecomment-4253083922
14:57:49 *** gerliczkowalczuk has quit IRC (Quit: User went offline on Discord a while ago)
15:27:10 *** Philemon is now known as Phileman
15:27:39 *** Phileman is now known as Philemon
16:14:33 *** Flygon has quit IRC (Read error: Connection reset by peer)
16:27:07 <peter1138> Should we continue fixing the broken metadata in DOS/Windows sample.cat?
16:41:49 *** merni has joined #openttd
16:41:49 <merni> Has there been a change that you can only see PRs when you're signed in?
16:42:03 <merni> I'm getting 401 whenever I try to view a PR
16:43:03 <talltyler> Interesting, I see the same when signed out. I assume it's a GitHub issue, not an intentional change.
16:45:03 *** Wolf01 has joined #openttd
16:51:38 <merni> yeah... seems to be the same across repositories, so either a nonsensical unannounced change by github or something broke
16:51:54 <merni> Can't search while signed out either
17:08:59 *** gelignite has joined #openttd
17:12:50 <audigex> will_marshall_: No, you can break all kinds of things with NML and NMLC will quite happily accept most of it
18:14:35 *** toktik is now known as Guest7344
18:14:42 *** toktik has joined #openttd
18:15:15 *** Guest7344 has quit IRC (Ping timeout: 480 seconds)
18:35:52 <brickblock19280> emperorjake: You could remove the variable they use for it
18:37:03 <brickblock19280> talltyler: Infinite loops were funny, I once made a grf that never stopped loading
18:37:52 <squirejames> "'m not sure the words 'audit' and 'OpenTTD' have ever been used together before"
18:37:52 <squirejames> They're coming to make sure you're paying tax on all those thousands of sweet, sweet Atari bucks that made you all millionaires, apparently 😄 (according to the nutters on Reddit and Steam)
18:38:09 <DorpsGek> [OpenTTD/catcodec] PeterN opened pull request #9: Fix: catcodec could not decode sample.cat from DOS versions of original data https://github.com/OpenTTD/catcodec/pull/9
18:38:40 <peter1138> 19:35 < brickblock19280> emperorjake: You could remove the variable they use for it
18:38:44 <peter1138> Not a bad idea... ;)
18:38:55 <peter1138> (It probably has other uses)
18:39:26 <brickblock19280> I can't think of any tbh, all others would be really weird
18:40:05 <brickblock19280> I might make a PR to fix the double flipping tho as that's easily dealt with
18:44:00 <DorpsGek> [OpenTTD/catcodec] PeterN updated pull request #9: Fix: catcodec could not decode sample.cat from DOS versions of original data https://github.com/OpenTTD/catcodec/pull/9
18:49:41 <peter1138> Urgh.
18:50:19 <LordAro> heh
18:50:23 <DorpsGek> [OpenTTD/catcodec] PeterN updated pull request #9: Fix: catcodec could not decode sample.cat from DOS versions of original data https://github.com/OpenTTD/catcodec/pull/9
18:50:26 <peter1138> Silly 32bit stuff instead of size_t.
18:50:34 <peter1138> Let's just hope you don't have a 4GB+ sample.cat...
18:51:44 <peter1138> Oh, std::format, of course.
18:53:10 <peter1138> Supposed to be in C++20 but in practice, isn't.
18:54:35 <DorpsGek> [OpenTTD/catcodec] PeterN updated pull request #9: Fix: catcodec could not decode sample.cat from DOS versions of original data https://github.com/OpenTTD/catcodec/pull/9
18:54:38 <peter1138> Sod it, no leading zeroes.
19:02:33 <andythenorth> Could we let players add badges to tiles? 👀
19:03:12 <andythenorth> Considering this in grf https://discord.com/channels/142724111502802944/337701432230805505/1493999306208710879
19:03:44 <andythenorth> https://www.youtube.com/watch?v=qYbReGWJXHw
19:10:51 <andythenorth> Also mariokart speed up arrows :p
19:11:06 <andythenorth> And a whistle board
19:14:25 <peter1138> Typical Hornby. Flat out, or not moving at all.
19:27:57 <DorpsGek> [OpenTTD/catcodec] PeterN updated pull request #3: Change: Allow embedding any file into a .cat file. https://github.com/OpenTTD/catcodec/pull/3
19:29:15 <DorpsGek> [OpenTTD/catcodec] PeterN commented on pull request #3: Change: Allow embedding any file into a .cat file. https://github.com/OpenTTD/catcodec/pull/3#issuecomment-4254861320
19:30:26 <DorpsGek> [OpenTTD/catcodec] PeterN updated pull request #3: Change: Allow embedding any file into a .cat file. https://github.com/OpenTTD/catcodec/pull/3
19:30:48 <DorpsGek> [OpenTTD/catcodec] rubidium42 approved pull request #9: Fix: catcodec could not decode sample.cat from DOS versions of original data https://github.com/OpenTTD/catcodec/pull/9#pullrequestreview-4116286732
19:35:09 <DorpsGek> [OpenTTD/catcodec] PeterN commented on pull request #9: Fix: catcodec could not decode sample.cat from DOS versions of original data https://github.com/OpenTTD/catcodec/pull/9#pullrequestreview-4116311324
19:44:24 <andythenorth> we should have Hornby option for reversing trains
19:44:29 <andythenorth> flat out or not at all
19:44:37 * andythenorth writing selenium tests for work
19:44:39 <andythenorth> oof
19:49:26 <peter1138> At quarter to 9?
19:49:41 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #15484: Add: conditional order to test if a train is driving backwards https://github.com/OpenTTD/OpenTTD/pull/15484
19:52:08 <DorpsGek> [OpenTTD/OpenTTD] Brickblock1 opened pull request #15485: Change: [NewGRF] xor reversed and driving backwards flags for modflags bit 8 https://github.com/OpenTTD/OpenTTD/pull/15485
19:54:51 <squirejames> andythenorth: The "Smokey Joe" parameter
19:57:58 <DorpsGek> [OpenTTD/catcodec] rubidium42 approved pull request #3: Change: Allow embedding any file into a .cat file. https://github.com/OpenTTD/catcodec/pull/3#pullrequestreview-4116441021
19:58:36 <peter1138> talltyler, you should convince them to run master, what with nice new features... ;-)
19:58:57 <peter1138> s/you/we/
19:59:45 <talltyler> I thought about it!
20:00:30 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #15484: Add: conditional order to test if a train is driving backwards https://github.com/OpenTTD/OpenTTD/pull/15484
20:05:52 *** gelignite has quit IRC ()
20:13:31 <DorpsGek> [OpenTTD/OpenTTD] zephyris approved pull request #15391: Add: Setting to disallow train magic flip, and reverse at reduced speed https://github.com/OpenTTD/OpenTTD/pull/15391#pullrequestreview-4116523516
20:13:40 <DorpsGek> [OpenTTD/catcodec] PeterN merged pull request #9: Fix: catcodec could not decode sample.cat from DOS versions of original data https://github.com/OpenTTD/catcodec/pull/9
20:13:58 <DorpsGek> [OpenTTD/catcodec] PeterN dismissed a review for pull request #3: Change: Allow embedding any file into a .cat file. https://github.com/OpenTTD/catcodec/pull/3#pullrequestreview-4116441021
20:14:01 <DorpsGek> [OpenTTD/catcodec] PeterN updated pull request #3: Change: Allow embedding any file into a .cat file. https://github.com/OpenTTD/catcodec/pull/3
20:14:54 <peter1138> The merge-base changed after approval.
20:14:57 <peter1138> Hmm, not seen that before.
20:19:12 <DorpsGek> [OpenTTD/nml] glx22 updated pull request #414: Update: changelog for 0.9.0 https://github.com/OpenTTD/nml/pull/414
20:21:11 <peter1138> I guess I need a tree resolver.
20:23:49 <DorpsGek> [OpenTTD/OpenTTD] carlhannes opened issue #15486: [Crash]: Loading town data in scenario editor in Counter clockwise crashes the game https://github.com/OpenTTD/OpenTTD/issues/15486
20:24:30 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler merged pull request #15391: Add: Setting to disallow train magic flip, and reverse at reduced speed https://github.com/OpenTTD/OpenTTD/pull/15391
20:24:37 <peter1138> Did I put the kettle on?
20:27:07 <peter1138> I did.
20:32:14 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #15487: Codechange: make GrfSpecFeature a scoped enum https://github.com/OpenTTD/OpenTTD/pull/15487
20:48:17 <DorpsGek> [OpenTTD/catcodec] rubidium42 approved pull request #3: Change: Allow embedding any file into a .cat file. https://github.com/OpenTTD/catcodec/pull/3#pullrequestreview-4116711619
20:49:43 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #15488: Codechange: make PaletteType a scoped enum https://github.com/OpenTTD/OpenTTD/pull/15488
20:50:12 <DorpsGek> [OpenTTD/OpenTTD] mattnotmitt opened pull request #15489: Change: Most used rail type takes ownership into account https://github.com/OpenTTD/OpenTTD/pull/15489
20:51:10 <peter1138> Turns out graphs are useful after all, huh
20:56:02 <DorpsGek> [OpenTTD/catcodec] PeterN merged pull request #3: Change: Allow embedding any file into a .cat file. https://github.com/OpenTTD/catcodec/pull/3
20:57:44 <peter1138> Hm
20:57:57 <peter1138> I wonder if the release workflow works :D
21:05:58 <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #15488: Codechange: make PaletteType a scoped enum https://github.com/OpenTTD/OpenTTD/pull/15488#pullrequestreview-4116814025
21:21:12 *** Flygon has joined #openttd
21:26:57 <DorpsGek> [OpenTTD/OpenTTD] zephyris updated pull request #15206: Change: Fix the UI workflow for entering the scenario editor https://github.com/OpenTTD/OpenTTD/pull/15206
21:32:28 <_glx_> peter1138: quickly looked at the workflow, should be fine if it builds
21:33:45 <_glx_> hmm issue could be `runs-on: ubuntu-20.04`
21:34:29 <_glx_> this image is not valid anymore
21:41:51 <DorpsGek> [OpenTTD/catcodec] PeterN opened pull request #10: Docs: convert changelog to markdown https://github.com/OpenTTD/catcodec/pull/10
21:42:36 <DorpsGek> [OpenTTD/catcodec] PeterN updated pull request #10: Doc: convert changelog to markdown https://github.com/OpenTTD/catcodec/pull/10
21:42:40 <peter1138> Pom-te-pom.
21:43:37 <DorpsGek> [OpenTTD/nml] 2TallTyler commented on pull request #414: Update: changelog for 0.9.0 https://github.com/OpenTTD/nml/pull/414#pullrequestreview-4117021330
21:45:19 <peter1138> I suppose I need to implement the rating graph then.
21:48:35 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #15488: Codechange: make PaletteType a scoped enum https://github.com/OpenTTD/OpenTTD/pull/15488
21:53:57 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
21:55:44 <DorpsGek> [OpenTTD/nml] glx22 updated pull request #414: Update: changelog for 0.9.0 https://github.com/OpenTTD/nml/pull/414
21:56:10 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #15481: Codechange: make Rail/RoadTypeSpriteGroups scoped enums https://github.com/OpenTTD/OpenTTD/pull/15481
21:56:13 <DorpsGek> [OpenTTD/OpenTTD] PeterN dismissed a review for pull request #15481: Codechange: make Rail/RoadTypeSpriteGroups scoped enums https://github.com/OpenTTD/OpenTTD/pull/15481#pullrequestreview-4109176970
22:01:07 <peter1138> > Major upgrade because the minimum python version changed.
22:01:18 <peter1138> Hmm, semver would suggest that would mean 1.0.0
22:07:31 <DorpsGek> [OpenTTD/nml] 2TallTyler approved pull request #414: Update: changelog for 0.9.0 https://github.com/OpenTTD/nml/pull/414#pullrequestreview-4117151742
22:10:17 <_glx_> yeah at some point nml should be considered stable enough to be 1.0.0
22:10:53 <DorpsGek> [OpenTTD/nml] glx22 merged pull request #414: Update: changelog for 0.9.0 https://github.com/OpenTTD/nml/pull/414
22:12:00 <peter1138> \o/
22:12:04 <DorpsGek> [OpenTTD/nml] glx22 created new tag: 0.9.0 https://github.com/OpenTTD/nml/releases/tag/0.9.0
22:12:37 <_glx_> hoping it still works 🙂
22:13:59 <_glx_> well `The configuration 'macos-13-us-default' is not supported`, let's hope it won't block other jobs
22:19:17 <_glx_> and of course it prevents the upload
22:20:13 <_glx_> files on github are there, but pip won't get them
22:20:59 <_glx_> deleted the release, I first need to fix the workflow
22:27:13 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #15484: Add: conditional order to test if a train is driving backwards https://github.com/OpenTTD/OpenTTD/pull/15484#pullrequestreview-4117235974
22:31:10 <DorpsGek> [OpenTTD/nml] glx22 opened pull request #423: Fix: [Actions] update macos images used for release https://github.com/OpenTTD/nml/pull/423
22:45:38 <DorpsGek> [OpenTTD/nml] PeterN approved pull request #423: Fix: [Actions] update macos images used for release https://github.com/OpenTTD/nml/pull/423#pullrequestreview-4117331331
22:53:53 <DorpsGek> [OpenTTD/nml] glx22 merged pull request #423: Fix: [Actions] update macos images used for release https://github.com/OpenTTD/nml/pull/423
22:54:45 <DorpsGek> [OpenTTD/nml] glx22 created new tag: 0.9.0 https://github.com/OpenTTD/nml/releases/tag/0.9.0
23:02:00 <DorpsGek> [OpenTTD/catcodec] glx22 opened pull request #11: Codechange: Use ubuntu-latest in release workflow https://github.com/OpenTTD/catcodec/pull/11
23:07:20 <DorpsGek> [OpenTTD/catcodec] PeterN approved pull request #11: Codechange: Use ubuntu-latest in release workflow https://github.com/OpenTTD/catcodec/pull/11#pullrequestreview-4117423702
23:07:53 <DorpsGek> [OpenTTD/catcodec] glx22 approved pull request #10: Doc: convert changelog to markdown https://github.com/OpenTTD/catcodec/pull/10#pullrequestreview-4117426141
23:13:26 <DorpsGek> [OpenTTD/catcodec] PeterN merged pull request #10: Doc: convert changelog to markdown https://github.com/OpenTTD/catcodec/pull/10
23:14:26 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #15484: Add: conditional order to test if a train is driving backwards https://github.com/OpenTTD/OpenTTD/pull/15484
23:14:29 <DorpsGek> [OpenTTD/catcodec] glx22 merged pull request #11: Codechange: Use ubuntu-latest in release workflow https://github.com/OpenTTD/catcodec/pull/11
23:15:01 *** MinchinWeb[m] has quit IRC (Ping timeout: 480 seconds)
23:15:27 *** MinchinWeb[m] has joined #openttd
23:27:35 *** MinchinWeb[m] has quit IRC (Ping timeout: 480 seconds)
23:27:53 *** MinchinWeb[m] has joined #openttd
23:38:08 <DorpsGek> [OpenTTD/OpenTTD] JGRennison opened pull request #15490: Fix: Coordinate transformation for JSON town placement when anticlockwise https://github.com/OpenTTD/OpenTTD/pull/15490
23:40:27 <reldred> _glx_: when it supports bridges 😛
23:40:55 <_glx_> oh yeah there's still missing features
23:43:28 <DorpsGek> [OpenTTD/OpenTTD] ZanderBrown updated pull request #15466: Update types used in the Squirrel helpers https://github.com/OpenTTD/OpenTTD/pull/15466
23:47:10 *** MinchinWeb[m] has quit IRC (Read error: Connection reset by peer)
23:47:26 *** MinchinWeb[m] has joined #openttd
23:59:36 *** MinchinWeb[m] has quit IRC (Ping timeout: 480 seconds)
continue to next day ⏵
