⏴ go to previous day
00:50:25 *** Koko has joined #openttd
00:52:57 <DorpsGek> [OpenTTD/grfcodec] PeterN merged pull request #59: Codechange: Replace AllocArray with vector<unique_ptr<>>. https://github.com/OpenTTD/grfcodec/pull/59
00:56:25 *** Koko has quit IRC (Quit: Page closed)
01:01:19 *** tokai|noir has joined #openttd
01:01:19 *** ChanServ sets mode: +v tokai|noir
01:08:19 *** tokai has quit IRC (Ping timeout: 480 seconds)
01:08:55 *** Wormnest has quit IRC (Quit: Leaving)
03:47:55 <mkdir7> In math_func.hpp there is a function:
03:47:55 <mkdir7> /**
03:47:55 <mkdir7> * Checks if a value is in an interval.
03:47:55 <mkdir7> *
03:47:55 <mkdir7> * Returns true if a value is in the interval of [min, max).
03:47:56 <mkdir7> *
03:47:56 <mkdir7> * @param x The value to check
03:47:58 <mkdir7> * @param min The minimum of the interval
03:47:58 <mkdir7> * @param max The maximum of the interval
03:48:00 <mkdir7> * @see IsInsideBS()
03:48:00 <mkdir7> */
03:48:02 <mkdir7> constexpr bool IsInsideMM(const size_t x, const size_t min, const size_t max) noexcept
03:48:02 <mkdir7> {
03:48:04 <mkdir7> return static_cast<size_t>(x - min) < (max - min);
03:48:04 <mkdir7> }
03:48:55 <mkdir7> But.. if `x=3, min=5, max=10` this will evaluate to true; `3-5=-2, 10-5=5 -> -2 < 5 => true`
03:49:19 <mkdir7> Is there some logic here I'm not understanding?
04:21:26 *** Zathras_1 has joined #openttd
04:21:34 *** Zathras_11 has joined #openttd
04:24:56 *** Zathras_4 has quit IRC (Ping timeout: 480 seconds)
04:24:56 *** Zathras_7 has quit IRC (Ping timeout: 480 seconds)
04:41:46 <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/fef0d830b245326382db0ee7dc41d445b1093be7
04:41:47 <DorpsGek> - Update: Translations from eints (by translators)
05:38:03 <DorpsGek> [OpenTTD/OpenTTD] Release workflow was not successful https://github.com/OpenTTD/OpenTTD/actions/runs/20052666249
06:34:06 <Rubidium> mkdir7: size_t is unsigned, so there won't be an intermediate -2 value
06:34:59 <mkdir7> Yea I didn't realize it was unsigned
06:35:25 <mkdir7> And then, I forgot unsigned isn't negative :RWkekdoge:
07:45:05 <LordAro> not immediately clear why it's not just `x >= min && x < max` though. is it trying to be clever and "beat" the compiler?
07:53:28 <Rubidium> I've not done any timing, but the codegen is at least different: https://godbolt.org/z/Yo9YdjYGe
07:57:09 <Rubidium> on ARM the codegen's different as well, though the opcount difference is swapped compare to x86-64: https://godbolt.org/z/rxKbWWKqM
08:37:58 <LordAro> neither seem to actually do anything "clever", the ops are just the code written out
09:11:06 <peter1138> It's "clever" in that it uses 1 compare instead of 2 compares.
09:12:22 <LordAro> i meant as far as the compiler's optimiser is concerned
09:12:42 <LordAro> but yeah, i guess that could factor in, doubt it's significant though with modern hardware
10:36:28 *** reldred has quit IRC (Quit: User went offline on Discord a while ago)
11:22:19 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1447911231942819961/image.png?ex=693957ea&is=6938066a&hm=0ed8cec17a27ceb498b42f07eb2c3e3198fad09c185e7e679b7e3fb799cc9875&
11:22:19 <xarick> progress
11:23:00 <LordAro> nicely presented :)
11:23:44 <xarick> lower is better
11:23:57 <xarick> I'm failing at erase without re-find
11:24:10 <xarick> 3.7 next step
11:25:25 <xarick> or rather, mr copilot is failing
11:30:07 <DorpsGek> [OpenTTD/OpenTTD] PikkaBird opened issue #14891: [Bug]: 'highest resolution sprites to use' seems to have an issue with newgrf roadtypes https://github.com/OpenTTD/OpenTTD/issues/14891
11:49:36 <peter1138> There's absolutely no way that can be feature-specific.
11:52:16 <kaibaneddy> maybe not, but it's where it's showing 🙂
13:20:25 <peter1138> `class PropData:public auto_array<PropData>`
13:20:35 <peter1138> Classic grfcodec. The class contains a pointer to an array of itself...
13:21:56 <LordAro> and classic lack of any spaces except what the language requires
13:24:39 <peter1138> Well, of course :-)
13:25:07 <LordAro> definitely get the impression it'd be `classPropData:publicauto_array<PropData>` if it was allowed
13:28:10 <DorpsGek> [OpenTTD/OpenTTD] paulcobbaut commented on issue #14889: [Bug]: fluidsynth error on stderr https://github.com/OpenTTD/OpenTTD/issues/14889
13:29:50 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #14889: [Bug]: fluidsynth error on stderr https://github.com/OpenTTD/OpenTTD/issues/14889
13:32:10 *** Flygon has quit IRC (Read error: Connection reset by peer)
13:46:32 <_glx_> https://cdn.discordapp.com/attachments/1008473233844097104/1447947528069845022/image.png?ex=693979b8&is=69382838&hm=1a48c6fdc6b0ff31f49c529bda31648d7d04675c72a7946391e3b6005b421b81&
13:46:32 <_glx_> vcpkg translations are so bad
13:46:50 <peter1138> https://groups.google.com/a/list.nist.gov/g/internet-time-service/c/Zd7VaR-vqV4
13:47:21 <peter1138> Translating command lines?
13:49:21 <_glx_> yeah AI is not smart for translations
13:50:03 <peter1138> "AI is not smart" was enuogh.
13:51:25 <xarick> oh no copilot got tired of working
13:51:35 <xarick> page doesnt load 🙂
14:11:09 <LordAro> peter1138: well that's exciting
14:31:39 <LordAro> Rubidium: just learnt about the analysis feature of compiler explorer https://godbolt.org/z/Pj7K7h46v
14:38:01 <peter1138> Wtf, this guy. Gave him the VPN settings and payment of the config.
14:38:07 <peter1138> He puts the details into Remote Desktop client...
14:44:13 <DorpsGek> [OpenTTD/OpenTTD] Rito13 commented on pull request #14365: Feature: Gamepad viewport scrolling for Windows and Linux https://github.com/OpenTTD/OpenTTD/pull/14365#pullrequestreview-3557440493
14:49:45 <mkdir7> LordAro: Honestly- that was my initial impression; the code itself seemed as if it was doing "extra work" for a simple range check
14:50:09 <mkdir7> SO at first I was thrown off- but thats why I deleted my message after I realized its unsigned its
14:50:16 <mkdir7> But yea- it was a little offputting
14:50:37 <mkdir7> It works and I can see the justification because it *is* a simple formula, its just not what I'm used to seeing
14:53:20 <peter1138> s/payment/picture/
15:21:43 <Rubidium> LordAro: can you explain which of the two is objectively better? :D
15:21:50 <LordAro> Rubidium: nope!
15:29:09 *** Zathras_1 has quit IRC (Quit: Connection reset by beer!)
15:34:13 <Rubidium> LordAro: ah... AoCO :D
15:34:22 *** gelignite has joined #openttd
15:34:32 <LordAro> :D
15:39:12 <DorpsGek> [OpenTTD/actions] glx22 opened pull request #108: Add: [setup-vcpkg] action to setup vcpkg https://github.com/OpenTTD/actions/pull/108
15:39:53 <_glx_> grfcodec will need it
15:42:32 <DorpsGek> [OpenTTD/actions] LordAro commented on pull request #108: Add: [setup-vcpkg] action to setup vcpkg https://github.com/OpenTTD/actions/pull/108#pullrequestreview-3558152113
15:46:02 <DorpsGek> [OpenTTD/actions] glx22 commented on pull request #108: Add: [setup-vcpkg] action to setup vcpkg https://github.com/OpenTTD/actions/pull/108#pullrequestreview-3558180945
15:52:53 <LordAro> curious CI issue
15:52:58 <LordAro> your branch looks up to date...
15:55:22 <_glx_> yeah it's weird
15:56:42 <DorpsGek> [OpenTTD/actions] glx22 updated pull request #108: Add: [setup-vcpkg] action to setup vcpkg https://github.com/OpenTTD/actions/pull/108
16:01:21 <peter1138> Hmm, I guess we could also check if the fluidsynth system config points to a valid file.
16:18:14 <DorpsGek> [OpenTTD/actions] glx22 updated pull request #108: Add: [setup-vcpkg] action to setup vcpkg https://github.com/OpenTTD/actions/pull/108
16:26:57 *** gelignite has quit IRC ()
16:53:30 <rito12_51026> rito12_51026: The mapping is wrong, the y axis is bound to right trigger and the x axis to y axis of the right stick.
16:53:30 <rito12_51026> I get the same issue in SuperTuxKart 😄 and I didn't 😭
17:02:09 <DorpsGek> [OpenTTD/actions] glx22 commented on pull request #108: Add: [setup-vcpkg] action to setup vcpkg https://github.com/OpenTTD/actions/pull/108#pullrequestreview-3558371524
17:02:31 <DorpsGek> [OpenTTD/grfcodec] rubidium42 commented on pull request #58: Remove boost dependency https://github.com/OpenTTD/grfcodec/pull/58#pullrequestreview-3558417405
17:04:26 <DorpsGek> [OpenTTD/grfcodec] LordAro commented on pull request #58: Remove boost dependency https://github.com/OpenTTD/grfcodec/pull/58#pullrequestreview-3558564765
17:09:06 <DorpsGek> [OpenTTD/actions] glx22 updated pull request #108: Add: [setup-vcpkg] action to setup vcpkg https://github.com/OpenTTD/actions/pull/108
17:27:39 <DorpsGek> [OpenTTD/grfcodec] PeterN opened pull request #60: Fix d5a7b850: Action 4 data broken for nforenum. https://github.com/OpenTTD/grfcodec/pull/60
17:29:05 <peter1138> These data formats are entirely... Why. Why would you design it like that?
17:30:04 <_glx_> <https://github.com/glx22/OpenTTD/pull/38/files> testing the action, actually the outputs could be toolchain and exe
17:30:22 <_glx_> and not just location
17:38:31 *** Wolf01 has joined #openttd
17:45:30 <LordAro> peter1138: why are we suddenly putting any effort into grfcodec, ooi?
17:49:10 <DorpsGek> [OpenTTD/actions] glx22 updated pull request #108: Add: [setup-vcpkg] action to setup vcpkg https://github.com/OpenTTD/actions/pull/108
17:58:15 <peter1138> Well, there was a bug report against nforenum which we can't replicate, so I, uh...
17:58:27 <peter1138> And then you came along and removed boost.
17:58:41 <LordAro> i did do that, yes
17:58:51 <Rubidium> let me guess... it's not really nice weather to ride on your bikes?
17:59:04 <LordAro> it's pretty awful, yes
17:59:12 <peter1138> Also OpenTTD uses nforenum when building its GRFs, so handy if it's not randomly broken.
17:59:54 <peter1138> Also I have some refactors for OpenTTD which would be silly to do so close to branching :D
18:13:40 <DorpsGek> [OpenTTD/grfcodec] rubidium42 commented on pull request #60: Fix d5a7b850: Action 4 data broken for nforenum. https://github.com/OpenTTD/grfcodec/pull/60#issuecomment-3633604623
18:28:04 <DorpsGek> [OpenTTD/actions] glx22 updated pull request #108: Add: [setup-vcpkg] action to setup vcpkg https://github.com/OpenTTD/actions/pull/108
18:31:12 <DorpsGek> [OpenTTD/actions] glx22 updated pull request #108: Add: [setup-vcpkg] action to setup vcpkg https://github.com/OpenTTD/actions/pull/108
18:35:22 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
18:45:38 <DorpsGek> [OpenTTD/actions] glx22 updated pull request #108: Add: [setup-vcpkg] action to setup vcpkg https://github.com/OpenTTD/actions/pull/108
19:14:08 <DorpsGek> [OpenTTD/grfcodec] PeterN updated pull request #60: Fix d5a7b850: Action 4 data broken for nforenum. https://github.com/OpenTTD/grfcodec/pull/60
19:16:41 <DorpsGek> [OpenTTD/grfcodec] rubidium42 approved pull request #60: Fix d5a7b850: Action 4 data broken for nforenum. https://github.com/OpenTTD/grfcodec/pull/60#pullrequestreview-3559273735
19:23:29 <DorpsGek> [OpenTTD/grfcodec] PeterN merged pull request #60: Fix d5a7b850: Action 4 data broken for nforenum. https://github.com/OpenTTD/grfcodec/pull/60
19:27:58 *** Zathras_4 has joined #openttd
20:02:16 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1448042082500477201/image.png?ex=6939d1c7&is=69388047&hm=7a893cabc90b0855ce1078492faddf5a8a07a30ad47e193d12b5939aaab43b38&
20:02:16 <xarick> owah
20:02:22 <xarick> it's approaching google's
20:05:25 <xarick> visual studio 18.1.0 is out
20:05:50 <peter1138> Now 30% more slop than the existing 30% slop?
20:06:09 <xarick> 🙂
20:08:38 <Rubidium> when's the slop too sloppy?
20:09:16 <andythenorth> this is sort of the future
20:09:30 <andythenorth> yolo vibe code by smushing all the previous code
20:09:43 <andythenorth> if it's faster on benchmarks....ship it
20:10:23 <peter1138> Rubidium, right at the beginning.
20:12:24 <andythenorth> I thought the prediction was we'd have AGI by now
20:12:29 <andythenorth> that was about a year ago
20:13:27 <andythenorth> yeah OpenAI had a clear path to AGI by end of 2025
20:13:30 <andythenorth> "lolwut"
20:13:39 <andythenorth> that was in late 2024
20:13:50 <audigex> They still have 3 weeks, to be fair
20:13:54 <andythenorth> true
20:14:04 <audigex> You’re gonna feel silly if they release it on the 29th
20:14:06 <DorpsGek> [OpenTTD/OpenTTD] TheMowgliMan opened issue #14892: [Crash]: Crash when Abandoning Game https://github.com/OpenTTD/OpenTTD/issues/14892
20:14:43 <andythenorth> I will feel very silly yes
20:14:50 <ahyangyi> Deadline rush during Christmas?
20:15:24 <jfkuayue> there are exams just after christmas... for me
20:15:36 <andythenorth> if AGI arrives you won't be needed
20:16:02 <DorpsGek> [OpenTTD/actions] glx22 updated pull request #108: Add: [setup-vcpkg] action to setup vcpkg https://github.com/OpenTTD/actions/pull/108
20:16:05 <andythenorth> my advice would be....assume the exams will happen 😛
20:16:12 <jfkuayue> *not meaning dec 26
20:16:20 <ahyangyi> andythenorth: How is that any different though 😛
20:16:24 <xarick> clang builds still errors about manifest file 🙁
20:16:31 <peter1138> That "if" is doing some heavy lifting.
20:16:47 <andythenorth> ahyangyi: what's the question?
20:17:58 <jfkuayue> https://cdn.discordapp.com/attachments/1008473233844097104/1448046036462342385/image.png?ex=6939d576&is=693883f6&hm=bd6c6612d543e9fa809dbee92db4dded7da71657e697824a5e8a12f005270f6b&
20:17:58 <jfkuayue> is this pretty bad for IRC people
20:20:27 <peter1138> Bad?
20:20:45 <jfkuayue> because JP+ is pretty bad.
20:20:55 <peter1138> It's missing traffic.
20:21:02 <peter1138> Hah.
20:21:52 *** Wormnest has joined #openttd
20:22:08 <peter1138> JP+ Tracks. And no, the result is good.
20:22:29 <jfkuayue> I'm glad.
20:22:37 <peter1138> The dropdown menu for railtype selection with JP+ is pretty bad, because it doesn't make sense.
20:22:44 <jfkuayue> *Opened a big pack of prawn crackers*
20:23:40 <peter1138> Text filter helps. Badge filter might help. Dunno about the hierarchical stuff, that felt a bit awkward to use, but possibly because I'd applied it on top of my text filter branch and it's kinda weird on that.
20:23:59 <locosage> at this point it's probably safe to assume we'll get AGI at about the same time as fusion power 🤭
20:25:31 <andythenorth> and the year of the linux desktop
20:25:45 <andythenorth> and newgrf airports with player defined state machines
20:27:26 <jfkuayue> https://cdn.discordapp.com/attachments/1008473233844097104/1448048417015070835/image.png?ex=6939d7ae&is=6938862e&hm=e87551be7e245f5778f37e7f93ace4cf834a9bace564538d966a5ec3ecc26b93&
20:27:26 <jfkuayue> linux desktop with nonsense.
20:31:55 *** reldred has joined #openttd
20:31:55 <reldred> Old version, the track menu is cleaned up a bit now
20:34:21 <_zephyris> The way things are going, it'll need fusion power to have enough energy
20:35:44 <jfkuayue> 0.68.2 is also the same track menu...?
20:35:56 <reldred> No, jp+ tracks
20:36:11 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #14891: [Bug]: 'highest resolution sprites to use' seems to have an issue with newgrf roadtypes https://github.com/OpenTTD/OpenTTD/issues/14891
20:37:22 <jfkuayue> I have v0.5.9 though
20:53:19 <jfkuayue> *finished a big pack of prawn crackers*
21:04:08 <Rubidium> LordAro: are you allowed to merge https://github.com/OpenTTD/bananas-api/pull/524 (I'm not)
21:05:25 <DorpsGek> [OpenTTD/actions] glx22 commented on pull request #108: Add: [setup-vcpkg] action to setup vcpkg https://github.com/OpenTTD/actions/pull/108#issuecomment-3634281975
21:09:33 <_glx_> squash or rebase rubidium ?
21:16:12 <Rubidium> I'd say rebase
21:16:32 <DorpsGek> [OpenTTD/bananas-api] glx22 merged pull request #524: Fix #499, #523: allow license.md/changelog.md/readme.md as well as metadata:url for base sets https://github.com/OpenTTD/bananas-api/pull/524
21:16:38 <DorpsGek> [OpenTTD/bananas-api] glx22 closed issue #499: Changelog, readme, and license cannot be .md https://github.com/OpenTTD/bananas-api/issues/499
21:16:41 <DorpsGek> [OpenTTD/bananas-api] glx22 closed issue #523: Baseset upload issues https://github.com/OpenTTD/bananas-api/issues/523
21:20:07 <DorpsGek> [OpenTTD/OpenTTD] OpenCharlie commented on pull request #14868: Improved Water Edges https://github.com/OpenTTD/OpenTTD/pull/14868#issuecomment-3634326485
21:29:49 <Rubidium> thanks _glx_
21:32:23 <_zephyris> Thank you too!
21:57:40 <xarick> 448 seconds
21:57:56 <xarick> don't know where I can extract more performance
22:00:01 <xarick> there's a lot of redudancy yet
22:00:22 <xarick> and tons of validation code, but I have put them under asserts
22:01:48 <xarick> going to experiment different branching factor values
22:12:27 <xarick> 64 still the sweet spot 😐
22:13:49 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1448075189027078144/image.png?ex=6939f09d&is=69389f1d&hm=9603387a252483899bddba0c2e4ad8f487f9d82b2a74d82a05696bc38b1fd416&
22:13:49 <xarick> 1874 lines of slop
22:29:00 *** Flygon has joined #openttd
22:39:38 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
23:14:11 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14893: Add: Setting to control minimum distance between towns. https://github.com/OpenTTD/OpenTTD/pull/14893
23:26:49 *** toktik has quit IRC (Remote host closed the connection)
23:27:39 *** toktik has joined #openttd
23:43:40 *** toktik is now known as Guest33253
23:43:47 *** toktik has joined #openttd
23:45:33 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #14893: Add: Setting to control minimum distance between towns. https://github.com/OpenTTD/OpenTTD/pull/14893#pullrequestreview-3560161707
23:50:16 *** Guest33253 has quit IRC (Ping timeout: 480 seconds)
continue to next day ⏵
