⏴ go to previous day
00:32:28 <DorpsGek> [OpenTTD/OpenTTD] Rito13 updated pull request #14798: Feature: Hidden and favourite rail/road/tram types. https://github.com/OpenTTD/OpenTTD/pull/14798
02:12:37 *** Zathras_4 has joined #openttd
02:25:36 *** Flygon has joined #openttd
02:42:49 *** gelignite is now known as Guest32010
02:42:52 *** gelignite has joined #openttd
02:50:09 *** Guest32010 has quit IRC (Ping timeout: 480 seconds)
02:56:32 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
02:56:34 *** WormnestAndroid has joined #openttd
02:59:27 *** Wormnest has quit IRC (Quit: Leaving)
03:45:46 *** gnu_jj_ has joined #openttd
03:49:05 *** gnu_jj has quit IRC (Ping timeout: 480 seconds)
04:13:52 *** Zathras_11 has joined #openttd
04:14:10 *** Zathras_1 has joined #openttd
04:17:10 *** Zathras has quit IRC (Ping timeout: 480 seconds)
04:17:34 *** Zathras_4 has quit IRC (Ping timeout: 480 seconds)
04:43:11 <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/7ffd3b2c480ea07ac2a2b8cb8accc4f027030e5f
04:43:12 <DorpsGek> - Update: Translations from eints (by translators)
05:56:58 *** WormnestAndroid has quit IRC (Remote host closed the connection)
05:57:01 *** WormnestAndroid has joined #openttd
06:40:59 *** tokai has joined #openttd
06:40:59 *** ChanServ sets mode: +v tokai
06:47:55 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)
06:55:23 *** gelignite has quit IRC ()
07:23:23 <DorpsGek> [OpenTTD/OpenTTD] Rito13 updated pull request #14798: Feature: Hidden and favourite rail/road/tram types. https://github.com/OpenTTD/OpenTTD/pull/14798
07:40:53 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
07:45:06 *** WormnestAndroid has joined #openttd
08:07:51 *** gelignite has joined #openttd
08:08:52 <DorpsGek> [OpenTTD/OpenTTD] Rito13 updated pull request #14717: Feature: New selection for rail types. https://github.com/OpenTTD/OpenTTD/pull/14717
08:33:42 *** Zathras_1 has quit IRC (Quit: Connection reset by beer!)
08:34:07 <DorpsGek> [OpenTTD/OpenTTD] Rito13 updated pull request #14717: Feature: New selection for rail types. https://github.com/OpenTTD/OpenTTD/pull/14717
08:47:08 <DorpsGek> [OpenTTD/OpenTTD] Rito13 updated pull request #14717: Feature: New selection for rail types. https://github.com/OpenTTD/OpenTTD/pull/14717
08:48:02 <rito12_51026> peter1138 I have fixed all deviations from the coding style that I could find, it should be all corect now.
09:05:38 *** keikoz has quit IRC ()
09:07:41 *** keikoz has joined #openttd
09:08:41 <DorpsGek> [OpenTTD/OpenTTD] mmtunligit commented on pull request #14813: Add: User-defined groups for saved items in the picker window https://github.com/OpenTTD/OpenTTD/pull/14813#issuecomment-3567653849
09:28:43 <DorpsGek> [OpenTTD/OpenTTD] github-advanced-security[bot] commented on pull request #14717: Feature: New selection for rail types. https://github.com/OpenTTD/OpenTTD/pull/14717#pullrequestreview-3497548302
09:42:33 <DorpsGek> [OpenTTD/team] MiguelNTNU opened issue #675: [gl_ES] Translator access request https://github.com/OpenTTD/team/issues/675
11:49:34 <xarick> hi
13:12:45 <xarick> i hate unpredictable performance 🙁
13:45:41 <DorpsGek> [OpenTTD/team] glx22 commented on issue #675: [gl_ES] Translator access request https://github.com/OpenTTD/team/issues/675
13:48:30 <DorpsGek> [OpenTTD/team] glx22 commented on issue #674: [es_ES] Translator access request https://github.com/OpenTTD/team/issues/674
13:48:53 <DorpsGek> [OpenTTD/team] glx22 commented on issue #673: [eo_EO] Translator access request https://github.com/OpenTTD/team/issues/673
13:49:32 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #14803: Change: Ensure generated towns have enough room https://github.com/OpenTTD/OpenTTD/pull/14803#issuecomment-3567975612
14:00:03 <peter1138> I need a pint to recover :o
14:12:43 <andythenorth> Me too
14:12:50 <andythenorth> From what though?
14:17:52 <peter1138> Being awake.
14:21:39 <mmtunligit> peter have you seen my question on gh? im at the point where loading from ini is the only revision i have left to do
14:49:10 <mmtunligit> actually, i should make the list, the new ini groups do not get placed as i was expecting
14:52:26 <xarick> > E:\Builds\openttd 20251031-scriptlist>openttd.exe -d -v null:ticks=1000000 -s null -m null -g -G 1 -x -QQ
14:52:26 <xarick> > [2025-11-23 14:46:21] dbg: [script:0] The script died unexpectedly.
14:52:26 <xarick> >
14:52:26 <xarick> > E:\Builds\openttd 20251031-scriptlist>openttd.exe -d -g -G 1
14:52:26 <xarick> > [2025-11-23 14:50:21] dbg: [misc:0] [GameTick] [AddOrSetItem] 1 calls in 2330131 us [avg: 2330131.0 us]
14:52:27 <xarick> > [2025-11-23 14:50:57] dbg: [script:0] The script died unexpectedly.
14:52:27 <xarick> Why is this not printing the misc line with v null?
15:02:50 <xarick> ah, i got it
15:03:25 <xarick> `static TicToc::State AddOrSetItem("AddOrSetItem", 1);` works
15:03:25 <xarick> `static TicToc::State AddOrSetItem("AddOrSetItem");` doesn't work
15:26:00 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1442174352450785441/image.png?ex=69247907&is=69232787&hm=725abed53007bb49e79aac6c8c842141a839531cd23588fcee9476487966fdbb&
15:26:00 <xarick> okay, the benefits from try_emplace are there, but
15:27:03 <xarick> each dbg line means adding 10 million items to the previous amount, starting at 10 million
15:27:39 <xarick> the 5th dgb line means base list has 50 million items, and adds 10 more million items to it
15:27:46 <xarick> unrealistic 🙁
15:30:13 <xarick> gonna try a different test
15:45:13 *** Wormnest has joined #openttd
15:52:43 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
15:58:22 *** kuka_lie has joined #openttd
16:08:18 <DorpsGek> [OpenTTD/OpenTTD] zephyris commented on pull request #14803: Change: Ensure generated towns have enough room https://github.com/OpenTTD/OpenTTD/pull/14803#pullrequestreview-3497872520
16:24:30 <mmtunligit> what am i doing wrong here?
16:24:30 <mmtunligit> ```std::set<std::string> collections;
16:24:30 <mmtunligit> std::string_view type = callbacks.ini_group + "-";
16:24:30 <mmtunligit> size_t location = callbacks.ini_group.size() + 2;
16:24:30 <mmtunligit> for (auto it = ini.groups.begin(); it != ini.groups.end(); it++) {
16:24:31 <mmtunligit> std::string_view name = it->name;
16:24:31 <mmtunligit> if (name.find(type) != 0) continue;
16:24:33 <mmtunligit> if (name[location] == '*') collections.insert(it->name);
16:24:33 <mmtunligit> }```
16:24:35 <mmtunligit> im trying to make a list of all the ini groups that match the ini group of the current callback with the wildcard but the final if never seems to go through even when there is a match
16:26:09 <peter1138> You can't use string_view there.
16:26:26 <peter1138> (The first one outside the loop.)
16:28:01 <mmtunligit> ah thanks, it compiled fine so i was real confused
16:34:13 <mmtunligit> hmm, its still not working even after makint type a std::string
16:42:34 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler updated pull request #14803: Change: Ensure generated towns have enough room https://github.com/OpenTTD/OpenTTD/pull/14803
16:43:53 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #14803: Change: Ensure generated towns have enough room https://github.com/OpenTTD/OpenTTD/pull/14803#pullrequestreview-3497886474
16:45:56 *** WormnestAndroid has joined #openttd
16:46:57 <mmtunligit> wait i dont need to do this bs at all, im already filtering out everything that wouldnt match
16:53:41 <xarick> there is a certain amount of elements a std::map can have until clearing becomes unbearably slow
16:59:47 <peter1138> Don't put so many elements in it then.
17:21:11 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1442203340640620707/image.png?ex=69249407&is=69234287&hm=7c0c8bb410a03923cd6476da99e06b5ad99f07c26ea67f137b750b2cc9acedd9&
17:21:11 <xarick> for this test, it tried a mix of existing items and non-existing items
17:21:55 <xarick> try_emplace does better... but this is dealing with ~30m to 180~ million items
17:23:04 <xarick> this is for AddList
17:23:25 <xarick> when the main list isn't empty
17:24:24 <xarick> next test: try smaller, realistic sized lists
17:24:36 <xarick> 10 items
17:47:24 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1442209937965977730/image.png?ex=69249a2c&is=692348ac&hm=67453d53ad183d275ee0efe14525389a19fd065bbfbb25c3e2b2a03569192768&
17:47:24 <xarick> Hint loses
17:57:38 <_glx_> unrealistic number of items
18:00:02 <_glx_> I think max number of vehicles per company might a realistic max size for tests
18:01:13 <_glx_> or the max ignoring company limit if GS
18:02:53 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #12088: Add: [Script] GetAirportPlaneCompatibility https://github.com/OpenTTD/OpenTTD/pull/12088#pullrequestreview-3497919775
18:03:38 *** Flygon has quit IRC (Quit: A toaster's basically a soldering iron designed to toast bread)
18:09:20 <rito12_51026> > "openttd" received signal SIGFPE, Arithmetic exception.
18:09:20 <rito12_51026> Can't divide by zero 🙁
18:09:45 <_glx_> yes that's expected
18:13:49 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1442216585174581310/image.png?ex=6924a05c&is=69234edc&hm=5fd7c66fa5375992b01d08d798f9161d057a8af243afad9c947665a55d8c3c94&
18:13:49 <xarick> my tests are flawed
18:14:46 <rito12_51026> Should it be allowed to hide all rail types?
18:16:53 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1442217357811515553/image.png?ex=6924a115&is=69234f95&hm=69dfaeb7a7a605772702faa32373f5d33e16a2b4f6924916537204cc364f8676&
18:21:04 <xarick> 5000 * 4 * 15 let's test
18:33:50 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1442221622122643466/image.png?ex=6924a50d&is=6923538d&hm=6fe5284e0d58aca057182bec193632aeedecf7f44f395ca605851dcce679ce27&
18:33:58 <xarick> hint wins
18:39:34 <xarick> what should be the initial iterator for hint?
18:39:48 <xarick> std::end or std::begin
18:40:30 <DorpsGek> [OpenTTD/OpenTTD] Rito13 updated pull request #14798: Feature: Hidden and favourite rail/road/tram types. https://github.com/OpenTTD/OpenTTD/pull/14798
18:41:04 <xarick> *it depends*
18:41:07 <xarick> ?
18:47:01 <DorpsGek> [OpenTTD/OpenTTD] Rito13 commented on pull request #14798: Feature: Hidden and favourite rail/road/tram types. https://github.com/OpenTTD/OpenTTD/pull/14798#issuecomment-3568233465
19:12:24 *** Wolf01 has joined #openttd
19:42:26 <xarick> I've been thinking
19:43:00 <xarick> would a Bulk Inserter function help speed up things?
19:43:38 <xarick> for stuff that are sorted, like indexes
19:45:01 <andythenorth> is cat?
19:45:25 <xarick> or maybe rework FillList
19:45:36 <xarick> adapt it a little bit
19:49:11 <DorpsGek> [OpenTTD/OpenTTD] zephyris commented on pull request #14803: Change: Ensure generated towns have enough room https://github.com/OpenTTD/OpenTTD/pull/14803#issuecomment-3568277349
20:01:58 <_zephyris> _glx_: Happy to help with an NML release if I can, if it's the tedious things like making a changelog which are the issue
20:10:03 <xarick> industry accepted cargoes are a vector...
20:10:48 <xarick> are the cargo types in the vector inserted in ascending order?
20:12:12 <peter1138> They are not sorted by cargo type, if that's what you're asking.
20:12:22 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1442246418126475334/image.png?ex=6924bc25&is=69236aa5&hm=162841d83a072655e6689a20510930e3f7f5e314bd10cf5e15639e2c810a7101&
20:12:22 <xarick> I'm doing some dumb stuff like this, but I expect ascending order during iteration
20:12:39 <xarick> okay :/ t.t
20:24:31 <_glx_> _zephyris: usually that's the annoying part, but nml is released often enough so it's not too hard
20:28:51 <DorpsGek> [OpenTTD/OpenGFX2] zephyris updated pull request #215: Fix: Rear 1x zoom suspension bridge overlapped bridge deck https://github.com/OpenTTD/OpenGFX2/pull/215
20:33:55 <DorpsGek> [OpenTTD/OpenGFX2] PeterN approved pull request #215: Fix: Rear 1x zoom suspension bridge overlapped bridge deck https://github.com/OpenTTD/OpenGFX2/pull/215#pullrequestreview-3497979765
20:40:02 <DorpsGek> [OpenTTD/OpenGFX2] zephyris merged pull request #215: Fix: Rear 1x zoom suspension bridge overlapped bridge deck https://github.com/OpenTTD/OpenGFX2/pull/215
20:40:07 <DorpsGek> [OpenTTD/OpenGFX2] zephyris closed issue #214: Concrete bridge railing clips with road https://github.com/OpenTTD/OpenGFX2/issues/214
20:44:24 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1442254482296541246/image.png?ex=6924c3a8&is=69237228&hm=8615cf938bcc0d70eebda1e63e37d94bc524694843446cca1aeedf82263ac16a&
20:44:24 <xarick> not sure about these lambdas
20:51:18 <_glx_> not sure hint will help, shared vehicles are not ordered
20:53:11 <xarick> is there a situation where try_emplace will do worse than emplace if I give bad hints?
20:53:22 <xarick> mr copilot suggests that nope
20:55:15 <_glx_> in general emplace_hint is the same as emplace, unless you are lucky with the hint
20:56:14 <xarick> oh, emplace_hint exists, new toy
20:56:59 <_glx_> you should really check cpp documentation
20:58:22 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14817: Codefix: Remove this-> from static call to CleaningPool. https://github.com/OpenTTD/OpenTTD/pull/14817
21:01:06 *** gelignite has quit IRC ()
21:01:24 <_glx_> https://en.cppreference.com/w/cpp/container/map/emplace_hint.html seems it's worse for maps
21:10:42 <DorpsGek> [OpenTTD/OpenTTD] mmtunligit updated pull request #14813: Add: User-defined groups for saved items in the picker window https://github.com/OpenTTD/OpenTTD/pull/14813
21:13:03 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14817: Codefix: Remove this-> from static call to CleaningPool. https://github.com/OpenTTD/OpenTTD/pull/14817#pullrequestreview-3498003951
21:26:08 *** Zathras_11 has quit IRC (autokilled: This host violated network policy. Mail support@oftc.net if you think this is in error. (2025-11-23 21:26:08))
21:27:03 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14818: Add: [NewGRF] Allow badges to be excluded from badge name list. https://github.com/OpenTTD/OpenTTD/pull/14818
21:35:48 <andythenorth> audigex: ^
21:36:45 <DorpsGek> [OpenTTD/nml] glx22 opened pull request #399: Update: changelog for 0.8.1 https://github.com/OpenTTD/nml/pull/399
21:37:46 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14817: Codefix: Remove this-> from static call to CleaningPool. https://github.com/OpenTTD/OpenTTD/pull/14817
21:41:36 <DorpsGek> [OpenTTD/OpenTTD] Brickblock1 opened issue #14819: [Bug]: Cargo weight doesn't take into account weight property https://github.com/OpenTTD/OpenTTD/issues/14819
21:43:32 <xarick> what is std::cout?
21:44:11 <mmtunligit> bruh
21:45:52 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1442269950667526364/image.png?ex=6924d210&is=69238090&hm=81c254e5f43acdc3fff94923ad01fb937e2e72739ce133701ac6edb4d5291d5d&
21:45:52 <xarick> ah, it printed to console
21:48:02 <reldred> Dude that is hello world level stuff
21:48:10 <DorpsGek> [OpenTTD/OpenTTD] PeterN closed issue #14819: [Bug]: Cargo weight doesn't take into account weight property https://github.com/OpenTTD/OpenTTD/issues/14819
21:48:13 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #14819: [Bug]: Cargo weight doesn't take into account weight property https://github.com/OpenTTD/OpenTTD/issues/14819
21:48:55 <xarick> https://cppreference.net/cpp/container/map/emplace_hint.html i copy pasted
21:50:41 <kaji_kaede> reldred: ...I feel more confident about myself now.
21:53:10 <LordAro> xarick didn't learn programming in anything that would be considered traditional
21:53:41 <LordAro> if it's not in ottd, it's irrelevant :D
21:54:57 <andythenorth> hmm
21:55:15 <andythenorth> I have not made much newgrf recently 😛
21:55:34 <xarick> "emplace using returned iterator" was actually slow, why
21:56:58 <DorpsGek> [OpenTTD/OpenTTD] Brickblock1 opened issue #14820: [Bug]: Cargo weight doesn't take into account weight property https://github.com/OpenTTD/OpenTTD/issues/14820
21:58:09 <_jgr_> I am unconvinced that there is any bug here
21:58:43 <mmtunligit> i mean, ive also learned about coding from this game and nearly nothing else, and even if i didnt know what it did from existing around programmers, i feel like i wouldve figured it out by now from reading the C++reference examples, its used constatnly
22:01:07 <kaji_kaede> LordAro: Neither did I. Xarick *still*... Kinda amazes me.
22:02:00 <kaji_kaede> To be fair. I *do* have a friend who is actually competent and has worked professionally with C++, so... I suppose I don't have *quite* as minimal of an existing starting point as I thought.
22:02:03 *** brickblock19280 has joined #openttd
22:02:03 <brickblock19280> https://cdn.discordapp.com/attachments/1008473233844097104/1442274018475380736/image.png?ex=6924d5da&is=6923845a&hm=f29368d06ed2430c8f5c7fcbdfa738aaba9f3e60f8928d5d54e871bd9875401c&
22:02:03 <brickblock19280> _jgr_: how so?
22:02:03 <brickblock19280> I feel like the numbers should add up
22:02:03 <brickblock19280> 21 + 41 =/= 185 though
22:02:28 <kaji_kaede> And your cargo weight multiplier is set to 1?
22:02:41 <brickblock19280> 16 since it is given in 16ths
22:02:48 <_jgr_> It's 4x in the screenshot
22:03:17 <kaji_kaede> brickblock19280: "Weight multiplier for freight [...]" in the settings.
22:03:41 <brickblock19280> 🤦♀️
22:03:55 <kaji_kaede> yet another feature reported as a bug let's goooo
22:04:08 <DorpsGek> [OpenTTD/nml] andythenorth commented on pull request #358: Add: "include" keyword, allowing to include files without external tools https://github.com/OpenTTD/nml/pull/358#issuecomment-3568380588
22:04:35 <DorpsGek> [OpenTTD/OpenTTD] Brickblock1 closed issue #14820: [Bug]: Cargo weight doesn't take into account weight property https://github.com/OpenTTD/OpenTTD/issues/14820
22:06:58 <kaji_kaede> to be fair the options menu isn't known as being easy to browse
22:08:06 <mmtunligit> openttd having bad UI? shocking!
22:09:09 <peter1138> lol, why would you open another issue for the same thing... wtf.
22:12:08 <brickblock19280> I couldn't reopen the old one
22:13:54 <peter1138> Why would you though?
22:14:30 <brickblock19280> because I thought you closed it because I filed it with jgrpp
22:14:42 <brickblock19280> not entierly convinced this isn't a bug tho yet
22:16:13 <kaji_kaede> ...It seems pretty cut-and-dry not a bug though?
22:16:26 <kaji_kaede> You checked the setting, right?
22:17:54 <brickblock19280> 42-17 = 25 other cargo
22:17:54 <brickblock19280> 67-17 = 40 sweets
22:20:45 <peter1138> You can still comment on closed issues.
22:24:28 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1442279662976761866/image.png?ex=6924db1b&is=6923899b&hm=897746263f27a1f4a92890ea877dd68aeae66919357f40d99b52da7c7834841c&
22:24:28 <xarick> am I dumb?
22:25:07 <xarick> for (int i = 0; i < n_operations; ++i) it = map.try_emplace(it, i, 0);
22:25:33 <xarick> why is it a bad result
22:38:47 <rito12_51026> github-advanced-secu: Let's see if constexpr makes it happy
22:38:59 <DorpsGek> [OpenTTD/OpenTTD] Rito13 updated pull request #14717: Feature: New selection for rail types. https://github.com/OpenTTD/OpenTTD/pull/14717
22:45:14 <DorpsGek> [OpenTTD/OpenTTD] Rito13 updated pull request #14717: Feature: New selection for rail types. https://github.com/OpenTTD/OpenTTD/pull/14717
22:50:37 <brickblock19280> "Use the cargo_capacity callback to avoid this effect." this doesn't seem to be the case anymore
22:51:41 <brickblock19280> at least when I use the callback it still applies everything as if it were the property
22:53:48 <brickblock19280> "Unlike regular refitted capacities (including those from callback 36), the return value is not subject to the usual division of capacities for cargoes other than passengers on trains and planes, instead the capacity is used exactly as returned by the callback."
22:56:32 <_jgr_> The wagon you mentioned in your bug reports uses the capacity callback and it appears to work fine
22:57:32 <brickblock19280> yeah I was using cb 36 instead of cb 15 sorry for wasting your time
23:12:18 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
23:20:12 *** Zathras_4 has joined #openttd
23:27:45 <jfkuayue> Successfully packed 4 days of winter clothes (+ wearing one), a Canon 650D camera (+ 2 lenses), a 16" laptop, iPad Air, lots of accessories like power bank, plug, and earphones, plus passport + documents into a ryanair bag
23:28:14 <jfkuayue> Thanks to the loosened baggage limit
23:28:40 <jfkuayue> Wait still need a plug converter
23:35:14 <DorpsGek> [OpenTTD/OpenGFX2] zephyris commented on pull request #220: Feature: Add bridge deck overlays https://github.com/OpenTTD/OpenGFX2/pull/220#issuecomment-3568464128
continue to next day ⏵
