IRC logs for #openttd on OFTC at 2025-05-06
            
00:07:10 *** WormnestAndroid has quit IRC (Remote host closed the connection)
00:07:13 *** WormnestAndroid has joined #openttd
01:22:53 <goddess_ishtar> _zephyris: are changes to the Online Content menu in-game on the table?
01:23:08 <goddess_ishtar> because I have a few minor annoyances with it
01:32:52 <goddess_ishtar> mostly that checked items arent'
01:33:04 <goddess_ishtar> mostly that checked items aren't at the top of the list, really
01:33:17 <goddess_ishtar> that's probably a simple PR, I could do that
01:56:17 *** Wormnest has quit IRC (Quit: Leaving)
02:21:09 *** gnu_jj_ has joined #openttd
02:24:40 *** gnu_jj has quit IRC (Ping timeout: 480 seconds)
02:55:35 *** Flygon has joined #openttd
04:08:03 *** keikoz has joined #openttd
04:19:52 *** WormnestAndroid has quit IRC (Remote host closed the connection)
04:20:01 *** WormnestAndroid has joined #openttd
04:46:43 <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/eca826b0a4d0334a1ddcd4237a7c2ef1724d6787
04:46:44 <DorpsGek> - Update: Translations from eints (by translators)
05:09:02 <truebrain> _zephyris: Ghehe, when thinking about my design I linked we did the same: what do other games do! πŸ˜„ best reference, how other geekie games solve problems πŸ˜›
05:09:02 <truebrain> Btw, to me the small padding between sections look weird. Isn't a bit more padding easier on the eye?
05:15:23 <truebrain> Ugh, should write my feedback in the PR πŸ˜›
05:30:59 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#issuecomment-2853324236
05:33:00 <truebrain> Now I am zorging PRs. Or does that require making pictures? πŸ˜›
06:19:57 *** reldred has joined #openttd
06:19:57 <reldred> requires writing two page essays that make the reader want to punch their screen
06:24:03 <DorpsGek> [OpenTTD/OpenTTD] zephyris commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#issuecomment-2853410677
06:25:42 <truebrain> Yes to both points Zeph πŸ˜„
06:31:20 <_zephyris> Looks like consensus won't be too hard!
06:31:31 <_zephyris> (famous last words)
07:09:44 <LordAro> consensus is easy, implementation on the other hand...
07:12:39 <truebrain> I still wonder where I left my in-game screenshot of my designs ... I had them somewhere πŸ˜›
07:13:22 <truebrain> https://github.com/TrueBrain/OpenTTD/discussions/22
07:13:22 <truebrain> There we go!
08:23:31 <peter1138[d]> <https://diziet.dreamwidth.org/19879.html>
08:24:13 <peter1138[d]> > Consensus is great but always requiring it is harmful
08:34:23 *** D-HUND is now known as debdog
08:40:00 <_zephyris> Indeed
08:42:54 <peter1138[d]> In 20 years we've never had a governance issue, so that's nice.
08:43:22 <peter1138[d]> (Or if there was, it was behind closed doors and I wasn't involved... :p)
08:45:23 <_zephyris> Interesting, makes me think about my real job
08:45:48 <_zephyris> Lots of science is consensus on interpretation of evidence
09:03:43 <reldred> I've thrown a few tantrums about governance, but that's about it πŸ˜›
09:04:11 <reldred> Nothing of any actual consequence πŸ˜›
09:07:45 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1369239204332310578/image.png?ex=681b22e1&is=6819d161&hm=d54a533c1642b76ad21c6fe53b1c77b05aa3218a4f4871850d41929e49ef695f&
09:07:45 <xarick> wow, it reached 900!
09:10:33 <DorpsGek> [OpenTTD/OpenTTD] zephyris commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#issuecomment-2853820461
09:12:10 <truebrain> That looks nice πŸ˜„
09:12:57 <truebrain> peter1138[d]: Does the "name that shall not be spoken" not count? πŸ˜„
09:23:18 <peter1138[d]> Well, maybe. But whatever it was was over in a blink.
09:25:23 <Rubidium> poor DeMorgan not getting mentioned anymore ;)
09:28:18 <DorpsGek> [OpenTTD/OpenTTD] planetmaker commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#issuecomment-2853873851
09:32:46 <DorpsGek> [OpenTTD/OpenTTD] zephyris commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#issuecomment-2853885607
09:33:51 <xarick> I have a better idea!
09:34:04 <xarick> ``` sleep_longer = {
09:34:04 <xarick> [AITile.TRANSPORT_RAIL] = 0,
09:34:04 <xarick> [AITile.TRANSPORT_ROAD] = 0,
09:34:04 <xarick> [AITile.TRANSPORT_WATER] = 0,
09:34:04 <xarick> [AITile.TRANSPORT_AIR] = 0,
09:34:06 <xarick> };```
09:34:17 <xarick> or false
09:34:19 <xarick> instead of 0
09:36:14 <xarick> but now i need an equivalent to "if all of these are true"
09:37:05 <peter1138[d]> Recreating EnumBitSet in squirrel, but...
09:42:12 <xarick> i can maybe use AIList for this, let's see
09:44:23 <xarick> oh, there is no HasValue 😦
09:47:08 <xarick> suggestion: Implement a ScriptList.HasValue
09:47:50 <peter1138[d]> Is that different from `HasItem()`?
09:47:55 <xarick> yes
09:48:22 <xarick> HasItem would be the AITile.TRANSPORT_XXX
09:49:03 <xarick> HasValue would be if any of the AITile.TRANSPORT_XXX has the value "value"
10:01:14 <DorpsGek> [OpenTTD/OpenTTD] Kuhnovic commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#issuecomment-2853973483
10:06:25 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#issuecomment-2853987553
10:12:54 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1369255600030220338/image.png?ex=681b3226&is=6819e0a6&hm=6bf83e825084b65a1138e8b28ad722bfe8ba902c710375788d51b103fc3259c8&
10:12:54 <xarick> let's see
10:15:00 *** Flygon has quit IRC (Read error: Connection reset by peer)
10:15:51 <DorpsGek> [OpenTTD/OpenTTD] zephyris commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#issuecomment-2854027418
10:16:25 *** debdog has quit IRC (autokilled: This host violated network policy. Contact support@oftc.net for further information and assistance. (2025-05-06 10:16:25))
10:17:27 <andythenorth> peter1138[d]: a correct thing from reddit https://www.reddit.com/r/openttd/comments/1kfzmac/doom_making_off/
10:17:57 <andythenorth> https://www.reddit.com/r/openttd/comments/1kf8870/doom_bum_bum/
10:30:18 <xarick> `while (AIController.Sleep(Utils.ListHasValue(this.sleep_longer, false) ? 1 : 74)) {`
10:30:23 <xarick> probably readable
10:36:20 <LordAro> if even you only think it's probably readable, it's probably not
10:38:58 <xarick> :/
10:39:23 <peter1138[d]> Oh, building the code is not enough. I have to actually run it as well.
10:40:23 <peter1138[d]> Hmm, and of course this bit of code doesn't even run at this point anyway. Bah.
10:43:39 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1369263338479161435/image.png?ex=681b395b&is=6819e7db&hm=fc1068c8f41a8582411b4ce8fd0682e014b7b9a531f8feba6be81914f5585bad&
10:43:39 <xarick> well, it's not sleeping the right amount
10:44:43 <xarick> those other 3 transport types being disabled would imply they could sleep longer, right?
10:49:23 <peter1138[d]> What issue are you trying to solve?
10:50:21 <xarick> the "waiting for money phase"
10:50:34 <xarick> where sleeping 1 tick makes no sense
10:52:19 <peter1138[d]> Also, random thought. Often not being able to run multiple GS at once has been a topic. Has anyone ever considered multiple AI in one company (similar to two players in one company in multiplayer...)
10:54:59 <_glx_> Same issue as multiple GS
10:55:02 <squirejames> I wonder if that could be used to make a slightly more intelligent (not really, but) competitor. Have three AIs in charge of a company, and two have to agree for an action to be taken
10:55:08 <squirejames> (like fly by wire and all that)
10:57:58 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1369266940614410311/image.png?ex=681b3cb6&is=6819eb36&hm=0a444c265dffc1067c1d906b35d78780b9895f15a5a17bbb2b59a183cc16ba31&
10:57:58 <xarick> nice, it's sleeping fine
10:58:03 <xarick> longer
10:58:22 <xarick> has no money for the next rail route, so it sleeps
10:59:10 <xarick> and the other transport modes are also saying "yeah, sleep longer, we're disabled"
11:06:06 <xarick> if a day has 74 ticks and 24 * 60 minutes, a real time tick would take ~19 minutes
11:07:46 <xarick> how did you settle for 74 ticks?
11:07:51 <xarick> why not 72
11:08:20 <xarick> multiple of 24
11:09:16 <xarick> 1440 / 72 = exact 20 minutes
11:09:24 <yiffgirl> has anyone consulted the literature on menu design? i really don't think the current screenshots in #14220 are it
11:09:43 <LordAro> "the literature"
11:12:33 <xarick> i forgot how long a tick actually takes in-game, i heard something like 27 ms?
11:12:38 <xarick> or whatever
11:13:07 <peter1138[d]> An implementation detail that shouldn't concern scripts.
11:19:33 <xarick> need some math skills, how many milliseconds there are in a day
11:20:30 *** WormnestAndroid has quit IRC (Remote host closed the connection)
11:20:32 <xarick> 24 * 60 * 60 * what = milliseconds per day
11:20:35 *** WormnestAndroid has joined #openttd
11:23:05 <peter1138[d]> game day or real day?
11:23:19 <xarick> i was looking for round numbers
11:23:24 <xarick> no decimals
11:23:33 <xarick> yes, real day
11:23:58 <peter1138[d]> How many milliseconds are in a second?
11:24:03 <xarick> 1000
11:24:15 <peter1138[d]> How many seconds in a minute?
11:24:23 <xarick> 60
11:24:42 <peter1138[d]> You know what. Nobody could possibly know.
11:24:50 <peter1138[d]> This seems like a job for our DHCP server.
11:24:56 <LordAro> lol
11:26:32 *** __abigail has joined #openttd
11:26:32 <__abigail> peter1138[d]: How many milliseconds in a day? How many IPv4 addresses in 10.0.0.0/8?
11:26:32 <__abigail> No one will ever know
11:28:19 <peter1138[d]> When your network uses 169.254/16...
11:29:31 <__abigail> Pfffffff no one will EVER need more than /24 for a subnet
11:29:31 <__abigail> Impossible
11:30:32 <peter1138[d]> Of course they will, that's why Class A, B, C, D, and E addresses exist!
11:30:42 <xarick> i fail at translating openttd to realife times again
11:32:16 <__abigail> peter1138[d]: Nonsense
11:33:12 <DorpsGek> [OpenTTD/OpenTTD] Moth-Tolias commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#issuecomment-2854235763
11:35:00 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1369276258348830871/image.png?ex=681b4563&is=6819f3e3&hm=3774971708dac78b02db26251808104f41d727aaeadf7e4535eda90ae1466fea&
11:35:00 <xarick> hmm...
11:35:09 <xarick> not sure i got them sleeping the right way
11:35:21 <xarick> they are all waiting for money, i probably failed
11:37:01 <Rubidium> __abigail: yet the default is only /64 ;D
11:38:07 <__abigail> Just make 2^56 subnets with 255 addresses each then smh πŸ€¦β€β™€οΈ
11:46:00 <_glx_> I only have a /60
11:57:52 <peter1138[d]> Oh, std::bitset.
12:01:47 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14223: Codechange: Replace `std::bitset<CONTENT_TYPE_END>` with `ContentTypes`. https://github.com/OpenTTD/OpenTTD/pull/14223
12:04:25 <peter1138[d]> Hmm, might be a header.
12:04:54 <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #14223: Codechange: Replace `std::bitset<CONTENT_TYPE_END>` with `ContentTypes`. https://github.com/OpenTTD/OpenTTD/pull/14223#pullrequestreview-2818007563
12:05:12 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14223: Codechange: Replace `std::bitset<CONTENT_TYPE_END>` with `ContentTypes`. https://github.com/OpenTTD/OpenTTD/pull/14223
12:05:22 <jfkuayue> PR14220 is confusing for me
12:06:48 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14223: Codechange: Replace `std::bitset<CONTENT_TYPE_END>` with `ContentTypes`. https://github.com/OpenTTD/OpenTTD/pull/14223#pullrequestreview-2818015753
12:08:31 <jfkuayue> There are 86400 seconds in a day. So 86.4 million (86400000) milliseconds.
12:11:15 <jfkuayue> And thus 3200000 ticks (27ms/tick) in a real day.
12:11:57 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #14223: Codechange: Replace `std::bitset<CONTENT_TYPE_END>` with `ContentTypes`. https://github.com/OpenTTD/OpenTTD/pull/14223#pullrequestreview-2818029910
12:17:33 <xarick> need to wait for Zorg, he will settle #14220 record straight
12:34:50 <xarick> okay china, you win the internet today https://www.youtube.com/watch?v=dz1MhkbPthI
13:06:02 <Rubidium> LordAro: https://doi.org/10.1016/S0020-7373(84)80018-8 seems to imply that all proposals have too many "selections" in the menu :D
13:08:56 <LordAro> :)
13:39:36 <peter1138[d]> Hmm.
13:59:45 *** nielsm has joined #openttd
14:07:07 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14212: Codechange: Clean up static variables. https://github.com/OpenTTD/OpenTTD/pull/14212#pullrequestreview-2818430583
14:11:00 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220
14:11:18 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#issuecomment-2854729561
14:12:17 <DorpsGek> [OpenTTD/OpenTTD] zephyris commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#issuecomment-2854732931
14:15:42 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#issuecomment-2854744695
14:19:49 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#issuecomment-2854758059
14:25:27 <xarick> I'm unable to make this sleeping system work when combined with the transport mode rotation system
14:28:34 <xarick> I don't know why I'm making it so elaborate
14:28:41 <xarick> i could just make it simple
14:30:43 <xarick> something else that's in the way is the timer
14:36:31 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1369321940879278250/image.png?ex=681b6fef&is=681a1e6f&hm=106cfd5ef52d6a654b14b4cba997ead11861e114a17dd0927b9afedd4b537af7&
14:36:31 <xarick> it kind of works once every transport type is waiting for money
14:42:34 <xarick> sometimes some transport modes are temporarily toggled off rotation, so their last sleep_longer should be ignored. hmm how to do this
14:50:14 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14222: Codechange: Access temporary storage only through ResolverObject. https://github.com/OpenTTD/OpenTTD/pull/14222
14:51:31 <xarick> oh multithreaded openttd!
15:05:44 <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #14221: Codechange: Remove global state from spritelayout processing. https://github.com/OpenTTD/OpenTTD/pull/14221#pullrequestreview-2818638768
15:07:32 <peter1138[d]> Urgh, trying to unify code that is repeated but is almost but not exactly the same :S
15:16:08 *** gelignite has joined #openttd
15:25:09 *** Wormnest has joined #openttd
15:26:40 <xarick> πŸ™‚
15:30:04 <LordAro> peter1138[d]: that's how i've spent most of my day
15:39:57 <DorpsGek> [OpenTTD/OpenTTD] planetmaker commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#issuecomment-2855062425
16:11:01 *** kuka_lie has joined #openttd
16:19:43 <peter1138[d]> Pom te pom
16:20:05 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14223: Codechange: Replace `std::bitset<CONTENT_TYPE_END>` with `ContentTypes`. https://github.com/OpenTTD/OpenTTD/pull/14223
16:27:35 <DorpsGek> [OpenTTD/OpenTTD] frosch123 opened pull request #14224: Add: [NewGRF] All callbacks returning D0xx strings, have now the option to return any string id via register 0x100. https://github.com/OpenTTD/OpenTTD/pull/14224
16:27:41 *** Flygon has joined #openttd
16:29:36 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14212: Codechange: Clean up static variables. https://github.com/OpenTTD/OpenTTD/pull/14212#issuecomment-2855216015
16:29:36 *** Wormnest has quit IRC (Ping timeout: 480 seconds)
16:29:47 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #14212: Codechange: Clean up static variables. https://github.com/OpenTTD/OpenTTD/pull/14212
16:30:46 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #14221: Codechange: Remove global state from spritelayout processing. https://github.com/OpenTTD/OpenTTD/pull/14221
16:31:17 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1369350822181339306/image.png?ex=681b8ad5&is=681a3955&hm=045cfc58863f6702ee7320a2d30f5a7e3bc58632d0368fc991b282953597cf14&
16:31:17 <andythenorth> FML, I mean it's just a grf πŸ˜›
16:31:24 <andythenorth> I seem to have been busy
16:32:43 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14222: Codechange: Access temporary storage only through ResolverObject. https://github.com/OpenTTD/OpenTTD/pull/14222
16:35:30 *** gelignite has quit IRC (Quit: Stay safe!)
16:40:15 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #14225: Codechange: use C-style memory functions less https://github.com/OpenTTD/OpenTTD/pull/14225
16:42:33 *** Wolf01 has joined #openttd
16:48:54 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
16:51:50 *** WormnestAndroid has joined #openttd
16:58:00 <peter1138[d]> frosch123: hmm, conversion to `StringConsumer` seems incomplete in `network.cpp`. There's a call to `std::from_chars()` with `buffer`, which no longer exists.
16:59:28 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #14207: Codechange: Use LRUCache for layouted line cache. https://github.com/OpenTTD/OpenTTD/pull/14207#pullrequestreview-2819028299
17:01:05 <frosch123> Ah ifdefs
17:02:13 <peter1138[d]> Yup.
17:02:31 <peter1138[d]> (There's an `(int)_current_company` that so
17:02:37 <peter1138[d]> ...
17:02:46 <peter1138[d]> That should probably lose the `(int)`, as well.
17:07:19 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14226: Codechange: Remove some unnecessary includes from network files. https://github.com/OpenTTD/OpenTTD/pull/14226
17:07:47 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14225: Codechange: use C-style memory functions less https://github.com/OpenTTD/OpenTTD/pull/14225#pullrequestreview-2819048453
17:09:04 <DorpsGek> [OpenTTD/OpenTTD] PeterN merged pull request #14207: Codechange: Use LRUCache for layouted line cache. https://github.com/OpenTTD/OpenTTD/pull/14207
17:11:53 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14149: Add: [NewGRF] Add explicit (Var)Action2 results for "callback failed" and "calculated result". https://github.com/OpenTTD/OpenTTD/pull/14149#pullrequestreview-2819058617
17:13:17 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #14225: Codechange: use C-style memory functions less https://github.com/OpenTTD/OpenTTD/pull/14225
17:15:34 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #14224: Add: [NewGRF] All callbacks returning D0xx strings, have now the option to return any string id via register 0x100. https://github.com/OpenTTD/OpenTTD/pull/14224#pullrequestreview-2819071410
17:15:35 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
17:16:14 *** WormnestAndroid has joined #openttd
17:31:48 <peter1138[d]> Hmm, mention of that GRFv9 thing.
17:32:48 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#issuecomment-2855394668
17:34:10 <frosch123> Grfv9 is also somewhere on my list. Got some new ideas
17:35:29 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
17:36:46 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #14225: Codechange: use C-style memory functions less https://github.com/OpenTTD/OpenTTD/pull/14225#pullrequestreview-2819124157
17:38:12 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #14226: Codechange: Remove some unnecessary includes from network files. https://github.com/OpenTTD/OpenTTD/pull/14226#pullrequestreview-2819127334
17:39:00 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14224: Add: [NewGRF] All callbacks returning D0xx strings, have now the option to return any string id via register 0x100. https://github.com/OpenTTD/OpenTTD/pull/14224#pullrequestreview-2819129143
17:40:45 *** WormnestAndroid has joined #openttd
17:41:11 <jfkuayue> Why are these dedicated linux laptops not cheaper than windows laptops?
17:41:21 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#pullrequestreview-2819135211
17:42:05 <Rubidium> jfkuayue: simple, because those a customers looking for something specific, which you can ask more money for
17:42:19 <jfkuayue> Oops
17:42:20 <peter1138[d]> Oh.
17:42:45 <peter1138[d]> I just realised the latest pictures are not in the PR yet πŸ™‚
17:43:41 <blathijs> jfkuayue: Also, smaller scale and market might be relevant
17:44:47 <peter1138[d]> You'd think it would simply be a case of "don't include windows" but alas...
17:45:18 <peter1138[d]> That then creates extra product lines etc..
17:46:05 <jfkuayue> Β£772.50… and with Intel N200
17:48:11 <peter1138[d]> Basically, none of them are an M4 ARM.
17:50:51 <jfkuayue> Also I tried installing Debian and dualbooting. The debian part was also troublesome, it could not find wifi modules…
17:51:06 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#pullrequestreview-2819158550
17:53:18 <_glx_> peter1138[d]: usually costs extra money to not include windows, and more than the price of a windows license
17:56:04 <frosch123> Added more pictures to my last comment
18:00:09 <andythenorth> cost of computer should be calculated per day πŸ˜›
18:00:26 <andythenorth> unless you have no access to finance, been there 😐
18:00:39 <andythenorth> "I'll just buy what I have actual real money for"
18:03:19 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#pullrequestreview-2819191850
18:03:39 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #14225: Codechange: use C-style memory functions less https://github.com/OpenTTD/OpenTTD/pull/14225
18:06:25 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220
18:14:49 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #14227: Codechange: reduce usage of memcmp https://github.com/OpenTTD/OpenTTD/pull/14227
18:26:37 <DorpsGek> [OpenTTD/OpenTTD] frosch123 opened pull request #14228: Codefix: Fix compilation with DEBUG_DUMP_COMMANDS. https://github.com/OpenTTD/OpenTTD/pull/14228
18:33:31 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14228: Codefix: Fix compilation with DEBUG_DUMP_COMMANDS. https://github.com/OpenTTD/OpenTTD/pull/14228#pullrequestreview-2819264254
18:37:57 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14227: Codechange: reduce usage of memcmp https://github.com/OpenTTD/OpenTTD/pull/14227#pullrequestreview-2819271457
18:44:08 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #14227: Codechange: reduce usage of memcmp https://github.com/OpenTTD/OpenTTD/pull/14227
18:50:14 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14217: Codechange: Turn _grow_town_result into a local variable in GrowTownAtRoad. https://github.com/OpenTTD/OpenTTD/pull/14217#pullrequestreview-2819300512
18:50:51 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14222: Codechange: Access temporary storage only through ResolverObject. https://github.com/OpenTTD/OpenTTD/pull/14222
18:56:29 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14224: Add: [NewGRF] All callbacks returning D0xx strings, have now the option to return any string id via register 0x100. https://github.com/OpenTTD/OpenTTD/pull/14224
18:58:20 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #14217: Codechange: Turn _grow_town_result into a local variable in GrowTownAtRoad. https://github.com/OpenTTD/OpenTTD/pull/14217
19:07:15 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14149: Add: [NewGRF] Add explicit (Var)Action2 results for "callback failed" and "calculated result". https://github.com/OpenTTD/OpenTTD/pull/14149
19:07:24 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14149: Add: [NewGRF] Add explicit (Var)Action2 results for "callback failed" and "calculated result". https://github.com/OpenTTD/OpenTTD/pull/14149#pullrequestreview-2819340277
19:08:25 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #14227: Codechange: reduce usage of memcmp https://github.com/OpenTTD/OpenTTD/pull/14227#pullrequestreview-2819343565
19:09:28 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #14222: Codechange: Access temporary storage only through ResolverObject. https://github.com/OpenTTD/OpenTTD/pull/14222#pullrequestreview-2819315321
19:11:34 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #14228: Codefix: Fix compilation with DEBUG_DUMP_COMMANDS. https://github.com/OpenTTD/OpenTTD/pull/14228
19:12:28 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14149: Add: [NewGRF] Add explicit (Var)Action2 results for "callback failed" and "calculated result". https://github.com/OpenTTD/OpenTTD/pull/14149
19:15:15 <xarick> https://www.tt-forums.net/viewtopic.php?t=92206 libraries were a mistake?
19:16:15 <andythenorth> deps have issues
19:16:41 <andythenorth> the key skill that separates programmers from non-programmers is resolving deps
19:16:47 <_glx_> deps are issues in most content managing system
19:16:54 <andythenorth> syntax, languages, algorithms are very secondary
19:17:01 <andythenorth> and LLMs can't resolve deps for us
19:17:09 <andythenorth> (they just make it worse)
19:18:35 <Rubidium> dependencies aren't the issue, unmaintained dependencies are
19:21:10 <_glx_> btw bananas usually works fine for deps
19:23:23 <_glx_> https://www.tt-forums.net/viewtopic.php?p=1274134#p1274134 impressive delay for a reply πŸ™‚
19:25:04 <Heiki> who needs OpenTTD 15 when people are happy with 0.6.3
19:25:20 <xarick> at first I was thinking of releasing my canal pathfinder as a library...
19:25:28 <xarick> then I went the other way
19:25:38 <xarick> much better this way
19:25:55 <_glx_> libraries are fine when using bananas
19:27:58 <xarick> and I dropped SCP support
19:28:22 <DorpsGek> [OpenTTD/OpenTTD] Rau771 commented on pull request #14220: Change: Remove all buttons form the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#issuecomment-2855695649
19:28:24 <xarick> Company Value SPC? something
19:29:07 <xarick> Script Communication Protocol
19:29:08 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14222: Codechange: Access temporary storage only through ResolverObject. https://github.com/OpenTTD/OpenTTD/pull/14222
19:29:21 <xarick> not even sure it still works, with all these string changes
19:29:24 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #14227: Codechange: reduce usage of memcmp https://github.com/OpenTTD/OpenTTD/pull/14227
19:29:51 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #14222: Codechange: Access temporary storage only through ResolverObject. https://github.com/OpenTTD/OpenTTD/pull/14222#pullrequestreview-2819395631
19:31:52 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #14222: Codechange: Access temporary storage only through ResolverObject. https://github.com/OpenTTD/OpenTTD/pull/14222#pullrequestreview-2819400913
19:34:04 <xarick> ```local best_built = unfinished ? this.air_build_manager.m_best_routes_built : ((((this.bestRoutesBuilt >> 4) & 3) & (1 << (cargo_class == AICargo.CC_PASSENGERS ? 0 : 1))) != 0);
19:34:04 <xarick> local all_built = unfinished ? this.air_build_manager.m_all_routes_built : (((this.allRoutesBuilt >> 4) & 3) & (1 << (cargo_class == AICargo.CC_PASSENGERS ? 0 : 1))) == 0;```
19:34:04 <xarick> hmm, one of these is wrong
19:34:23 <_glx_> too much ternaries
19:35:10 <xarick> I'm attempting to get rid of those bit manipulations
19:38:45 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1369398000475963572/image.png?ex=681bb6c5&is=681a6545&hm=a2e4afbc2d4952ea5f3242d567ff141dde727217a8d7cda84db0b1940ead90f8&
19:38:45 <xarick> hmm
19:38:55 <xarick> I'm surprised I documented this thing
19:41:23 <xarick> all_routes_built[transport_type][cargo_class] maybe?
19:43:52 *** Flygon has quit IRC (Quit: A toaster's basically a soldering iron designed to toast bread)
19:47:03 <xarick> a table of tables of bool?
19:47:13 <xarick> or a single integer
19:48:08 <xarick> wondering when will I reach the "too long to start error"
19:50:18 *** akimoto has joined #openttd
19:54:37 *** gelignite has joined #openttd
19:55:21 <peter1138[d]> Speaking of ternaries.
19:55:36 <peter1138[d]> This code `uint x = 5 << axis`
19:56:48 <peter1138[d]> Is actually something like `axis == AXIS_X ? DiagDirections{DIAGDIR_NE, DIAGDIR_SW} : DiagDirections{DIAGDIR_SE, DIAGDIR_NW}`
19:58:16 <peter1138[d]> So bitshift + incomprehensible code vs conditional + understandable code
19:59:18 <peter1138[d]> Heh, could be a 2-element array to avoid a conditional, but...
20:00:31 <xarick> is that about DECLARE_ENUM_AS_ADDABLE(DiagDirection)
20:01:11 <xarick> nop
20:02:28 <_glx_> totally unrelated πŸ™‚
20:02:58 <xarick> I remember i saw a 5 somewhere
20:03:38 <_glx_> I guess it's most likely a 0x101
20:04:45 <xarick> ```inline Direction AxisToDirection(Axis a)
20:04:45 <xarick> {
20:04:45 <xarick> assert(IsValidAxis(a));
20:04:45 <xarick> return (Direction)(5 - 2 * a);
20:04:45 <xarick> }```
20:06:08 <xarick> Axis to DiagDir is 2 - a
20:06:33 <xarick> it's too much magic happening in direction_func.h
20:10:24 <xarick> 5 << 0 = 0101, 5 << 1 = 1010
20:12:38 <xarick> NE = 0001, SW = 0100
20:24:45 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #14222: Codechange: Access temporary storage only through ResolverObject. https://github.com/OpenTTD/OpenTTD/pull/14222
20:28:58 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #14224: Add: [NewGRF] All callbacks returning D0xx strings, have now the option to return any string id via register 0x100. https://github.com/OpenTTD/OpenTTD/pull/14224
20:32:02 <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14229: Codechange: Add and use (Diag)Directions EnumBitSets. https://github.com/OpenTTD/OpenTTD/pull/14229
20:42:34 <xarick> ```this.routes_built = {};
20:42:34 <xarick> foreach (member in ["best", "all"]) {
20:42:34 <xarick> this.routes_built.rawset(member, {});
20:42:34 <xarick> foreach (transport_mode in [AITile.TRANSPORT_RAIL, AITile.TRANSPORT_ROAD, AITile.TRANSPORT_WATER, AITile.TRANSPORT_AIR]) {
20:42:34 <xarick> this.routes_built[member].rawset(transport_mode, {});
20:42:36 <xarick> foreach (cargo_class in [AICargo.CC_PASSENGERS, AICargo.CC_MAIL]) {
20:42:36 <xarick> this.routes_built[member][transport_mode].rawset(cargo_class, false);
20:42:38 <xarick> }
20:42:38 <xarick> }
20:42:40 <xarick> }```
20:42:40 <xarick> how's this for initialization?
20:44:52 <DorpsGek> [OpenTTD/OpenTTD] Rau771 commented on pull request #14220: Change: Remove all buttons from the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#issuecomment-2855930357
20:49:54 <xarick> i should be using arrays, not tables
20:50:06 <xarick> arrays are sorted by insertion
20:50:12 <xarick> tables are sorted by random
20:51:30 <andythenorth> for 14224 will we be able to hide the implementation details from authors in nml?
20:55:00 <peter1138[d]> Let's see 14220./
20:55:17 <peter1138[d]> Such compilation time.
20:58:19 <peter1138[d]> Heh, the branch title affects it a bit πŸ™‚
21:04:57 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #14220: Change: Remove all buttons from the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#pullrequestreview-2819635098
21:07:22 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)
21:08:25 *** gelignite has quit IRC (Quit: Stay safe!)
21:10:19 <_glx_> andythenorth: not easy as authors manually fill registers
21:14:41 <xarick> `((((this.allRoutesBuilt >> 4) & 3) & (1 << (cargo_class == AICargo.CC_PASSENGERS ? 0 : 1))) == 0)`
21:14:41 <xarick> what have I done here? I'm trying to pass around the current state of MAIL or PASSENGERS air routes
21:15:17 <xarick> what do I have on the other side
21:15:45 <xarick> if (pick_mode != 1 && this.m_all_routes_built
21:17:12 <xarick> so if it's false, I pass a true but i want it to behave as !current_state
21:18:10 <xarick> I'm so dumb this.m_all_routes_built is working correctly, but in a very confusing way
21:18:17 <_glx_> seems over complicated, you don't need to shift left and mask, just shift right more to test your bit
21:18:33 *** kuka_lie has quit IRC (Quit: Lost terminal)
21:19:43 <_glx_> `(this.allRoutesBuilt & (1 << (cargo_class == AICargo.CC_PASSENGERS ? 4 : 5)) == 0)` should do the same
21:20:59 <xarick> there are more bits on the other side
21:21:20 <xarick> which i want to not include
21:21:23 <_glx_> but you are only testing a bit
21:21:52 <_glx_> others are not tested anyway
21:22:28 <xarick> oh, i see
21:23:15 <xarick> im still trying to transform this into a table of tables
21:24:20 <xarick> but the code sometimes tests if both mail and pass are used
21:25:22 <xarick> i have some `!= 3` around
21:26:24 <_glx_> `((this.allRoutesBuilt >> 4) & 3) != 3` for that
21:26:52 <xarick> how to make it more humanly readable
21:27:36 <peter1138[d]> Avoid bitstuffing.
21:28:23 <xarick> I'm trying, but I end up going from a problem to a different kind of problem
21:29:05 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #14229: Codechange: Add and use (Diag)Directions EnumBitSets. https://github.com/OpenTTD/OpenTTD/pull/14229
21:32:09 <_glx_> a single table should work, and you'll gain named fields
21:33:06 <_glx_> or even a class so you can put accessors in it
21:34:05 *** keikoz has quit IRC (Ping timeout: 480 seconds)
21:36:09 *** WormnestAndroid has quit IRC (Remote host closed the connection)
21:37:21 *** WormnestAndroid has joined #openttd
21:39:30 <xarick> I got too many changes in the oven
21:39:37 <xarick> better start commiting
21:46:19 *** tokai|noir has joined #openttd
21:46:19 *** ChanServ sets mode: +v tokai|noir
21:53:13 *** tokai has quit IRC (Ping timeout: 480 seconds)
21:56:20 *** nielsm has quit IRC (Ping timeout: 480 seconds)
22:12:15 <xarick> uploaded!
22:13:35 <xarick> problem with tables
22:14:22 <xarick> number of opcodes to access them is varied I think
22:14:26 <xarick> from run to run
22:14:49 <xarick> and the iteration order is random
22:15:26 *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)
22:15:26 <xarick> iterating with foreach
22:18:14 <xarick> that second problem I am sure, I ran across it when picking the best engine wagon combinations on iron horse, it was always different from run to run
22:18:23 <xarick> but I'm not entirely sure of the first
22:19:36 *** WormnestAndroid has joined #openttd
22:35:39 <DorpsGek> [OpenTTD/OpenTTD] ladysadie commented on pull request #12690: Feature: Add font resizing sliders to the game options UI. https://github.com/OpenTTD/OpenTTD/pull/12690#issuecomment-2856309097
23:04:32 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)
23:10:05 *** WormnestAndroid has joined #openttd
23:10:26 <DorpsGek> [OpenTTD/OpenTTD] WenSimEHRP commented on pull request #14220: Change: Remove all buttons from the intro screen, which are already present in the relevant sub windows. https://github.com/OpenTTD/OpenTTD/pull/14220#issuecomment-2856436510
23:34:37 <DorpsGek> [OpenTTD/OpenTTD] WenSimEHRP started discussion #14230: Documentation translation https://github.com/OpenTTD/OpenTTD/discussions/14230