β΄ go to previous day
00:39:41 <DorpsGek> [OpenTTD/survey-web] ingstem closed issue #66: Weekly summary creation is failing https://github.com/OpenTTD/survey-web/issues/66
00:39:44 <DorpsGek> [OpenTTD/survey-web] ingstem commented on issue #66: Weekly summary creation is failing https://github.com/OpenTTD/survey-web/issues/66
00:48:05 <peter1138> 22:17 < yozora3> Wasn't what I was looking though, can't hide the badge name from vehicle description
00:48:10 <peter1138> I mean... isn't that what it does?
00:50:05 *** SigHunter has quit IRC (Remote host closed the connection)
00:51:07 *** SigHunter has joined #openttd
01:37:04 <yozora3> peter1138: Sorry forgot to update, it's a jgrpp specific bug, all works on vanilla
01:37:43 <peter1138> Might just not exist in jgrpp yet.
01:49:48 *** MinchinWeb[m] has quit IRC (Remote host closed the connection)
01:50:54 *** MinchinWeb[m] has joined #openttd
01:51:40 *** firepup650 has quit IRC (Ping timeout: 480 seconds)
02:43:13 *** MinchinWeb[m] has quit IRC (Ping timeout: 480 seconds)
02:50:05 *** MinchinWeb[m] has joined #openttd
03:05:14 *** Wormnest has quit IRC (Quit: Leaving)
04:00:46 *** tokai|noir has joined #openttd
04:00:46 *** ChanServ sets mode: +v tokai|noir
04:05:31 *** WormnestAndroid has quit IRC (Remote host closed the connection)
04:05:34 *** WormnestAndroid has joined #openttd
04:07:46 *** tokai has quit IRC (Ping timeout: 480 seconds)
04:49:03 <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/2d8cb5314c34af6c8f5be57868b3c8b6a2192110
04:49:04 <DorpsGek> - Update: Translations from eints (by translators)
06:05:38 *** Smedles has quit IRC (Quit: http://quassel-irc.org - Chat comfortably. Anywhere.)
06:05:54 *** Smedles has joined #openttd
06:24:28 <DorpsGek> [OpenTTD/OpenTTD] jnmbk opened pull request #15101: Change: Currency: update turkish lira value and symbol https://github.com/OpenTTD/OpenTTD/pull/15101
06:40:15 <DorpsGek> [OpenTTD/OpenTTD] Rito13 commented on pull request #12249: Add: [Script] Unbunch order flag https://github.com/OpenTTD/OpenTTD/pull/12249#pullrequestreview-3654099045
07:52:36 <peter1138> What an incoherent mess.
07:53:24 <reldred> Last night was that good, huh?
08:00:36 <LordAro> gesturing broadly at the world?
08:00:42 <LordAro> or a little more focused?
08:49:23 <DorpsGek> [OpenTTD/OpenTTD] nemmerich opened issue #15102: [Bug]: AI Script Breakout https://github.com/OpenTTD/OpenTTD/issues/15102
08:52:31 <LordAro> oh yeah, forgot about that
08:52:38 <LordAro> did see it at the time
09:13:31 <DorpsGek> [OpenTTD/OpenTTD] hantu85 commented on issue #15085: [Crash]: Crash on macOS starting OpenTTD with large interface scale https://github.com/OpenTTD/OpenTTD/issues/15085
09:18:10 <peter1138> The link is 404 for me, so that's my excuse.
09:18:16 <LordAro> mm
09:19:05 <LordAro> other two issues could be fixed though :)
09:19:22 <peter1138> What issues?
09:19:33 <LordAro> "additional issues"
09:19:53 <LordAro> underflow & negative indices
09:19:53 <peter1138> I don't know what you looking at.
09:20:16 <LordAro> oh, you're saying the advisory is a 404 link
09:20:19 <peter1138> Yes.
09:20:30 <LordAro> i'm saying the file attachment inside the advisory is a 404 :)
09:21:12 <peter1138> Probably want more than 1 person with access to advisories ;)
09:21:41 <LordAro> talltyler & _glx_ should be able to see it fine :p
09:21:42 <peter1138> I can see a draft from 2023, but nothing else.
09:22:27 <LordAro> let me have a look through the settings...
09:23:45 <LordAro> hmm, seems fixed unless i make you an admin
09:23:52 <LordAro> which i suppose i could... ;)
09:37:15 <DorpsGek> [OpenTTD/OpenTTD] jnmbk updated pull request #15101: Change: Currency: update turkish lira value and symbol https://github.com/OpenTTD/OpenTTD/pull/15101
09:37:24 <xarick> hi
09:42:39 <peter1138> You can add collaborators to advisories individually.
09:43:52 <DorpsGek> [OpenTTD/OpenTTD] jnmbk updated pull request #15101: Change: Currency: update turkish lira value and symbol https://github.com/OpenTTD/OpenTTD/pull/15101
09:49:03 <LordAro> so i can
09:49:12 <LordAro> try now
09:49:56 <LordAro> it's quite amusing to see such functions written out in squirrel
09:51:34 <xarick> oh, I see what Rito13 means
09:52:32 <xarick> but it's safeguarded in order_gui.cpp, at least last time I investigated this
09:54:09 <DorpsGek> [OpenTTD/OpenTTD] Release workflow was not successful https://github.com/OpenTTD/OpenTTD/actions/runs/20642598043
09:54:28 <peter1138> If you're trying to reply to a PR comment, try doing it on the PR comment itself.
09:55:00 <orudge> Logged into GOG and trying to see what's up with it, but looks like I can't just re-run the failed job for that now. They e-mailed us to ask why we hadn't published 15.0 basically.
09:56:10 <LordAro> orudge: the tl;dr is that the CI workflow we have no longer works, probably due to the "galaxy pipeline builder" we have available being too old
09:56:16 <LordAro> and being unable to find a newer version
09:56:24 <LordAro> if you've logged in, perhaps you can find a newer download link
09:56:38 <LordAro> https://docs.gog.com/pb-quick-start/ etc
09:57:00 <orudge> Yes, there's a version released in September
09:57:19 <LordAro> February 2022 might be too old :p
09:57:59 <LordAro> https://github.com/OpenTTD/OpenTTD/actions/runs/20642598043/job/59276828683#step:7:1 the tricky bit might be if there's no longer an "open" download link
09:58:56 <orudge> OK, hmm, the build creator doesn't seem to have changed since 2019.
09:59:00 <orudge> The Pipeline Builder is recommended tool now.
10:01:04 <orudge> OK, looks like it shouldn't be too difficult to fix I hope
10:01:36 <LordAro> \o/
10:02:21 <LordAro> 15.0 will need publishing manually though
10:02:24 <orudge> Yes
10:02:29 <orudge> I'm working on that just now
10:02:49 <LordAro> \o/
10:03:12 <orudge> I don't know who else sees the info@openttd.org e-mails apart from me, I know TrueBrain does but not sure who else has access these days
10:03:27 <LordAro> not many...
10:03:34 <LordAro> i think maybe talltyler?
10:03:40 <LordAro> (& frosch)
10:19:54 <talltyler> I do not receive those emails.
10:24:33 <orudge> OK, I've uploaded 15.0 to GOG and published it.
10:24:43 <orudge> I don't have GOG myself so can't really test it but can only hope and assume it's correct :D
10:24:55 <orudge> I'll submit a PR with the workflow changes
10:32:28 <DorpsGek> [OpenTTD/OpenTTD] orudge opened pull request #15103: Fix: [CI] Use the latest version of GOGGalaxyPipelineBuilder to upload releases to GOG https://github.com/OpenTTD/OpenTTD/pull/15103
10:34:26 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #15103: Fix: [CI] Use the latest version of GOGGalaxyPipelineBuilder to upload releases to GOG https://github.com/OpenTTD/OpenTTD/pull/15103#pullrequestreview-3655086378
10:34:54 <LordAro> nice
10:40:37 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1460584314570801327/image.png?ex=696772a4&is=69662124&hm=f7cc7e46e4341b9616c71060b921c660798c889bd0e577b2f449b6351f382a0b&
10:40:37 <xarick> This error never happens, I hacked the code to force an error to show just to make a point: I ctrl-click on a depot which makes an unbunch at depot order, but this is the error I would get with the 2 tier error message system
10:40:59 <xarick> it's not a station, it's a depot
10:56:57 <kuhnovic> Great, now you can fix a bug that doesn't exist
11:02:49 <peter1138> ^^
11:11:47 <DorpsGek> [OpenTTD/OpenTTD] orudge merged pull request #15103: Fix: [CI] Use the latest version of GOGGalaxyPipelineBuilder to upload releases to GOG https://github.com/OpenTTD/OpenTTD/pull/15103
11:12:54 <peter1138> orudge to the rescue... in basically 20 minutes :-)
11:13:21 <LordAro> how many months has that been failing for now? :p
11:15:43 <orudge> Now I just need to get around to trying to automate the Windows Store uploads, though last time I looked into that it involved setting up Active Directory gubbins and so on
11:15:53 <orudge> though given we now have that set up for Azure signing, I guess perhaps it wouldn't be so hard
11:16:26 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick commented on pull request #12249: Add: [Script] Unbunch order flag https://github.com/OpenTTD/OpenTTD/pull/12249#pullrequestreview-3655253041
11:17:02 <xarick> everyone is mad at me :|
11:17:33 <LordAro> nah.
11:17:37 <LordAro> frustrated? tired? sure.
11:18:45 <xarick> basically talltyler made a booboo which I'm trying to revert
11:18:49 <xarick> that is all
11:22:51 <xarick> this derailed slightly into "make AIs aware of 2 tier error messages"
11:23:18 <LordAro> perhaps it would be more productive to close this one and start again
11:30:14 <peter1138> > [Bus Route Builder] Unhandled error building bus stop: ERR_UNKNOWN Trying again.
11:30:48 <LordAro> that's oddly familiar
11:31:31 <peter1138> Well, it is AroAI :)
11:31:45 <LordAro> yeah, but i've not looked at it since 2012 :p
11:32:34 <peter1138> Then there's the AI that just spams bus stops everywhere.
11:43:41 *** Flygon has quit IRC (Read error: Connection reset by peer)
11:54:19 <LordAro> _glx_: feel like rebasing that other advisory PR?
11:57:09 <_glx_> Which one?
12:06:04 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1460605820717760557/B72L99CCrFH3AAAAAElFTkSuQmCC.png?ex=696786ac&is=6966352c&hm=ee9e0bb645a057c59f5f50025b05e728399d80ec3d0bba4134a085758f730740&
12:06:36 <orudge> OK, I can programmatically retrieve info on the MS Store submissions, though of course this tool uses JSON rather than the CSV that the web interface uses.
12:07:08 <orudge> I suspect for automation purposes we ideally just need to use default generic notes rather than specific notes for each version (which I don't think we can easily do via GitHub submissions)
12:07:23 <orudge> but hopefully it won't be too difficult to get this automated. I'll probably have to wait for 15.1 to test it out though. :P
12:11:08 <peter1138> Hmm, so.
12:13:01 <DorpsGek> [OpenTTD/website] LordAro opened pull request #371: Fix: Filter out security patches from security page https://github.com/OpenTTD/website/pull/371
12:15:50 <DorpsGek> [OpenTTD/website] PeterN approved pull request #371: Fix: Filter out security patches from security page https://github.com/OpenTTD/website/pull/371#pullrequestreview-3655483418
12:16:01 <DorpsGek> [OpenTTD/website] LordAro merged pull request #371: Fix: Filter out security patches from security page https://github.com/OpenTTD/website/pull/371
12:16:13 <LordAro> well that's one part of several
12:22:16 *** J_Darnley has joined #openttd
12:37:41 <_glx_> Will rebase the PR in next hours
12:37:48 <LordAro> <3
13:09:46 <ahyangyi> I love that files have names.
13:12:00 <DorpsGek> [OpenTTD/OpenTTD] abi9ail opened pull request #15104: Fix #15098: Map generation town road check in #12503, and associated Changes https://github.com/OpenTTD/OpenTTD/pull/15104
13:14:28 <LordAro> ahyangyi: :D
13:16:33 <peter1138> It wet.
13:18:32 <__abigail> abi9ailviaGitHub: I completely fucking forgot about my last PR, oops...
13:18:43 <LordAro> i love it when we get new developers
13:19:07 <LordAro> ...that's not intended to be read as sarcastic, in light of your comment
13:19:21 <__abigail> lmao
13:19:46 <__abigail> I am just (hopefully temporarily) unemployed with too much time on my hands and not enough to do in this shithole with nothing around
13:20:05 *** belajalilija has joined #openttd
13:20:05 <belajalilija> Yooo
13:20:10 <belajalilija> Youβre a dev now?
13:20:17 <LordAro> contributors*
13:20:25 <LordAro> one step at a time :p
13:20:29 <belajalilija> Both good
13:20:59 <DorpsGek> [OpenTTD/OpenTTD] Rito13 commented on pull request #12249: Add: [Script] Unbunch order flag https://github.com/OpenTTD/OpenTTD/pull/12249#pullrequestreview-3655767617
13:21:15 <__abigail> belajalilija: Depends... will you give me money? π₯Ί
13:25:26 <belajalilija> I wish i had money to give
13:27:42 <rito12_51026> kuhnovic: It exist in multiplayer
13:29:24 *** isosys has joined #openttd
13:29:25 <isosys> Hello everybody,
13:29:25 <isosys> Is there anywhere that explains WWT_MATRIX, and NWID_MATRIX? The differences between the two, the use case for each?
13:36:56 <peter1138> Not really. They are basically entirely different beasts.
13:37:46 <peter1138> NWID_MATRIX is basically only used by the common picker window now.
13:38:32 <peter1138> So if in doubt, you probably want WWT_MATRIX in most cases.
13:51:50 <_glx_> Hmm actually <https://github.com/OpenTTD/OpenTTD/pull/10883> is already applied so rebasing the advisory ends up with "warning: skipped previously applied commit 48716357ee"
13:56:53 <peter1138> So it should be closed?
13:59:45 <LordAro> or published
14:00:27 <LordAro> or a proper CVE post should be put together ;)
14:00:30 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #15101: Change: Currency: update turkish lira value and symbol https://github.com/OpenTTD/OpenTTD/pull/15101#issuecomment-3744494495
14:01:14 <_glx_> it's fixed since openttd 13.2
14:01:30 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #15101: Change: Currency: update turkish lira value and symbol https://github.com/OpenTTD/OpenTTD/pull/15101#issuecomment-3744499890
14:07:58 <peter1138> It was policy to not touch currencies, yes.
14:22:14 *** SigHunter has quit IRC ()
14:23:49 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick commented on pull request #12249: Add: [Script] Unbunch order flag https://github.com/OpenTTD/OpenTTD/pull/12249#pullrequestreview-3656083600
14:25:06 *** SigHunter has joined #openttd
14:32:44 <isosys> peter1138: Thanks. I guess I'll have to investigate and write it myself π
14:32:44 <isosys> I really thought they were similar, just one was easier to work with for certain scenarios/use cases.
14:33:24 <peter1138> I... don't know what you are talking about :)
14:34:10 <DorpsGek> [OpenTTD/OpenTTD] krizb8 opened issue #15105: [Crash]: When generating new map with czech names https://github.com/OpenTTD/OpenTTD/issues/15105
14:41:50 <_glx_> can confirm
14:43:08 <_glx_> oh yeah, OOB
14:44:06 <peter1138> Hm.
14:44:21 <peter1138> Array out of bounds or string formatter? I couldn't reproduce.
14:45:02 <_glx_> it asserts in basic_string::operator[] for me
14:45:26 <_glx_> it's inside MakeCzechTownName
14:45:57 <LordAro> i can't reproduce either (tried with a 4k map)
14:46:15 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on issue #15105: [Crash]: When generating new map with czech names https://github.com/OpenTTD/OpenTTD/issues/15105
14:46:54 <LordAro> stricter libstdc++ on fedora, perhaps?
14:47:01 <LordAro> (i'm on WSL Ubuntu)
14:47:10 <LordAro> fedora & windows*
14:47:56 <LordAro> mm, not a fun line of code
14:47:58 <_glx_> with postlen == 2 in my case
14:48:28 <_glx_> and poststr is "an"
14:48:45 <_glx_> so of course [2] is a problem
14:50:35 <LordAro> mm
14:51:48 <peter1138> The curse of switching from char[] to std::string_view, I suppose.
14:51:57 <_glx_> yeah
14:53:41 <_glx_> most likely started with https://github.com/OpenTTD/OpenTTD/pull/14113
14:56:17 <peter1138> Step 1) work out what the intention of the code is...
14:57:32 <_glx_> building the commit before the switch to see what it's supposed to do
14:57:46 <LordAro> looks like it actually started here: https://github.com/rubidium42/OpenTTD/commit/d0f9cc27a3b1709ca0a6a8b466f87be8d6e5f8a7#diff-cd834da270f35738b7886e564015f3f58c56521e41c699c96d7aa0fea85bb040R458-R462
14:57:51 <LordAro> :p
14:58:00 *** Wormnest has joined #openttd
14:58:04 <LordAro> oop, managed to link RB's fork
14:59:29 <_glx_> pff as always with "old" commits I need to cherry-pick a Win32 fix
15:02:15 <peter1138> LordAro, back then it would hit the NUL byte, instead of being OOB.
15:02:24 <peter1138> T
15:02:31 <LordAro> ah yes
15:03:48 <_glx_> yeah looks like an off by one in the check, should have been <=
15:05:27 <_glx_> I like when it crashes in unexpected places like TriggerAirportAnimation because grffile in nullptr but `var18_extra |= ats->grf_prop.grffile->cargo_map[cargo_type] << 8;`
15:05:46 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #15104: Fix #15098: Map generation town road check in #12503, and associated Changes https://github.com/OpenTTD/OpenTTD/pull/15104#issuecomment-3744853423
15:06:36 <LordAro> ...i hope that's unrelated to czech town names
15:07:49 <peter1138> Well, we could use .at() in more places.
15:09:23 <LordAro> exceptions tho
15:09:58 <_glx_> yeah it's already fixed with <https://github.com/OpenTTD/OpenTTD/commit/d85f6dccc504a5dcd7e2d7c5e87b17f099a39c15>
15:10:21 <_glx_> let's cherry-pick that too
15:11:27 <peter1138> Cherry pick for what?
15:13:23 <_glx_> to be able to run the commit before the string_view switch
15:14:12 <_glx_> and yeah it was pure luck with char* due to extra `\0`
15:15:06 <_glx_> funny how all postfix strings are always 2 chars long
15:15:37 <_glx_> this code is so weird
15:41:09 <_glx_> to me it never worked
15:44:17 <_glx_> and the `/* k-i -> c-i, h-i -> z-i */` never happens
15:45:53 <DorpsGek> [OpenTTD/OpenTTD] Rito13 updated pull request #15078: Codechange: Add docs for undocumented economy enum members and make them enum classes. https://github.com/OpenTTD/OpenTTD/pull/15078
15:51:33 *** MinchinWeb[m] has quit IRC (Ping timeout: 480 seconds)
15:51:55 *** MinchinWeb[m] has joined #openttd
15:54:53 *** tokai|noir has quit IRC (Quit: c('~' )o)
16:01:35 *** gelignite has joined #openttd
16:09:31 <DorpsGek> [OpenTTD/OpenTTD] Rito13 commented on pull request #12249: Add: [Script] Unbunch order flag https://github.com/OpenTTD/OpenTTD/pull/12249#pullrequestreview-3656631444
16:23:54 <xarick> someone agrees with me! impecable
16:24:13 <J_Darnley> Why can't I find a grf in the in-game downloader but I can see it on the bananas webpage?
16:27:02 <J_Darnley> Specifically this one https://bananas.openttd.org/package/newgrf/444410c1
16:28:27 <_glx_> it needs openttd 15
16:29:17 <_glx_> and previous versions are only for savegames
16:30:55 <J_Darnley> I see that now I click through to a specific "file"
16:50:02 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #15078: Codechange: Add docs for undocumented economy enum members and make them enum classes. https://github.com/OpenTTD/OpenTTD/pull/15078#pullrequestreview-3656833647
16:50:45 <DorpsGek> [OpenTTD/OpenTTD] nielsmh commented on pull request #15101: Change: Currency: update turkish lira value and symbol https://github.com/OpenTTD/OpenTTD/pull/15101#issuecomment-3745374333
16:53:25 <DorpsGek> [OpenTTD/OpenGFX2] outofcreativity commented on issue #235: new naming of OpenGFX2 files inside the archive breaks extract script https://github.com/OpenTTD/OpenGFX2/issues/235
16:55:13 <peter1138> CVS is a horrible version control system.
16:55:28 <LordAro> good ol openbsd
16:55:37 <LordAro> even freebsd has moved to git now
17:00:12 <DorpsGek> [OpenTTD/OpenTTD] jnmbk commented on pull request #15101: Change: Currency: update turkish lira value and symbol https://github.com/OpenTTD/OpenTTD/pull/15101#issuecomment-3745419933
17:00:12 *** MinchinWeb[m] has quit IRC (Remote host closed the connection)
17:01:25 *** MinchinWeb[m] has joined #openttd
17:02:36 <peter1138> > ${INSTALL_DATA} ${WRKSRC}/*.{grf,obg} ${PREFIX}/share/openttd/data/
17:05:47 <peter1138> So extracting it...
17:08:08 <LordAro> mm, still not sure why it's relevant
17:08:19 <LordAro> other than wanting to list the file being extracted explicitly i suppose
17:08:34 <LordAro> i can understand not wanting to distribute container formats
17:08:36 <LordAro> sort of
17:15:38 <DorpsGek> [OpenTTD/OpenTTD] jnmbk updated pull request #15101: Change: Currency: update turkish lira value and symbol https://github.com/OpenTTD/OpenTTD/pull/15101
17:17:57 *** gelignite has quit IRC ()
17:22:04 *** Wolf01 has joined #openttd
17:34:17 *** kuka_lie has joined #openttd
17:52:36 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #15106: Fix: [Script] Raise error if array index is not valid. https://github.com/OpenTTD/OpenTTD/pull/15106
18:07:32 <DorpsGek> [OpenTTD/OpenTTD] Rito13 updated pull request #15078: Codechange: Add docs for undocumented economy enum members and make them enum classes. https://github.com/OpenTTD/OpenTTD/pull/15078
18:14:09 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #15106: Fix: [Script] Raise error if array index is not valid. https://github.com/OpenTTD/OpenTTD/pull/15106#pullrequestreview-3657177875
18:14:58 <DorpsGek> [OpenTTD/OpenTTD] Rito13 updated pull request #15078: Codechange: Add docs for undocumented economy enum members and make them enum classes. https://github.com/OpenTTD/OpenTTD/pull/15078
18:17:36 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #15106: Fix: [Script] Raise error if array index is not valid. https://github.com/OpenTTD/OpenTTD/pull/15106#pullrequestreview-3657194245
18:18:58 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #15106: Fix: [Script] Raise error if array index is not valid. https://github.com/OpenTTD/OpenTTD/pull/15106#pullrequestreview-3657198580
18:24:38 <DorpsGek> [OpenTTD/OpenTTD] Rito13 opened pull request #15107: Cleanup: Remove client side checks for unbunch order. https://github.com/OpenTTD/OpenTTD/pull/15107
18:29:29 <_glx_> hmm actually except operator[] only validating via assert, other accesses seem to be protected
18:41:52 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #15078: Codechange: Add docs for undocumented economy enum members and make them enum classes. https://github.com/OpenTTD/OpenTTD/pull/15078#pullrequestreview-3657284837
18:42:59 <LordAro> yay consistency
18:44:43 <Rubidium> oh... I was just about to complain about some inconsistency...
18:45:56 <Rubidium> or maybe you don't have that inconsistency problem? In GH issues and PRs in the navigation bar have a count. Security does too when it's about CodeQL, but not the other stuff...
18:52:24 *** keimfrei has joined #openttd
18:52:24 <keimfrei> Greetings, One Questions. Where can i find the file with the high score entries? π
18:53:35 <DorpsGek> [OpenTTD/OpenTTD] zephyris commented on pull request #15101: Change: Currency: update turkish lira value and symbol https://github.com/OpenTTD/OpenTTD/pull/15101#issuecomment-3745871406
18:54:33 <_glx_> hs.dat in documents/openttd
19:10:25 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #15106: Fix: [Script] Raise error if array index is not valid. https://github.com/OpenTTD/OpenTTD/pull/15106
19:49:03 <peter1138> Ok
20:04:46 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #15099: Codechange: some documentation for enumerations https://github.com/OpenTTD/OpenTTD/pull/15099
20:06:49 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #15107: Cleanup: Remove client side checks for unbunch order. https://github.com/OpenTTD/OpenTTD/pull/15107#pullrequestreview-3657752097
20:13:58 *** MinchinWeb[m] has quit IRC (Read error: Connection reset by peer)
20:14:15 *** MinchinWeb[m] has joined #openttd
20:23:12 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #15106: Fix: [Script] Raise error if array index is not valid. https://github.com/OpenTTD/OpenTTD/pull/15106#pullrequestreview-3657807024
20:25:34 <DorpsGek> [OpenTTD/OpenTTD] LordAro merged pull request #15107: Codefix: Remove unnecessary client side checks for setting an unbunch order https://github.com/OpenTTD/OpenTTD/pull/15107
20:27:57 <LordAro> not sure if #15107 should be backported, but i've added the label
20:28:51 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #15106: Fix: [Script] Raise error if array index is not valid. https://github.com/OpenTTD/OpenTTD/pull/15106#pullrequestreview-3657825930
20:29:53 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #15106: Fix: [Script] Raise error if array index is not valid. https://github.com/OpenTTD/OpenTTD/pull/15106#pullrequestreview-3657829326
20:30:16 <LordAro> _glx_: feels like this one should have multiple reviews :)
20:34:28 <LordAro> and probably rebase-merged too
20:35:04 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #15099: Codechange: some documentation for enumerations https://github.com/OpenTTD/OpenTTD/pull/15099#pullrequestreview-3657844837
20:35:48 *** MinchinWeb[m] has quit IRC (Ping timeout: 480 seconds)
20:35:51 *** MinchinWeb[m] has joined #openttd
20:39:55 *** Flygon has joined #openttd
20:53:03 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #15106: Fix: [Script] Raise error if array index is not valid. https://github.com/OpenTTD/OpenTTD/pull/15106#pullrequestreview-3657899896
20:54:14 *** gelignite has joined #openttd
20:54:29 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #15099: Codechange: some documentation for enumerations https://github.com/OpenTTD/OpenTTD/pull/15099
20:59:07 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #15106: Fix: [Script] Raise error if array index is not valid. https://github.com/OpenTTD/OpenTTD/pull/15106#pullrequestreview-3657918836
21:07:15 <xarick> oh
21:11:44 <xarick> woah, just like that! Rito13 comes to the rescue
21:12:10 <xarick> rito12_51026: thx
21:13:41 <rito12_51026> Thanks Rubidium and LordAro
21:17:06 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #15106: Fix: [Script] Raise error if array index is not valid. https://github.com/OpenTTD/OpenTTD/pull/15106#pullrequestreview-3657976761
21:18:30 <xarick> visual studio 18.2.0 is out, more AI stuff maybe?
21:22:00 <DorpsGek> [OpenTTD/OpenTTD] Rito13 commented on pull request #15100: Codechange: Lower PNG compression level from 6 to 4 https://github.com/OpenTTD/OpenTTD/pull/15100#issuecomment-3746631599
21:24:52 *** gelignite has quit IRC ()
21:24:56 <LordAro> rito12_51026: are those sizes the wrong way round?
21:25:30 <LordAro> i would not expect the sizes to go down...
21:26:29 <LordAro> though if it's increasing that much i'm not sure it's a desirable change
21:35:11 <rito12_51026> LordAro: Yes they are I have mismatched them with the times, I will fix that
21:49:59 <peter1138> Okay, but do we compress the image in a separate thread?
21:50:38 <andythenorth> did we want more or fewer pixels posted? π
22:03:51 <peter1138> AIAI does like to place billions of road stops.
22:03:57 <peter1138> I wonder what error we're misreporting.
22:21:52 *** kuka_lie has quit IRC (Quit: Lost terminal)
22:23:16 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
22:25:14 <peter1138> Hmm, there are no town-specific script errors.
22:25:32 <peter1138> We have STR_ERROR_TOO_CLOSE_TO_EDGE_OF_MAP_SUB and STR_ERROR_TOO_CLOSE_TO_ANOTHER_TOWN
22:42:31 <DorpsGek> [OpenTTD/OpenTTD] jonnyawsom3 commented on pull request #15100: Codechange: Lower PNG compression level from 6 to 4 https://github.com/OpenTTD/OpenTTD/pull/15100#issuecomment-3746898027
22:44:00 <peter1138> An 8bpp user :)
23:16:25 <xarick> ctests suddenly taking longer to finish <https://gist.github.com/SamuXarick/019db9db1791b877d80bae69e6b11b45>
23:19:54 <xarick> microslop keeps ruining visual studio
23:22:14 <xarick> nearly 3 minutes...
23:22:26 <xarick> what used to be ~50 seconds
23:31:50 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
23:32:26 *** WormnestAndroid has joined #openttd
23:38:57 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #12249: Add: [Script] Unbunch order flag https://github.com/OpenTTD/OpenTTD/pull/12249
23:46:15 <xarick> I untouched order_cmd.cpp entirely. There is the minor caveat that Halt + Unbunch flags combined isn't being properly denied at the DoCommand level though... but I made sure this combo is not passed by script_order
23:58:40 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #15106: Fix: [Script] Raise error if array index is not valid. https://github.com/OpenTTD/OpenTTD/pull/15106#pullrequestreview-3658352871
continue to next day β΅