Config
Log for #openttd on 13th June 2023:
Times are UTC Toggle Colours
13:44:00  *** D-HUND is now known as debdog
13:44:59  *** nielsm has joined #openttd
13:54:46  *** Ttech has quit IRC
14:01:25  <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1118178316504006717/iron-horse.grf.zip
14:01:25  <andythenorth> CK2347: you can be a tester...
14:05:36  *** Ttech has joined #openttd
14:10:11  <CK2347> Thank you!
14:10:21  <CK2347> I definitely would be happy to
14:27:07  *** supermop_Home has joined #openttd
14:27:13  <supermop_Home> hello
14:33:56  <TrueBrain> it's me
14:34:09  <TrueBrain> I was wonering if after all these year you'd like to meet
14:39:30  * LordAro hands TrueBrain some cake
14:44:11  <TrueBrain> nom nom nom
14:44:25  *** _aD has joined #openttd
14:44:39  <Eddi|zuHause> the cake is a lie
15:00:40  <TrueBrain> that is the lie we tell you, yes
15:05:58  *** HerzogDeXtEr has joined #openttd
15:07:10  *** Flygon has joined #openttd
15:16:34  <TrueBrain> okay, having a node in your cluster that is out of memory gives .. "fun" effects 😄
15:16:41  <TrueBrain> up to the point I can't even login, or AWS can't even terminate it 😛
15:21:02  <pickpacket> wait a minute. Is there a 13.3 released?
15:21:36  <glx[d]> after 13.2 and 13.2.1
15:22:04  <pickpacket> sounds like I'll be doing some compiling tonight
15:22:23  <glx[d]> we had 3 releases this week end
15:22:38  <pickpacket> that's awesome, but why?
15:23:30  <TrueBrain> why not? 😄
15:24:12  <Rubidium_> not enough people playing recent enough nightlies and noticing issues
15:24:24  <pickpacket> TrueBrain: that, my friend, is a very good question!
15:24:42  <glx[d]> 13.2 had a bug on windows in fullscreen mode, 13.2.1 was fixing the bug with an attempt to be network compatible (didn't end well) and finally we just did 13.3 to fix both issues
15:24:47  <pickpacket> Rubidium_: ah, so 13.2 and then patch releases?
15:26:38  <petern> Did you also notice the nightlie release is a week old?
15:26:47  <glx[d]> again ?
15:27:31  <glx[d]> fixed
15:28:50  <DorpsGek> [OpenTTD/OpenTTD] zephyris commented on pull request #11001: Feature: Transparency option for cost and income indicators https://github.com/OpenTTD/OpenTTD/pull/11001#issuecomment-1589543792
15:29:07  <glx[d]> and nobody told us
15:29:29  <petern> Well, I did 😉
15:29:40  <petern> (Because I just noticed)
15:45:09  *** keikoz has joined #openttd
15:51:50  <TrueBrain> that moment you scale down your cluster, and AWS is like: I am going to ignore your request to kill the oldest server, and I just pick this one! MWHAHAHAHAHA
15:55:57  <TrueBrain> okay, I see why, but that is not what is documented; lol ..
16:09:29  <pickpacket> huh. Pre-signals seem pretty useless for a terminus station
16:11:22  <Eddi|zuHause> yes.
16:12:21  <Eddi|zuHause> there's exactly one use case of presignals
16:12:42  <Eddi|zuHause> which is "priority signalling"
16:13:08  <Eddi|zuHause> if you're into that sort of thing
16:17:14  *** Xaroth has joined #openttd
16:20:22  <andythenorth> ahem, logic
16:20:41  <andythenorth> how else can you build a half-adder?
16:22:32  *** Wolf01 has joined #openttd
16:24:59  <petern> https://cdn.discordapp.com/attachments/1008473233844097104/1118214444011503636/image.png
16:24:59  <petern> Correct!
16:26:19  <discord_user_03329cf> Yeah
16:33:23  <pickpacket> https://lounge.warmedal.se/uploads/8ac928e1d5a203b0/Untitled.png The blue part of the terminus blocks *a lot* less than the red part
16:33:36  <pickpacket> Happy that I've been able to improve it that much
17:04:57  <pickpacket> petern: I'm a cat with a hat on discord??? :D :D :D :D
17:05:20  <glx[d]> IRC users are cats yes
17:06:05  <pickpacket> https://lounge.warmedal.se/uploads/ebdeee421bff6d4b/image.png <-- my actual discord avatar
17:19:06  *** supermop_Home has quit IRC
17:22:05  <petern> Yeah but you're not using that account.
17:24:20  <Rubidium_> *sigh* stupid STR_JUST_STRING
17:24:41  <Rubidium_> why are you used so often with sub strings that require parameters?
17:25:30  <glx[d]> but it's "invalid" to consume params with {STRING}
17:26:57  <Rubidium_> ... but not enforced
17:27:30  <Rubidium_> so one of my string patches changes that and now it crashes/malfunctions left-and-right
17:28:09  <glx[d]> if there's only one string it's probably fine, but IIRC sometimes STR_JUST_STRING might be use in multi {STRING}
17:28:53  <glx[d]> and then the substrings start to use X: markers to get the right param
17:29:23  <petern> Those are big tea leaves 😄
17:34:29  <pickpacket> petern: nope. Because IRC is faster
17:35:15  <pickpacket> Even though I'm using a web client (thelounge.chat) on a single board computer it's still about ten times faster than using the discord web app
17:36:03  <petern> Sure but it's not going to be using your actual Discord account's avatar.
17:36:05  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #11002: Fix: when a string consumes more parameters than allowed, nullptr is attempted to be formatted https://github.com/OpenTTD/OpenTTD/pull/11002
17:36:31  <glx[d]> oh that feels very bad
17:37:09  <pickpacket> petern: of course not :) Didn't expect it to
17:38:20  <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #11002: Fix: when a string consumes more parameters than allowed, nullptr is attempted to be formatted https://github.com/OpenTTD/OpenTTD/pull/11002#pullrequestreview-1477735285
18:04:22  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #11003: Fix: do not use {STRING} when {STRING1} or {STRING2} is needed https://github.com/OpenTTD/OpenTTD/pull/11003
18:12:09  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #11002: Fix: when a string consumes more parameters than allowed, nullptr is attempted to be formatted https://github.com/OpenTTD/OpenTTD/pull/11002#pullrequestreview-1477793477
18:12:59  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #11002: Fix: when a string consumes more parameters than allowed, nullptr is attempted to be formatted https://github.com/OpenTTD/OpenTTD/pull/11002#pullrequestreview-1477794813
18:14:04  <Rubidium_> I was already wondering if you might've misread the change
18:15:42  <petern> It was obviously going to be spotted just as soon as I pressed "send"...
18:40:41  <DorpsGek> [OpenTTD/OpenTTD] DorpsGek pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/f1ec2f274c75cb5742f5ebebd082bd62ef4efced
18:40:42  <DorpsGek>   - Update: Translations from eints (by translators)
18:53:52  *** kuka has joined #openttd
18:56:16  *** kuka has left #openttd
18:58:50  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #11002: Fix: when a string consumes more parameters than allowed, nullptr is attempted to be formatted https://github.com/OpenTTD/OpenTTD/pull/11002
19:10:36  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #11004: Codechange: use StringParameters for remapping the NewGRF string control codes https://github.com/OpenTTD/OpenTTD/pull/11004
19:15:03  <Ahyangyi> https://github.com/OpenTTD/bananas-api/pull/373
19:15:28  <Ahyangyi> There are still a few checks "waiting for statuses to be reported". Should I rebase the branch (and wait for another approval)?
19:15:40  <TrueBrain> nope; do absolutely nothing is my advise 🙂
19:15:52  <Ahyangyi> Nice
19:18:48  <TrueBrain> Ahyangyi: in a bit more detail, as I mentioned in my comment in that PR, I am making changes to the infra, so currently we cannot deploy anything new to production. Hopefully by the end of the week that is fixed 🙂
19:18:58  <glx[d]> required checks have not been updated yet 🙂
19:19:13  <glx[d]> but they will
19:19:14  <TrueBrain> they have; the PR is only from before 🙂 But again, it will all be fine 🙂
19:19:16  <DorpsGek> [OpenTTD/OpenTTD] michicc approved pull request #11004: Codechange: use StringParameters for remapping the NewGRF string control codes https://github.com/OpenTTD/OpenTTD/pull/11004#pullrequestreview-1477901536
19:19:59  <glx[d]> oh right
19:22:29  <TrueBrain> I now have to battle why I can't make a server leave via nomads CLI .. it returns a 501 .. bit weird
19:25:28  <TrueBrain> ah, improper handling of invalid server names .. lovely 🙂
19:25:35  <TrueBrain> why do user validation when you can just .. not 😛
19:26:49  <TrueBrain> problem with working on infra stuff like cluster join/leave, it takes for ever .. spin up new server, wait for it to join and settle ... remove it again, repeat 😛
19:34:35  <LordAro> sometimes i wish i got to deal with stuff like that
19:34:40  <LordAro> other times, i don't
19:38:20  <TrueBrain> `Total Physical Source Lines of Code (SLOC)                = 2,062` .. for infra alone .. and this is just website + wiki + bananas .. there are a few more to come 😛
19:40:16  <TrueBrain> it mostly is a lot of figuring out how software wants to work .. like .. I now have the issue that if I replace a node, I drain it .. and that means all containers are shut down. But it doesn't wait till a new one is online, it just drains it instantly .. which is not what I would like
19:40:22  <TrueBrain> so reading the freaking configuration it is again
19:41:44  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #11001: Feature: Transparency option for cost and income indicators https://github.com/OpenTTD/OpenTTD/pull/11001#pullrequestreview-1477939682
19:43:31  <LordAro> best i get is a couple of increasingly overloaded lxd hosts and a windows domain
19:43:44  <LordAro> oh, and a boat load of ansible to pull it all together
19:44:08  <LordAro> and i have no idea how windows sysadmining works, it's just such a black box
19:44:33  <TrueBrain> haha, nobody knows 😛
19:44:36  <TrueBrain> people make really good money doing that
19:44:44  <TrueBrain> self-created-money-sink 😄
19:45:20  <TrueBrain> lol .. so during an upgrade it can start a "canary", where it temporary violates your "maximum instances" .. but when migrating, it cannot do that 😛
19:45:30  <TrueBrain> we have too many services that can only run once on a cluster .. it hurts availability!! 😄
19:49:03  <TrueBrain> changing cookies to JWTs would help there, but okay .. let's focccccuuuusssssss
19:52:42  <TrueBrain> all that is now missing for bananas to go to production in the new infra, are reload triggers .. let's see if we can get way from those weird AWS Lambdas .. they are so silly: if you haven't used them in a while, they take minutes to boot back up, making the AWS CLI fail on timeouts ..
20:11:25  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #11004: Codechange: use StringParameters for remapping the NewGRF string control codes https://github.com/OpenTTD/OpenTTD/pull/11004
20:39:28  *** nielsm has quit IRC
20:48:24  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #11005: Codechange: replace StringParameters' GetDataPointer https://github.com/OpenTTD/OpenTTD/pull/11005
21:02:24  <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #11005: Codechange: replace StringParameters' GetDataPointer https://github.com/OpenTTD/OpenTTD/pull/11005#pullrequestreview-1478049276
21:09:54  <DorpsGek> [OpenTTD/actions] TrueBrain opened pull request #49: Add: reusing workflow to call the reload trigger for a Nomad service https://github.com/OpenTTD/actions/pull/49
21:10:23  <TrueBrain> almost didn't do it, as I got lazy .. but it is needed a few times, so yeah .. 😄
21:13:20  <DorpsGek> [OpenTTD/actions] glx22 approved pull request #49: Add: reusing workflow to call the reload trigger for a Nomad service https://github.com/OpenTTD/actions/pull/49#pullrequestreview-1478063038
21:13:43  <DorpsGek> [OpenTTD/actions] TrueBrain dismissed a review for pull request #49: Add: reusing workflow to call the reload trigger for a Nomad service https://github.com/OpenTTD/actions/pull/49#pullrequestreview-1478063038
21:13:45  <TrueBrain> sorry .. spotted I copied the `deploy` variant a bit too much 😄
21:13:46  <DorpsGek> [OpenTTD/actions] TrueBrain updated pull request #49: Add: reusing workflow to call the reload trigger for a Nomad service https://github.com/OpenTTD/actions/pull/49
21:15:13  *** keikoz has quit IRC
21:24:10  <DorpsGek> [OpenTTD/actions] TrueBrain merged pull request #49: Add: reusing workflow to call the reload trigger for a Nomad service https://github.com/OpenTTD/actions/pull/49
21:24:30  *** Wolf01 has quit IRC
21:26:21  <DorpsGek> [OpenTTD/OpenTTD] zephyris commented on pull request #11001: Feature: Transparency option for cost and income indicators https://github.com/OpenTTD/OpenTTD/pull/11001#pullrequestreview-1478077094
21:28:02  <DorpsGek> [OpenTTD/actions] TrueBrain created new tag: v4.4.0 https://github.com/OpenTTD/actions/releases/tag/v4.4.0
21:31:51  <DorpsGek> [OpenTTD/wiki-data-staging] TrueBrain opened pull request #10: Fix: [CI] migrate reload triggers to new backend https://github.com/OpenTTD/wiki-data-staging/pull/10
21:35:17  <DorpsGek> [OpenTTD/wiki-data] TrueBrain opened pull request #22: Fix: [CI] migrate reload triggers to new backend https://github.com/OpenTTD/wiki-data/pull/22
21:38:22  *** _aD has quit IRC
21:49:01  <TrueBrain> lol, so changing a GitHub Secrets via Pulumi has a nice bug
21:49:04  <TrueBrain> as it wants to replace it
21:49:09  <TrueBrain> and the default for replace is create + delete
21:49:11  <TrueBrain> on the same key
21:49:18  <TrueBrain> so efficiently, there is no more key after the change
21:49:22  <TrueBrain> not exactly the intention 😄
21:49:36  *** HerzogDeXtEr has quit IRC
21:49:50  <DorpsGek> [OpenTTD/wiki-data-staging] TrueBrain merged pull request #10: Fix: [CI] migrate reload triggers to new backend https://github.com/OpenTTD/wiki-data-staging/pull/10
21:51:50  <DorpsGek> [OpenTTD/wiki-data] TrueBrain merged pull request #22: Fix: [CI] migrate reload triggers to new backend https://github.com/OpenTTD/wiki-data/pull/22
21:53:07  <TrueBrain> lol, stupid typo
21:53:17  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #11005: Codechange: replace StringParameters' GetDataPointer https://github.com/OpenTTD/OpenTTD/pull/11005
21:53:22  <DorpsGek> [OpenTTD/wiki-data] TrueBrain opened pull request #23: Fix: [CI] service-name is called "wiki-prod" for the production wiki https://github.com/OpenTTD/wiki-data/pull/23
21:53:29  <DorpsGek> [OpenTTD/wiki-data] TrueBrain merged pull request #23: Fix: [CI] service-name is called "wiki-prod" for the production wiki https://github.com/OpenTTD/wiki-data/pull/23
21:56:06  *** seriphyn has joined #openttd
21:56:06  <seriphyn> Are there any plans for a "Vanilla-style" and "New-style" game settings master switch? things that include all the QoL changes, cargo distribution, etc all at once without having to dig int othe settings?
21:56:10  <seriphyn> esp for new players
21:56:38  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #11006: Codechange: pass string parameters by reference https://github.com/OpenTTD/OpenTTD/pull/11006
22:00:49  <TrueBrain> seriphyn: we had various of talk about something similar recently .. not a real plan yet, but clearly we need to do something here 😄
22:03:42  <seriphyn> Oh neat! Yeah because in the past I've said how a lot of the essential features are turned off by default, because ye olde players who get OTTD just wanting to play what they remember back in the 90s makes sense...but then you have newer players who hop on the bandwagon, and stuck with "outdated" features (e.g. TpF1/2 et al having destinations) that might confuse/frustrate at first
22:04:40  <TrueBrain> We have been pusing a bit of those "new style" settings lately, but we have another one in the pipeline we don't actually want to enable for old players
22:04:43  <TrueBrain> but do want to enable for new players
22:05:02  <TrueBrain> we have a bunch of options here, from a GUI on first-start to "you already played the game so you get the old settings"
22:05:35  <Kuhnovic> Curious now, what setting are you talking about?
22:05:38  <TrueBrain> https://github.com/OpenTTD/OpenTTD/pull/10938
22:05:46  <TrueBrain> was just looking for the ticket 🙂
22:06:07  <TrueBrain> ideas and suggestions are very welcome .. it is a bit tricky 🙂
22:06:17  <seriphyn> Kuhnovic: cargo distribution is the big one that comes to mind...I honestly for the life of me swear there are others but I can't remember lol
22:08:40  <Kuhnovic> TrueBrain: I like the idea! But I see how this can turn into a flame war quickly :p
22:09:01  <TrueBrain> it is more: how do you present this cleanly, and that there are not many edge-cases
22:09:17  <TrueBrain> that is not fully trivial .. as well, we now have 14.0, so sure, we can do a GUI: old or new?
22:09:24  <TrueBrain> but in 15.0 we might add another setting .. do we ask again?
22:18:49  <Kuhnovic> Yeah you don't want to bother the user too much with things like this
22:19:02  <Kuhnovic> Don't be a Clippy
22:20:26  <DorpsGek> [OpenTTD/OpenTTD] Kuhnovic commented on pull request #10938: Feature: Option to disable activate-on-release behaviour of toolbar dropdown buttons. https://github.com/OpenTTD/OpenTTD/pull/10938#issuecomment-1590131103
22:22:16  <Kuhnovic> TrueBrain: Can you slap a "preview" label on it? Then we can play around with it a bit
22:22:39  <TrueBrain> I expected Tyler to have done that already 😛 Done!
22:23:57  *** Deep3D has joined #openttd
22:26:02  <EmperorJake> Don't assume it's game-breaking for old users 🙂  For example, selecting a railtype from the bottom of a long list is quite awkward using a trackpad, and downright impossible if the list is long enough to have a scroll bar. So I'd enable this setting even as a 15 year veteran of the game
22:27:03  <TrueBrain> knowing the ... volatility of some of our more vocal players, you might just be the minority here 😄 But tnx, good to know 🙂
22:31:34  <petern> Yeah, for #10938 is the easiest solution is to default to the new behaviour. TBH even make it an option may be too much 😉
22:32:48  <TrueBrain> less settings is better, right? 😄
22:34:14  <TrueBrain> lol, there are still 3 servers running 1.3.0 active
22:34:30  <TrueBrain> sadly, password protected
22:50:12  <Eddi|zuHause> that must obviously be when the best features happened
23:16:47  <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #11006: Codechange: pass string parameters by reference https://github.com/OpenTTD/OpenTTD/pull/11006#pullrequestreview-1478197849

Powered by YARRSTE version: svn-trunk