IRC logs for #openttd on OFTC at 2025-05-30
            
00:02:04 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
00:06:50 *** WormnestAndroid has joined #openttd
00:31:17 *** WormnestAndroid has quit IRC (Remote host closed the connection)
00:38:18 *** bryjen has quit IRC (Quit: Leaving)
00:39:01 *** WormnestAndroid has joined #openttd
02:22:11 *** Wormnest has joined #openttd
02:45:45 *** gnu_jj has joined #openttd
02:49:06 *** gnu_jj_ has quit IRC (Ping timeout: 480 seconds)
02:58:12 *** Wormnest has quit IRC (Quit: Leaving)
03:00:06 *** herms has quit IRC (Quit: bye)
03:04:05 *** herms has joined #openttd
03:23:20 *** godbed has joined #openttd
03:23:26 *** Zathras has joined #openttd
03:26:46 *** debdog has quit IRC (Ping timeout: 480 seconds)
03:26:49 *** D-HUND has quit IRC (Ping timeout: 480 seconds)
04:14:41 *** keikoz has joined #openttd
05:26:08 *** Flygon has joined #openttd
05:48:03 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14303: Codechange: [Script] Don't report multiple errors on valuator/filter failure https://github.com/OpenTTD/OpenTTD/pull/14303#pullrequestreview-2880239766
05:55:42 <DorpsGek> [OpenTTD/OpenTTD] Release workflow was not successful https://github.com/OpenTTD/OpenTTD/actions/runs/15339859072
07:03:01 <andythenorth> well
07:03:03 <andythenorth> coffee?
07:13:29 <peter1138[d]> Yes.
07:15:09 *** kuhnovic has joined #openttd
07:15:09 <kuhnovic> Definitely
07:38:57 *** toktik has quit IRC (Read error: Connection reset by peer)
07:39:15 *** toktik has joined #openttd
07:57:21 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14309: Codechange: De-duplicate common parts of graph windows with cargo legend. https://github.com/OpenTTD/OpenTTD/pull/14309
08:14:33 <peter1138[d]> What's the betting that MS Teams will require me to sign in again as soon as a video meeting starts...
08:17:30 <andythenorth> well only so it can then ask you to update
08:31:03 <DorpsGek> [OpenTTD/OpenGFX] zephyris commented on pull request #101: Feature: Badge menu vertical ellipsis icon https://github.com/OpenTTD/OpenGFX/pull/101#issuecomment-2921610683
08:31:59 <_zephyris> I swear login session expiry and updates are timed deliberately
08:32:00 <_zephyris> β˜•
08:35:15 <LordAro> it does seem to have been a bit funky recently, outlook web keeps signing me out, and another user recently complained about something similar
08:58:55 <xarick> hi
09:50:06 <DorpsGek> [OpenTTD/OpenTTD] zephyris commented on discussion #12886: Push-pull: What if trains could back up? https://github.com/OpenTTD/OpenTTD/discussions/12886
10:02:56 <andythenorth> any reason railtype prop 0x1D couldn't be extend to dword?
10:05:43 <andythenorth> hmm nvm, I've found the necessary clause in the Standardised Scheme πŸ™‚
10:05:49 <xarick> the number of times that game director says "like".... <https://www.youtube.com/watch?v=01eP5vTQMvI>
10:07:49 <peter1138[d]> andythenorth: You mean the list of compatible types?
10:08:00 <andythenorth> I mean alternative types
10:08:11 <peter1138[d]> Erm, yes, that one.
10:08:25 <peter1138[d]> Interesting fact, you can just specify the property multiple times.
10:08:31 <andythenorth> yes, it's additive
10:08:32 <peter1138[d]> NML might have fun with that, mind you.
10:08:42 <andythenorth> that's not documented, but the standardised scheme relies on it
10:08:59 <andythenorth> as the convention is to extend RAIL and ELRL
10:11:04 <andythenorth> for basic narrow gauge, the scheme provides for `NAAN`, which needs to then set prop 0x1D to all the other possible narrow gauge labels. So `NA[foo]N` for non-electrified, where `foo` is supposed to be A to E and a to e.
10:11:15 <andythenorth> But it has been extended now to F and presumably f
10:11:37 <andythenorth> I was just going to autogenerate labels in the compile for A to Z and a to z
10:11:59 <andythenorth> should fit in a byte though
10:13:32 <andythenorth> wonder if I can one line python that πŸ˜›
10:17:59 <andythenorth> "please run that python code" -> GPT -> output -> "did you run the code" -> GPT -> "no I interpolated what the output would be"
10:18:19 <andythenorth> but you know, programming is a dead skill
10:19:37 <andythenorth> this is quite neat though πŸ™‚ `labels = [f"NA{char}N" for char in string.ascii_letters]`
10:50:29 <LordAro> is it just the list comprehension that's neat, or am i missing something? :p
11:06:53 <andythenorth> it's just neat the ascii_letters exists, and happens to map (currently) to the railtype thingy
11:22:27 *** WormnestAndroid has quit IRC (Remote host closed the connection)
11:22:36 *** WormnestAndroid has joined #openttd
11:32:50 <peter1138[d]> Well, labels are actually 4 arbitrary bytes.
11:41:14 <andythenorth> aren't they just πŸ™‚
11:54:33 *** WormnestAndroid has quit IRC (Remote host closed the connection)
11:54:45 *** WormnestAndroid has joined #openttd
12:06:17 *** WormnestAndroid has quit IRC (Remote host closed the connection)
12:06:20 *** WormnestAndroid has joined #openttd
12:14:02 <DorpsGek> [OpenTTD/OpenTTD] orudge opened pull request #14310: Codechange: [CI] Azure Code Signing has been rebranded 'Trusted Signing' https://github.com/OpenTTD/OpenTTD/pull/14310
12:14:20 *** Tirili has joined #openttd
12:35:56 <andythenorth> lunch?
12:38:44 <DorpsGek> [OpenTTD/OpenTTD] glx22 merged pull request #14303: Codechange: [Script] Don't report multiple errors on valuator/filter failure https://github.com/OpenTTD/OpenTTD/pull/14303
12:39:50 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1377989883087556709/image.png?ex=683af895&is=6839a715&hm=54662b3eeeafc8d4b3370f8c39f78716969a8f4bd009304b3351dc09c7209d7b&
12:39:50 <xarick> how much value would these 2 functions add?
12:56:40 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1377994120576106516/image.png?ex=683afc88&is=6839ab08&hm=8d45e1354124855d6390702020ee43716750de9aa22a878dcb52a74e0ca87f94&
12:56:40 <peter1138[d]> Okay, accruing a daily average gives best results, just a shame it needs a pool iteration once a day. Could maybe spread it out over ticks?
12:57:31 <peter1138[d]> Average is `average * 30 + current_value) / 31` which I believe is reasonable for a rolling monthly average,
12:59:26 <talltyler> Hello PIRS
12:59:32 <andythenorth> hello lunch πŸ™‚
12:59:33 <talltyler> I love PIRS
12:59:47 <talltyler> and lunch, but it’s not quite time for that where I am
13:01:14 <peter1138[d]> Hmm, `OnTick_Industry()` iterates the pool every tick anyway.
13:04:46 <peter1138[d]> talltyler: Never actually played a game with it, it doesn't seem to get much mention around here.
13:08:01 <peter1138[d]> I like the idea of adding more things but not ALL the things.
13:08:08 <peter1138[d]> (A bit like rail types ;-))
13:13:53 <talltyler> It's very well designed for gameplay, one of my favorite industry sets for solo games.
13:17:42 *** WormnestAndroid has quit IRC (Remote host closed the connection)
13:17:44 *** WormnestAndroid has joined #openttd
13:27:56 *** WormnestAndroid has quit IRC (Remote host closed the connection)
13:28:03 *** WormnestAndroid has joined #openttd
13:28:48 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick opened pull request #14311: Add: [Script] ScriptList::GetSorterType and ScriptList::GetSorterDirection https://github.com/OpenTTD/OpenTTD/pull/14311
13:29:20 <peter1138[d]> Hmm, is there an easily accessed flag to see if stockpiling is used?
13:31:31 <andythenorth> I can't see one, iirc, the production callback being enabled might be a flag for it
13:31:43 <andythenorth> but what the grf actually does....is private
13:31:46 <andythenorth> badges? πŸ˜›
13:32:03 <andythenorth> wonder why `tile_is_railtype` returns True for missing railtypes
13:34:02 <andythenorth> oh because of the silly nml fallbacks
13:34:11 <andythenorth> I'm testing for a type that has ELRL in fallbacks
13:36:09 <andythenorth> ok how about badges for railtypes? πŸ˜›
13:38:19 <peter1138[d]> Like property 1E?
13:40:21 <andythenorth> well πŸ™‚
13:40:26 <andythenorth> maybe
13:40:34 <andythenorth> readable from vehicle though? πŸ˜›
13:40:40 <andythenorth> no scope for that
13:41:08 <andythenorth> we have the vehicle var 0x63 stuff though
13:43:22 <peter1138[d]> That allows testing of power and compatibility.
13:48:07 <peter1138[d]> Okay, so let's say var 65 will test for a badge on the rail/road/tram type of the current vehicle.
13:49:24 <peter1138[d]> Or maybe just implement the nearby tile test for vehicles, so they can test for badges in stations too.
13:51:19 <andythenorth> intriguing idea πŸ™‚
13:51:27 <andythenorth> station specific vehicle sprites πŸ˜›
13:58:35 <DorpsGek> [OpenTTD/workflows] glx22 opened pull request #69: Fix: Detection of latest nightly ref https://github.com/OpenTTD/workflows/pull/69
13:59:10 *** Wormnest has joined #openttd
13:59:10 <andythenorth> forgot to lunch
13:59:46 <_glx_> that's why latest nighly build failed, it should not have started at all
14:09:00 <DorpsGek> [OpenTTD/workflows] LordAro commented on pull request #69: Fix: Detection of latest nightly ref https://github.com/OpenTTD/workflows/pull/69#pullrequestreview-2881479238
14:09:35 *** Wormnest has quit IRC (Read error: No route to host)
14:09:52 <peter1138[d]> How has it been working recently if it's since Dec 2024?
14:13:39 <_glx_> it didn't, always triggered a new nightly even if no changes
14:15:11 <peter1138[d]> Okay, I guess that is roundaboutly explained in the PR, but wasn't clear to me.
14:20:49 <DorpsGek> [OpenTTD/workflows] glx22 updated pull request #69: Fix: Detection of latest nightly ref https://github.com/OpenTTD/workflows/pull/69
14:21:44 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #14310: Codechange: [CI] Azure Code Signing has been rebranded 'Trusted Signing' https://github.com/OpenTTD/OpenTTD/pull/14310#pullrequestreview-2881517595
14:24:14 <DorpsGek> [OpenTTD/workflows] TrueBrain approved pull request #69: Fix: Detection of latest nightly ref https://github.com/OpenTTD/workflows/pull/69#pullrequestreview-2881526191
14:24:39 <DorpsGek> [OpenTTD/workflows] TrueBrain commented on pull request #69: Fix: Detection of latest nightly ref https://github.com/OpenTTD/workflows/pull/69#issuecomment-2922543908
14:25:59 <truebrain> Clearly there haven't been much days without a change, as I have seen that build failure only a handful amount of times πŸ˜„
14:27:43 <peter1138[d]> Well, I am obviously broken because I didn't understand the description 😦
14:28:51 <DorpsGek> [OpenTTD/workflows] LordAro commented on pull request #69: Fix: Detection of latest nightly ref https://github.com/OpenTTD/workflows/pull/69#pullrequestreview-2881542050
14:30:14 <truebrain> peter1138[d]: Haha, or you are just not that deep into this part of our release stuff πŸ˜›
14:31:46 <peter1138[d]> If it said something like "nightlies are being built when there is no new nightly" I would've probably understood it. It just says "triggers a nightly build" but in my mind, the nightly build was already triggered otherwise this piece wouldn't be running.
14:32:40 <truebrain> I get where you are coming from. In my mind, this workflow isn't the nightly yet: it is checking whether it should trigger one
14:32:53 <truebrain> And somewhere there is the disconnect, I guess :d
14:34:17 <truebrain> It used to be needed to have it in another repo .. but I have been wondering if that still is the case
14:34:29 <truebrain> Maybe we should just move it into OpenTTD repo
14:34:54 <peter1138[d]> Anyway, is it coffee time, or tea time?
14:35:28 <truebrain> Beer time
14:36:13 <andythenorth> I had coffee, might have wine
14:42:44 <peter1138[d]> Oof.
14:49:55 *** Wormnest has joined #openttd
15:02:05 *** Tirili has quit IRC (Quit: Leaving)
15:06:01 <peter1138[d]> Does FIRS stockpile?
15:08:00 <andythenorth> nope
15:08:17 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1378027244240371752/image.png?ex=683b1b61&is=6839c9e1&hm=ae66be1525aa143a6fb3202c7f35988392157e0876ab128ef54c779098e47acc&
15:08:17 <peter1138[d]> Does this difference in order matter?
15:08:26 <andythenorth> well, it uses the stockpile, because that's the implementation in OpenTTD
15:08:28 <peter1138[d]> (Say no please)
15:08:33 <andythenorth> but it clears the stockpile immediately on processing
15:08:42 <peter1138[d]> Yeah.
15:08:43 <talltyler> Order doesn't matter to me, personally
15:08:47 <andythenorth> the order doesn't matter to me
15:08:54 <andythenorth> I will change my opinion next time I notice ;P
15:08:56 <andythenorth> ignore that
15:09:34 <talltyler> There's a draft PR somewhere about letting the player sort the graph, so they can choose alphabetical or value
15:09:49 *** yiffgirl has joined #openttd
15:09:49 <yiffgirl> as someone who brought pitchforks when changing the order on the right was pr'd, yeah it's fine
15:10:09 <peter1138[d]> I'm not touching the industry window, so that's not changing.
15:10:48 <peter1138[d]> I've touched the graph window, so it's now sorted by name (like it already is for the payment rate graph)
15:11:08 <talltyler> Makes sense to me.
15:15:30 <peter1138[d]> Hmm, okay, the text that says in the industry window how much is waiting / stockpiled is driven by callbacks.
15:15:42 <peter1138[d]> So no simple on/off property I can test 😦
15:21:53 *** SigHunter has quit IRC ()
15:24:45 *** SigHunter has joined #openttd
15:46:36 <_glx_> ECS uses stockpiling I think
15:47:41 <_glx_> but I don't know if it uses the callbacks
15:47:57 <peter1138[d]> Hmm, AIs don't seem to do anything with freight in IOTC
15:51:49 <peter1138[d]> Watching John Romero play the new Doom Dark Ages.
16:06:13 <DorpsGek> [OpenTTD/workflows] glx22 merged pull request #69: Fix: Detection of latest nightly ref https://github.com/OpenTTD/workflows/pull/69
16:09:41 <talltyler> Uh, IOTC uses callbacks for cargo value, AI might not know what to do with that πŸ™‚
16:09:58 <talltyler> (Custom cargo value which doesn’t depend on distance, and can be negative)
16:15:20 <xarick> what is IOTC ? internet of things cute?
16:15:41 <_glx_> industries of the carribean
16:15:50 <xarick> oh, a newgrf?
16:20:03 <xarick> it's now 9 commits <https://github.com/OpenTTD/OpenTTD/compare/master...SamuXarick:OpenTTD:slow-valuate>
16:20:45 <xarick> but I haven't dealt with filter functions, they can also crash, it's just that nobody uses them yet
16:24:06 <_glx_> filter is in constructor, the only way to work around opcode limit is replacing the constructor with filter with constructor without filter, valuate using filter, keep value 1
16:24:55 <peter1138[d]> Squishy squashy.
16:31:50 <xarick> oops mistakes on my code 😦
16:32:34 <xarick> list[item] = but there is no list anymore, it's just this
16:34:12 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1378048866615689216/image.png?ex=683b2f84&is=6839de04&hm=9e9224e82ac93b4c01976eb09462ba7a1d2a94926b8c2adc6e496260413845a4&
16:36:10 *** Smedles has quit IRC (Quit: http://quassel-irc.org - Chat comfortably. Anywhere.)
16:36:25 *** Smedles has joined #openttd
16:38:08 <xarick> `this[item] = `
16:43:02 <peter1138[d]> Hmm, that's something I hadn't considered.
16:43:16 <peter1138[d]> Sometimes you just gotta clean your ball.
16:58:20 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14312: Add: [Badges] Vehicle variable to query badges of current rail or road types. https://github.com/OpenTTD/OpenTTD/pull/14312
16:59:11 <andythenorth> πŸ‘€
16:59:35 <andythenorth> shall I ?
17:03:36 <andythenorth> hmm doesn't build πŸ™‚
17:03:52 <andythenorth> let's see
17:04:19 <andythenorth> oh same as this :Dhttps://github.com/OpenTTD/OpenTTD/actions/runs/15351734494/job/43201406041?pr=14312
17:08:40 <peter1138[d]> Whoops.
17:08:59 <andythenorth> "Friday Afternoon Mini Metro"
17:09:16 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14312: Add: [Badges] Vehicle variable to query badges of current rail or road types. https://github.com/OpenTTD/OpenTTD/pull/14312
17:09:21 <peter1138[d]> It's a bit warm.
17:12:57 <xarick> its 30
17:14:21 <andythenorth> should I patch nml? πŸ˜›
17:17:16 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1378059701866074272/image.png?ex=683b399c&is=6839e81c&hm=5afc1db8c2276c26dc7af3b1a59085eda850e4ed4330d039457480196237c675&
17:17:16 <andythenorth> peter1138[d]: works
17:17:48 *** Wormnest has quit IRC (Quit: Leaving)
17:22:01 <peter1138[d]> Now you just need a standardised badge scheme for standardised rail types and... yeah... not all that useful :p
17:23:06 <andythenorth> dunno
17:23:33 <andythenorth> it reduces the yak-shaving for compatible and alternative and powered
17:23:40 <andythenorth> which is 6x harder than needed
17:44:34 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
17:47:58 *** WormnestAndroid has joined #openttd
17:53:00 <DorpsGek> [OpenTTD/OpenTTD] Ruizaki opened pull request #14313: Update townname.h https://github.com/OpenTTD/OpenTTD/pull/14313
17:53:29 <DorpsGek> [OpenTTD/OpenTTD] Ruizaki closed pull request #14313: Update townname.h https://github.com/OpenTTD/OpenTTD/pull/14313
17:53:32 <DorpsGek> [OpenTTD/OpenTTD] Ruizaki commented on pull request #14313: Update townname.h https://github.com/OpenTTD/OpenTTD/pull/14313#issuecomment-2923035364
17:53:56 <DorpsGek> [OpenTTD/OpenTTD] Ruizaki reopened pull request #14313: Update townname.h https://github.com/OpenTTD/OpenTTD/pull/14313
17:54:27 <LordAro> uh huh
17:54:31 <peter1138[d]> Ah the old "update file" commit message
17:55:00 <andythenorth> 14312 is probably fine, can you merge it? πŸ˜›
17:55:09 <andythenorth> I can do the one line nml patch
17:56:15 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1378069514431893554/image.png?ex=683b42bf&is=6839f13f&hm=82b19edb12f0bb9d362875f74100fc4304231edca85dcd159ac85730316a4db4&
17:56:15 <xarick> hmm how do I proceed from there
18:14:12 *** Wolf01 has joined #openttd
18:16:47 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
18:23:25 *** WormnestAndroid has joined #openttd
18:27:32 *** kuka_lie has joined #openttd
18:37:35 <fairyflossy> ...so we are getting badges on tracks in the end?
18:42:27 <peter1138[d]> Badges already exist on tracks.
18:43:10 <peter1138[d]> They are not involved in power and compatibility decisions.
18:47:16 <andythenorth> they're an elegant solution for stuff like 'go faster if fancy signals'
18:47:23 <andythenorth> they might solve rack rail too
18:47:33 <andythenorth> rack rail description in the standardised scheme is lolz
18:47:40 <andythenorth> "do this, and do this, but neither works"
18:49:54 <andythenorth> or `badge rail_behaviour/has_rack` (argue the words later)
18:50:11 <andythenorth> then change engine props accordingly
18:50:50 <andythenorth> hmm waypoint badges
18:51:02 <andythenorth> 'whistle/true'
18:55:33 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1378084435265585262/image.png?ex=683b50a4&is=6839ff24&hm=0fd8b2c17f6f2598ecb6a22570f0d20506c7a9f4c6cd0ea605c91222bea0f5aa&
18:55:33 <xarick> hmm
18:55:53 <xarick> hmm yeah well
18:57:52 <xarick> GSVehicleList.constructorCompat15 <- GSVehicleList.constructor this cursed coding works, kinda
19:14:52 <peter1138[d]> Oops, side-tracked again.
19:20:04 <andythenorth> was it lunch?
19:20:51 <peter1138[d]> Tempted to go much further with #14309
19:21:25 <andythenorth> grf graphs?
19:21:56 <andythenorth> action 0 props:
19:21:56 <andythenorth> - axes labels
19:21:56 <andythenorth> - axes ticks
19:21:56 <andythenorth> - colours
19:21:56 <andythenorth> - legend style
19:22:06 <peter1138[d]> Ok.
19:22:12 <peter1138[d]> And the data comes from?
19:22:31 <andythenorth> πŸ€·β€β™‚οΈ
19:22:35 <andythenorth> I'm just the project manager
19:24:21 <andythenorth> oops seems I posted in the wrong channels
19:24:27 <_glx_> xarick: that fills very wrong
19:24:29 <andythenorth> hmm `railtype_has_badge` though?
19:24:46 <andythenorth> `roadtype_has_badge`, `tramtype_has_badge`?
19:25:06 <andythenorth> I think we can just duplicate the assignments for road and tram to var 0x65 with same params, different token
19:25:14 * andythenorth maybe
19:25:51 <andythenorth> or we use the existing `tile_[predicate_foo]` pattern
19:26:02 <andythenorth> which assumes that tile == railtype, for vehicles
19:26:54 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1378092324034449408/image.png?ex=683b57fd&is=683a067d&hm=96a9d3c46b80b1f1b45d077265d1f8e7e94c9726ca82406916a76356e1286b69&
19:26:54 <xarick> I made this far
19:27:08 <xarick> i was trying to print class name but don't know how
19:27:42 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1378092528229941348/image.png?ex=683b582e&is=683a06ae&hm=3018f916a92ca494863768b8e17ad227a7644e94a1a7e96dbbfd35c33aea0165&
19:29:06 <andythenorth> xarick: maybe you need a code generator πŸ™‚
19:29:18 <_glx_> there's no way to print classname IIRC
19:30:00 <andythenorth> there might be if you store the name in a prop on the class in advance
19:30:17 <_glx_> squirrel doesn't do that IIRC
19:31:45 <andythenorth> AI author can though
19:32:17 <andythenorth> there are some extensible print methods, I forgot what they do
19:39:20 <_glx_> there's `typeof` but it defaults to "instance" (I once made it return the proper type, but it broke many exisiting strings so I reverted)
19:40:36 <andythenorth> hmm I can't base an nml patch on master
19:40:47 <andythenorth> it relies on badge support in 359
19:41:05 <andythenorth> `badge_parameter`
19:41:24 <andythenorth> can I branch from 359, or is that not really done?
19:45:56 <xarick> this.getclass() gives me an unique memory address i think
19:46:13 <_glx_> yes an OT_CLASS
19:46:19 <_glx_> but no name in it
19:46:34 <DorpsGek> [OpenTTD/nml] andythenorth opened pull request #378: Support vehicle var 0x65 https://github.com/OpenTTD/nml/pull/378
19:46:57 <andythenorth> yolo swag?
19:47:34 <_glx_> but you don't need the class name
19:48:17 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1378097704823033936/image.png?ex=683b5d00&is=683a0b80&hm=575f7ad91948c6a6cc5e04d8f1869868155643d9cf8c7a4470bac5e73c4e80f8&
19:48:17 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1378097705095794698/image.png?ex=683b5d00&is=683a0b80&hm=1b3218d298be7aeeb6853cf4719cce7c4a960d65c501ea84f1db93571aa1e959&
19:48:40 <_glx_> all that just for debug statement
19:50:51 <_glx_> you could do `GSTownList._typeof <- function() { return "GSTownList"; }`
19:51:46 <_glx_> then you can print `typeof this` in your debug line
19:52:29 <_glx_> but that will break anything expecting `typeof this` to return "instance"
20:03:18 <andythenorth> you could add __class__ namespace πŸ˜›
20:03:21 <andythenorth> and add __name__
20:04:04 <_glx_> squirrel relies on metamethod for that
20:04:40 <andythenorth> I've not touched GS for 12 months, have forgotten all squirrel πŸ™‚
20:05:15 <_glx_> there's usually no need to know the name
20:14:50 <andythenorth> _looking at 359_ https://github.com/OpenTTD/nml/pull/359
20:15:57 <xarick> local list = this.constructorCompat15(); this doesn't do what I want
20:18:34 <xarick> constructor doesn't return anything right? 😦 oh well
20:18:45 <xarick> returns null
20:21:43 <andythenorth> ha ha we have been busy πŸ˜„ v
20:21:47 <andythenorth> https://wiki.openttd.org/en/Development/NewGRF/Specification%20Status#openttd-15-0
20:26:33 <_glx_> xarick: IIRC `constructor` doesn't return anything
20:27:03 <_glx_> but it should create the c++ object and link it to `this`
20:27:45 <xarick> I just realized that :p
20:30:15 <peter1138[d]> Hmm. So much side-tracking 😦
20:34:04 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1378109227926814833/image.png?ex=683b67bb&is=683a163b&hm=c23f03e941bcea8c7254fec932d9dca8745a70b8f139e660412d174d883aab1e&
20:34:04 <xarick> probably works
20:34:27 <_glx_> something like ```
20:34:27 <_glx_> try {
20:34:27 <_glx_> this.constructorCompat15(filter[, arg1, ...]);
20:34:27 <_glx_> } catch (e) {
20:34:27 <_glx_> if (!e.find("CPU")) throw e;
20:34:29 <_glx_> this.Valuate(filter, [, arg1, ...]); // using your slow valuate probably
20:34:29 <_glx_> this.RemoveValue(0);
20:34:31 <_glx_> }```
20:35:58 <xarick> squirrel accepts [, ...] things?
20:45:04 <xarick> nop
20:46:15 <_glx_> no it's just syntax simplification
20:47:32 <_glx_> but your version should partially be fine except when constructor has required args before the filter
20:50:40 <xarick> case 1: foreach(item, _ in this) if (!vargv[0]()) this[item] = null; return;
20:52:10 <xarick> case 2: foreach (item, _ in this) if (!vargv[0](vargv[1])) this[item] = null; return;
20:52:20 <xarick> so ugly coding
20:57:21 *** kuka_lie has quit IRC (Remote host closed the connection)
21:01:44 <xarick> hmm wrong number of parameters, i failed
21:03:22 <_glx_> !vargv[0](item)
21:03:45 <xarick> indeed
21:03:47 <_glx_> it's like valuate
21:16:42 <_zephyris> andythenorth: I had no idea this existed! Great document, and looks like a big thank you to frosch123
21:19:13 <xarick> I'm not checking if the filter_function returns bool
21:19:25 <xarick> maybe I should
21:26:12 *** Flygon has quit IRC (Read error: Connection reset by peer)
21:27:46 *** keikoz has quit IRC (Ping timeout: 480 seconds)
21:28:18 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14314: Add: Hover on graph legend to highlight line. https://github.com/OpenTTD/OpenTTD/pull/14314
21:35:17 <andythenorth> naptime?
21:37:01 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1378125070739705866/image.png?ex=683b767d&is=683a24fd&hm=65a7e861a146c172bda9761c16c0ec9f2e7634b5fad02f033b37044c0ce8c20b&
21:37:17 <xarick> this gonna be costly in ops
21:44:04 <xarick> oh, the actual filter function: `local filter_function = function(item, out, max) { return GSBase.Chance(out, max); };`
21:44:08 <xarick> for testing
21:48:33 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14314: Add: Hover on graph legend to highlight line. https://github.com/OpenTTD/OpenTTD/pull/14314
21:57:48 <andythenorth> now I see your sidequest
21:58:38 <peter1138[d]> Heh
21:59:35 <andythenorth> do you get a badge for each quest completed?
22:01:10 *** tokai|noir has joined #openttd
22:01:10 *** ChanServ sets mode: +v tokai|noir
22:02:43 <peter1138[d]> 40+ apparently.
22:08:11 *** tokai has quit IRC (Ping timeout: 480 seconds)
22:09:19 <andythenorth> I've forgotten what quest I was on
22:09:25 <andythenorth> something about trains
22:13:52 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
22:15:11 <andythenorth> oh I was playing OpenTTD
22:15:29 <andythenorth> then I built a PR and patched my grf
22:15:35 <andythenorth> wonder if I've broken my savegame πŸ˜›
22:16:35 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1378135027694567455/image.png?ex=683b7fc3&is=683a2e43&hm=32f492065c0bc66f32221fafed847992c7705fffacc9c511343d26c2ff8496c9&
22:16:35 <peter1138[d]> Sidequest did you say?
22:17:06 <andythenorth> have you considered faking the data to test if anyone is checking? πŸ™‚
22:17:46 <peter1138[d]> So this is 1) let's add history for cargo accepted as well as produced 2) let's add highlighting because now there's a lot of data 3) let's extend it from just months to quarters and years as well.
22:18:07 <peter1138[d]> It's easier to just run a few years with AIs.
22:18:08 <andythenorth> oh dear, I failed to save for some time
22:19:35 <andythenorth> "errors of omission and comission"
22:19:35 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1378135784313454673/image.png?ex=683b8077&is=683a2ef7&hm=a45a0e0b2cd96fe69f45cd22810dc798eb9b95fa8bd226d002325eb62463a04d&
22:19:35 <peter1138[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1378135784632356914/image.png?ex=683b8077&is=683a2ef7&hm=52047f4ac08ac1bc05d3bf7af1555ab688d74a8eb49d3daef06916ffe7ca004b&
22:19:41 <peter1138[d]> Quarters can be more revealing...
22:20:24 <andythenorth> inclined to try and use this in FIRS GS
22:20:36 <peter1138[d]> Use what?
22:20:40 <andythenorth> production history
22:20:48 <peter1138[d]> Ah.
22:20:50 <andythenorth> GS stuff is usually patchable
22:20:58 <peter1138[d]> Yeah, there's no script interface for this.
22:21:04 <andythenorth> I started writing some concept about industry happiness
22:21:07 <peter1138[d]> Also you "joked" about NewGRF graphs earlier.
22:21:12 <andythenorth> I forget waht, it was all squirrel
22:21:15 <peter1138[d]> But actually, GS graphs could be a thing.
22:21:30 <andythenorth> GS, The Revival
22:21:32 <andythenorth> GS 2
22:21:38 <andythenorth> goes with GS 1
22:21:40 <peter1138[d]> Cargo monitors have their uses, but... monitoring everything is kinda awkward.
22:22:32 <andythenorth> it's a bit yak-shavey
22:22:39 <andythenorth> I think I just set up some massive loop for it
22:23:09 <peter1138[d]> Cargo monitors support per-company stats though.
22:23:27 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1378136755861192855/image.png?ex=683b815f&is=683a2fdf&hm=9ee11e09aecab57f31a2e12f6de50f49291dbb410e528a8452980b03bc454310&
22:23:27 <andythenorth> ha ha
22:23:28 <andythenorth> what did I do?
22:23:48 <peter1138[d]> Who knows.
22:24:31 <andythenorth> not me
22:24:33 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1378137032110506094/image.png?ex=683b81a1&is=683a3021&hm=4c1170b2f1a542c6e611c1777b6910fa44efb005bb8acf9e76e9d828fa0218fa&
22:24:51 <andythenorth> whatever this was supposed to be, is buried under an Iron Horse rewrite
22:25:02 <andythenorth> I'm sure it was a good idea at the time
22:25:27 <andythenorth> GS is under-used πŸ™‚
22:28:28 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14314: Add: Hover on graph legend to highlight line. https://github.com/OpenTTD/OpenTTD/pull/14314
22:36:16 <andythenorth> hmm so you're saying with #14314 I could stop setting FIRS payment rates just to space them out on the graph?
22:36:28 <andythenorth> there's an algorithm that forces them apart
22:36:33 <andythenorth> "such a balanced game"
22:36:50 <peter1138[d]> Uh...
22:37:46 <andythenorth> https://github.com/andythenorth/firs/blob/main/src/economy.py#L30
23:53:51 *** tateisukannanirase has joined #openttd
23:53:51 <tateisukannanirase> What IDE and debugging tools are people using these days? I want to try and fix the bus dt stop occupancy issue again but I will try and start with a bit more tooling than Sublime and a terminal window this time.