IRC logs for #openttd on OFTC at 2022-11-01
            
00:15:36 *** wallabra_ has joined #openttd
00:17:17 *** wallabra has quit IRC (Ping timeout: 480 seconds)
00:17:17 *** wallabra_ is now known as wallabra
00:19:22 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
00:19:48 *** WormnestAndroid has joined #openttd
00:35:51 *** Wormnest has joined #openttd
01:16:49 <DorpsGek> [OpenTTD/OpenTTD] DefinitelyNotRau117 opened issue #10117: [Bug]: The build_object_frame_burst parameter allows building 1 fewer tiles https://github.com/OpenTTD/OpenTTD/issues/10117
01:29:02 *** Wormnest has quit IRC (Ping timeout: 480 seconds)
01:38:23 *** Flygon has joined #openttd
01:41:05 *** _aD has quit IRC (Quit: leaving)
02:06:02 *** Wormnest has joined #openttd
02:09:54 <DorpsGek> [OpenTTD/OpenTTD] DefinitelyNotRau117 opened issue #10118: [Bug]: «Сycle trough signal types» set to «Path signals only» by default https://github.com/OpenTTD/OpenTTD/issues/10118
02:36:29 *** Wormnest has quit IRC (Quit: Leaving)
02:41:17 <glx[d]> ahah
02:43:44 <EmperorJake> Wasn't that guy banned from the github?
04:00:38 <TallTyler> Yes, and his username doesn’t even try to hide it 🤨
04:02:09 *** D-HUND has joined #openttd
04:05:27 *** debdog has quit IRC (Ping timeout: 480 seconds)
05:49:54 *** sla_ro|master has joined #openttd
06:50:55 *** sla_ro|master has quit IRC ()
07:23:08 <LordAro> tbf, does feel like the signal cycling should depend on the current window size, not the setting
07:26:37 <LordAro> also hi
07:26:43 <LordAro> seems i missed some stuff yesterday :)
07:26:50 <LordAro> TallTyler: no feature freeze until RC
07:53:35 *** HerzogDeXtEr has joined #openttd
08:07:13 *** Smedles has joined #openttd
08:24:23 <DorpsGek> [OpenTTD/team] chchatzop opened issue #355: [el_GR] Translator access request https://github.com/OpenTTD/team/issues/355
08:24:26 *** gelignite has joined #openttd
08:24:34 *** Wolf01 has joined #openttd
08:48:40 *** Kitrana has joined #openttd
08:54:42 *** Kitrana1 has quit IRC (Ping timeout: 480 seconds)
09:10:12 *** Kitrana has quit IRC (Ping timeout: 480 seconds)
09:11:14 <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1036930407624626206/unknown.png
09:11:14 <petern> Back with the juicy vectors...
09:18:15 <andythenorth> renegade master
09:22:42 <reldred> D4 damager
09:42:20 *** D-HUND is now known as debdog
11:10:34 <DorpsGek> [OpenTTD/team] glx22 commented on issue #355: [el_GR] Translator access request https://github.com/OpenTTD/team/issues/355
11:16:56 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #10112: Fix #10023: Allow negative input in text fields when needed https://github.com/OpenTTD/OpenTTD/pull/10112#issuecomment-1298372304
11:19:07 *** gelignite has quit IRC (Quit: Stay safe!)
12:01:05 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
12:01:08 *** WormnestAndroid has joined #openttd
12:24:48 *** tokai|noir has joined #openttd
12:24:48 *** ChanServ sets mode: +v tokai|noir
12:31:39 *** tokai has quit IRC (Ping timeout: 480 seconds)
12:54:15 *** Merni has quit IRC (Quit: User went offline on Discord a while ago)
13:11:06 *** Kitrana has joined #openttd
14:12:33 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler opened pull request #10119: Change: Cycle through visible signals by default https://github.com/OpenTTD/OpenTTD/pull/10119
14:16:40 *** WormnestAndroid has quit IRC (Remote host closed the connection)
14:21:23 *** WormnestAndroid has joined #openttd
14:24:37 *** Kitrana has quit IRC (Ping timeout: 480 seconds)
14:24:59 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler opened pull request #10120: Fix #10117: Decrement object burst limit after build check https://github.com/OpenTTD/OpenTTD/pull/10120
14:25:40 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
14:25:47 *** WormnestAndroid has joined #openttd
14:37:01 <TallTyler> For the release notes, we might not want to phrase disabling local authority control as "rubber-stamping" — to me that sounds like a copy-and-paste feature that gets requested on a regular basis.
15:03:39 *** nielsm has joined #openttd
15:43:21 *** Flygon has quit IRC (Quit: A toaster's basically a soldering iron designed to toast bread)
16:30:24 <petern> 0Hi
16:37:16 *** wallabra has quit IRC (Ping timeout: 480 seconds)
16:37:32 <LordAro> 1Hi
16:48:43 <pickpacket> Which NewGRF is the best if I just want to add trams in general? I.e. I’m not looking for any specific trams
17:10:42 *** Merni has joined #openttd
17:10:42 <Merni> Generic Tram Set and eGRVTS (which includes lots of trams and road vehicles) are pretty good
17:27:22 *** mindlesstux has quit IRC (Ping timeout: 480 seconds)
17:27:33 *** mindlesstux has joined #openttd
17:30:06 *** gelignite has joined #openttd
17:50:14 <andythenorth> yo
17:53:51 <pickpacket> Merni: thanks!
18:05:10 *** sla_ro|master has joined #openttd
18:06:35 *** Wormnest has joined #openttd
18:14:03 <andythenorth> what shall we do today brains?
18:14:23 <petern> Make lots of little PRs?
18:14:30 <andythenorth> nice idea
18:15:37 <andythenorth> I like this small PR https://github.com/OpenTTD/OpenTTD/pull/10114
18:43:20 *** sla_ro|master has quit IRC (Read error: No route to host)
18:44:09 *** sla_ro|master has joined #openttd
19:04:11 *** wallabra has joined #openttd
19:13:37 *** rightnut_ has joined #openttd
19:14:14 *** gregdek_ has joined #openttd
19:17:37 *** rightnut has quit IRC (Ping timeout: 480 seconds)
19:17:37 *** gregdek has quit IRC (Ping timeout: 480 seconds)
19:17:37 *** rightnut_ is now known as rightnut
19:17:37 *** gregdek_ is now known as gregdek
19:24:46 <TrueBrain> GOG also now has 13.0-beta1 published 🙂 Sorry for the delay!
19:24:54 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #10114: Feature: Variable interface scaling https://github.com/OpenTTD/OpenTTD/pull/10114#issuecomment-1299005250
19:31:03 <frosch> a gog 🙂 they still send these "0 units sold" mails, but in random intervals
19:55:44 <michi_cc[d]> TallTyler: If you pinky-promise to fix the inevitable bugs, I'll glady make the beta outdated right away (the PR needs a rebase though).
19:56:42 <DorpsGek> [OpenTTD/OpenTTD] michicc approved pull request #10120: Fix #10117: Decrement object burst limit after build check https://github.com/OpenTTD/OpenTTD/pull/10120#pullrequestreview-1163962558
19:58:56 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler merged pull request #10120: Fix #10117: Decrement object burst limit after build check https://github.com/OpenTTD/OpenTTD/pull/10120
19:58:59 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler closed issue #10117: [Bug]: The build_object_frame_burst parameter allows building 1 fewer tiles https://github.com/OpenTTD/OpenTTD/issues/10117
19:59:13 <petern> andythenorth: I have ability to make it smallerable in related chunks.
20:01:47 <petern> TallTyler: is that your first merge?
20:02:03 <TallTyler> Yes, did I do it wrong?
20:02:50 <petern> No. Congrats!
20:02:58 <TrueBrain> Party like it is 1999!
20:02:59 <TrueBrain> 😄
20:03:28 <TrueBrain> one small point of improvement: please always squash if possible; that way, the PR that made the change is added to the commit message, making changelogs slightly easier 🙂
20:03:34 <TrueBrain> only rebase if there is more than one commit in the PR 🙂
20:03:41 <TallTyler> Ah, okay!
20:03:49 <michi_cc[d]> And just a FYI: PR can be merged by rebasing and by squashing. If there is more than one commit, rebase to keep.
20:03:59 <TrueBrain> sorry michi_cc[d] 😄
20:04:03 <michi_cc[d]> And of course squash allows to cheat on the commit message check 🙂
20:04:15 <TallTyler> Right
20:05:02 <petern> I still use regular merges on my work stuff, a bit verbose but seems to work okay.
20:06:14 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler updated pull request #9931: Feature: Multi-track level crossings https://github.com/OpenTTD/OpenTTD/pull/9931
20:07:06 <TallTyler> michi_cc[d]: I pinky-promise to fix any bugs 🙂
20:08:05 <DorpsGek> [OpenTTD/OpenTTD] michicc approved pull request #9931: Feature: Multi-track level crossings https://github.com/OpenTTD/OpenTTD/pull/9931#pullrequestreview-1163975850
20:10:09 <TallTyler> Just so I'm clear, obviously nobody can approve their own PRs, but after that happens, is merging your own PR generally an accepted thing? Or does that depend on the size of the PR, where you'd want two people to "turn the keys" to approve and merge?
20:11:53 <frosch> merging your own PRs is fine. but I like to keep PRs open for a few days, so everyone has a chance to comment, if they want to
20:11:57 <michi_cc[d]> We like when somebody else does the busy-work 🙂
20:12:04 <petern> Yea
20:17:15 <TallTyler> This one's been open since June, so I suppose if you didn't get your thoughts in since then, you have some time while the actions run 🙂
20:24:10 <frosch> no problem here 🙂 sometimes someone has a great idea at 2 AM, and someone else agrees at 3 AM, and at 10 AM everyone thinks WTF 🙂
20:25:02 <TallTyler> Ah, that makes sense
20:26:55 <andythenorth> or there are 7 different opinions, and then we all give up 😛
20:46:47 *** wallabra has quit IRC (Ping timeout: 480 seconds)
20:51:38 <TallTyler> #yolo
20:51:58 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler merged pull request #9931: Feature: Multi-track level crossings https://github.com/OpenTTD/OpenTTD/pull/9931
20:52:13 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler closed pull request #8444: Feature: Close adjacent level crossings as if they were one large crossing https://github.com/OpenTTD/OpenTTD/pull/8444
20:52:21 <TallTyler> Let the bug reports begin!
20:53:07 <andythenorth> \o/
20:54:40 <TallTyler> Another upstreamed JGRPP feature for players to yawn at 😛
20:55:48 *** wallabra has joined #openttd
20:55:54 <andythenorth> thanks
20:56:15 *** sla_ro|master has quit IRC ()
20:58:35 <reldred> Blz add auto seperate
20:58:55 <reldred> Then I can start playing OpenTTD without pulling my hair out
20:59:12 <TallTyler> I'd really like to
20:59:31 <reldred> Complimacated?
20:59:36 <reldred> I’d imagine it is.
20:59:44 <TallTyler> Yes, and split among a bunch of commits I believe
21:00:00 <andythenorth> but also existing auto-separate exists
21:00:15 <andythenorth> and removing things isn't allowed generally by the community
21:00:25 <andythenorth> due to the now over-used XKCD
21:01:04 <petern> You mean the existing auto-separate which is really hard to use, and hidden by ctrl probably?
21:01:16 <andythenorth> and is known to not work for ships
21:01:39 <TallTyler> Existing autoseparate doesn't keep vehicles separated, it just sets starting dates automatically based on the number of vehicles in the shared orders
21:01:47 <petern> I somehow doubt that vehicle type has any bearing on it.
21:01:51 <FLHerne> JGR's also doesn't, unless that changed quite recently
21:02:03 <FLHerne> (work for ships)
21:02:20 <TallTyler> My thought was to have timetable automation and autoseparation turned on automatically for vehicles without Full Load orders
21:02:23 <andythenorth> it's weird that it's vehicle type specific eh 🙂
21:03:03 <TallTyler> Although it wouldn't show up in the timetable window. Also if you manually timetable anything, it would disable the hidden autoseparation and automation
21:04:39 <andythenorth> autoseparate, load-through stations, profit
21:04:51 <andythenorth> not even sure I would use load-through stations, I just think it's clever
21:05:05 <andythenorth> not even sure I know how to use them 😛
21:05:14 <andythenorth> what do I do?
21:07:38 <TallTyler> I think there are some sticking points in the existing implementation though, like vehicles which can't overtake (trams and trains on some networks) getting out of order and blocking each other waiting for their timetabled departure. I'd like to see what can be done with vehicles determining the correct sequence position automatically, or maybe just having vehicles which arrive much too early, when a depot order exists, keep runni
21:07:38 <TallTyler> in the depot
21:08:24 <andythenorth> oh that frigging ffwd latch on save
21:08:35 <andythenorth> is disabling auto-save the correct fix?
21:08:58 <TallTyler> No...?
21:09:22 <TallTyler> Maybe a good bug report which explains how to consistently reproduce it would be the correct fix 😛
21:10:22 <andythenorth> touche
21:10:35 <andythenorth> I have disabled autosave for now
21:10:42 <andythenorth> my diligence is worn thin currently
21:11:01 <andythenorth> how do I do through load?
21:11:01 <andythenorth> I can't find an order for it
21:11:43 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1037111723473248326/unknown.png
21:12:27 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1037111908089729085/unknown.png
21:12:37 <petern> hmm
21:13:14 <petern> Oh, that's some other game.
21:13:14 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1037112105041678367/unknown.png
21:13:14 <andythenorth> it's not in departures either
21:13:23 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1037112142232563773/unknown.png
21:13:50 <andythenorth> must be a vehicle orders thing, probably right in front of my face, but not seeing it
21:14:39 <frosch> lol, a two-column gui 🙂
21:14:55 <frosch> the challenges steel town provides 🙂
21:15:15 <frosch> i think win3x file browser was also 2-columns ?
21:15:42 <frosch> andythenorth: just build a train longer than the platform?
21:15:50 <frosch> why would it need an explicit order?
21:16:25 <JGR> FLHerne: It does, but like everything with ships, it can be a bit slow to take effect
21:16:38 <andythenorth> frosch: we can check http://assets.metacade.com/emulators/win311vr.html
21:17:00 <petern> Oh that reminds me, the small map cargo/industry list scrolling...
21:17:10 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1037113091655217152/unknown.png
21:17:10 <andythenorth> this train doesn't through load
21:17:24 <frosch> i don't recall whether i ever started win3x in dosbox
21:17:29 <andythenorth> it just stays in place loading, slowly
21:17:36 <frosch> but my 98 vm no longer works since my last cpu upgrade
21:17:51 <frosch> it's yet another "cpu too fast for OS" bug
21:17:53 <andythenorth> Netscape 2.0!
21:17:54 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #10121: Codechange: rename and move strings to make them more consistent https://github.com/OpenTTD/OpenTTD/pull/10121
21:18:01 <andythenorth> I wonder if this VM has a web connection
21:18:06 <frosch> there are even patches on github for it, but they did not work for me
21:18:07 <JGR> andythenorth: <https://github.com/JGRennison/OpenTTD-patches/wiki/Order-window-features#through-load>
21:18:25 <andythenorth> thanks
21:18:45 <andythenorth> working now 🙂
21:19:02 <andythenorth> wonder what happens when it runs out of headshunt space
21:19:51 <frosch> oh, i better merge 10111 before 10121 breaks it :p
21:19:56 <andythenorth> oh it just leaves
21:19:57 <TallTyler> Drive off the end of the track?
21:20:05 <andythenorth> nah it just stops loading and goes to next order
21:21:17 <andythenorth> it looks nice eh
21:22:08 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #10111: Codechange: Use positional parameters in the base language either for all or for no parameters. https://github.com/OpenTTD/OpenTTD/pull/10111
21:22:21 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1037114399971885187/unknown.png
21:22:21 <andythenorth> can do this
21:22:37 <andythenorth> if stations are 1 tile now, could delete FIRS outpost layours
21:22:52 <petern> Who needs bendy stations?
21:23:15 <TallTyler> PR from Andy when?
21:23:19 <TallTyler> I'll review it 🙂
21:23:54 <andythenorth> I prefer talking about PRs than doing PRs 😛
21:23:58 <andythenorth> armchair developer
21:24:06 <andythenorth> I have commits to my name 😛
21:24:12 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #10116: Fix: Too many braces make old gcc sad. https://github.com/OpenTTD/OpenTTD/pull/10116#pullrequestreview-1164070597
21:26:11 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #10121: Codechange: rename and move strings to make them more consistent https://github.com/OpenTTD/OpenTTD/pull/10121#pullrequestreview-1164072602
21:33:15 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #10119: Change: Cycle through visible signals by default https://github.com/OpenTTD/OpenTTD/pull/10119#issuecomment-1299223877
21:45:18 *** nielsm has quit IRC (Ping timeout: 480 seconds)
21:47:10 <TallTyler> Another upstream should be NewGRF road stops 🙂
21:48:56 <frosch> is that that unfinished/unreviewe PR, or something else?
21:49:35 <frosch> https://github.com/OpenTTD/OpenTTD/pull/7955
21:49:40 <TallTyler> It's fully implemented in JGRPP
21:49:49 <TallTyler> Based on that PR, but with the missing pieces
21:49:51 <frosch> i never looked at it, roadtypes is such a red flag for me 🙂
21:51:38 <glx[d]> https://github.com/OpenTTD/OpenTTD/actions/runs/3335549609/jobs/5519644202#step:6:870 <-- I really don't understand how it's possible (the PR doesn't touch anything related to this code)
21:53:52 *** gelignite has quit IRC (Quit: Stay safe!)
21:54:02 <JGR> TallTyler: I ended up making more changes than just adding the missing bits
21:54:03 <TallTyler> I'd imagine that road stops are more similar to NewGRF stations than they are to roadtypes, but I guess I don't know 🤷
21:54:21 <JGR> The NewGRF interface in my branch does not match that in the PR
21:54:33 <petern> NewGRF stations are also a red flag.
21:55:16 <JGR> Roadstops is closest to NewGRF objects
21:55:38 <JGR> At least in terms of all the drawing/tile stuff
21:57:27 <petern> Layout specified in the action 2 chain?
21:58:13 <glx[d]> yeah using layouts like objects/industry tiles is the way to go
21:58:17 <petern> Yup.
21:58:20 <JGR> <https://newgrf-specs.tt-wiki.net/wiki/Action2/Sprite_Layout>
21:58:56 <glx[d]> stations is weird because little/lots
21:59:02 <petern> JGR, yes, I meant as opposed to the NewStations way that nothing else uses ;D
21:59:06 <glx[d]> (and predates layout)
21:59:13 <petern> Stations are weird because they predate, yes.
21:59:28 <petern> For NewGRF docks, I went with the action 2 way too.
21:59:32 <petern> I think 😄
22:14:36 <reldred> Yeah I’ve got a semi-released newgrf for roadstops, I’m not putting it on bananas yet because there’s no point IMO if it’s JGRPP only.
22:15:45 <reldred> https://github.com/reldred/re_foars
22:16:13 <reldred> The feature works well
22:16:22 <glx[d]> bananas supports JGRPP only stuff
22:16:46 <reldred> Yeah I haven't sussed that out yet
22:17:19 <reldred> but yes, jgrpp's implementation of custom road stops works well, filter just for drive through or terminating, filter for cargo or pax,
22:17:32 <reldred> I'm only doing drive through for both cargo and pax in my set though
22:18:08 <petern> So does it need extracting from jgrpp?
22:18:36 <reldred> plz do so garryg can start making fifty million .grf's for it
22:18:58 <reldred> fridaemon and I have been messing with it, but being a jgrpp only feature has limited adoption
22:19:22 <reldred> I still need to play with jgrpp's newrocks support but I need sprites...
22:20:09 <petern> Extracting is pretty hard, everything gets intertwined.
22:20:27 *** _aD has joined #openttd
22:24:02 <reldred> Yeah tell me about it. Kinda why I just stick to jgrpp any time I go playing with code. The codebases have started to diverge a bit.
22:24:06 <petern> There is a road-stops branch, it's not "clean" though.
22:24:23 <reldred> Probably not a 13.0 inclusion then 😛
22:25:06 <JGR> Pretty much nothing in my branch is going to apply cleanly on master
22:25:29 <reldred> Just ship JGRPP for 13.0, call it a day.
22:25:31 <reldred> wcgw?
22:25:39 <reldred> *ducks*
22:25:52 <petern> That undoes quite a bit.
22:27:04 <petern> It's difficult to know if there are other fixes for a feature not included in the branch.
22:27:38 <JGR> The road_stops branch in my repo is out of date, it could probably be deleted TBH
22:27:57 <petern> Exactly.
22:28:07 <JGR> Trying to keep endless feature branches up to date stopped being practical for me a long time ago
22:28:20 <reldred> Yeah there was a couple of tweaks I asked for after it went live iirc
22:28:59 <reldred> hey speaking of which, JGR did you NMLC get updated to include newstations support from trunk?
22:29:04 <petern> So extracting everything need for a feature... :/
22:29:08 <reldred> your, even
22:29:31 <JGR> Yes, there have been a few more fixes since I last merged though
22:29:43 <JGR> I can sort that out soon enough
22:30:02 <reldred> Got it, well, don't hurry on my behalf, I'm still a long way off summoning the motivation to GRF again
22:34:02 <petern> Extended NewGRF variable parameter support. No idea.
22:35:08 <JGR> I don't control the NewGRF spec and the ID spaces are all very small
22:36:20 <petern> Ah so it's jgrpp extensions, not related to say "extended bytes" 🙂
22:36:41 <JGR> <https://jgrennison.github.io/OpenTTD-patches/newgrf-additions.html>
22:43:27 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
23:01:51 *** HerzogDeXtEr has quit IRC (Read error: Connection reset by peer)
23:06:33 <petern> reldred: Basically, it's a rewrite :p
23:10:33 <reldred> 😭
23:27:09 <petern> When did we remove $Id$? Can't be that long ago since the OpenGL blitter has it...
23:32:12 <glx[d]> <https://github.com/OpenTTD/OpenTTD/commit/13cc8a0ceec90def39cbcb84135a0bf039793a6f> 3 years ago
23:38:02 <petern> Explains it, newgrf docks is 4+ years old 😢
23:50:38 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #10122: Cleanup: Remove svn `$Id$` comments. https://github.com/OpenTTD/OpenTTD/pull/10122
23:53:48 <DorpsGek> [OpenTTD/OpenTTD] michicc merged pull request #10116: Fix: Too many braces make old gcc sad. https://github.com/OpenTTD/OpenTTD/pull/10116
23:57:04 <DorpsGek> [OpenTTD/OpenTTD] michicc approved pull request #10122: Cleanup: Remove svn `$Id$` comments. https://github.com/OpenTTD/OpenTTD/pull/10122#pullrequestreview-1164201710