IRC logs for #openttd on OFTC at 2025-10-24
#openttd / 2025 / 10 
            

00:04:54  *** gelignite has quit IRC ()



00:15:46  *** lobster has quit IRC (Read error: Connection reset by peer)



00:16:05  <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #14727: Codechange: Prefer string equality instead of comparison. https://github.com/OpenTTD/OpenTTD/pull/14727#pullrequestreview-3373611363



00:16:23  *** lobster has joined #openttd



01:34:40  *** Flygon has joined #openttd



01:58:16  *** tokai has joined #openttd



01:58:16  *** ChanServ sets mode: +v tokai



01:58:24  *** Wormnest has quit IRC (Quit: Leaving)



02:05:16  *** tokai|noir has quit IRC (Ping timeout: 480 seconds)



02:09:55  *** gnu_jj_ has joined #openttd



02:13:14  *** gnu_jj has quit IRC (Ping timeout: 480 seconds)



03:10:10  *** WormnestAndroid has quit IRC (Remote host closed the connection)



03:10:13  *** WormnestAndroid has joined #openttd



03:57:17  *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)



03:57:30  *** WormnestAndroid has joined #openttd



03:57:45  *** Zathras has joined #openttd



03:57:49  *** Zathras_1 has joined #openttd



04:01:06  *** Zathras_11 has quit IRC (Ping timeout: 480 seconds)



04:01:11  *** Zathras_4 has quit IRC (Ping timeout: 480 seconds)



04:16:26  *** keikoz has joined #openttd



06:03:04  *** Flygon has quit IRC (Remote host closed the connection)



07:05:16  <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #14727: Codechange: Prefer string equality instead of comparison. https://github.com/OpenTTD/OpenTTD/pull/14727#pullrequestreview-3374811927



07:32:02  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #14727: Codechange: Prefer string equality instead of comparison. https://github.com/OpenTTD/OpenTTD/pull/14727#pullrequestreview-3374890719



07:35:09  <DorpsGek> [OpenTTD/OpenTTD] LordAro commented on pull request #14727: Codechange: Prefer string equality instead of comparison. https://github.com/OpenTTD/OpenTTD/pull/14727#pullrequestreview-3374900793



07:35:23  <DorpsGek> [OpenTTD/OpenTTD] LordAro approved pull request #14727: Codechange: Prefer string equality instead of comparison. https://github.com/OpenTTD/OpenTTD/pull/14727#pullrequestreview-3374901713



08:03:32  *** gelignite has joined #openttd



08:20:27  <peter1138> :o



08:44:29  <xarick> hi



08:45:50  <xarick> in landscape_grid.html, i wish the "type" class could be increased to 16 bits



08:58:14  <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1431205131466047499/image.png?ex=68fc9125&is=68fb3fa5&hm=49882412a508105cc876edbee1dc140605b54dd91042275d38297548ef3874eb&



08:58:14  <xarick> that's a weird place to display the error message



08:59:04  *** Zathras_1 has quit IRC (Quit: http://quassel-irc.org - Chat comfortably. Anywhere.)



09:35:21  *** herms2 has quit IRC (Quit: Ping timeout (120 seconds))



09:35:40  *** herms2 has joined #openttd



10:14:42  <peter1138> Is today a good day to resign?



10:17:13  <LordAro> :o



11:03:48  *** gelignite has quit IRC ()



11:43:46  <andythenorth> I'm not planning to



11:44:00  <andythenorth> not as far as I know anyway



11:44:08  <andythenorth> the day has hours left in it though



11:45:03  <peter1138> If you ever need a C# or C++ developer... :o



11:47:08  <andythenorth> I'm not sure about what I'm like to work for 🙂



11:47:11  <andythenorth> but eh



11:49:20  <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1431248190040178760/image.png?ex=68fcb93f&is=68fb67bf&hm=9471368e970a483622f3cdc003632b9563180127279e99bc4893f8400a51c97f&



11:49:20  <andythenorth> hmm a button to sort variants by badge somehow? 👀



11:49:22  <andythenorth> dunno



11:49:30  <ahyangyi> What's special about today, a power of 2?



11:55:35  <jfkuayue> Doesn't work for Americans



11:55:59  <jfkuayue> Wait no



11:56:09  <jfkuayue> Doesn't work for british...



11:57:44  <ahyangyi> Dec 05 would work though



12:16:58  *** kuhnovic has joined #openttd



12:16:58  <kuhnovic> peter1138: C#? Blasphemy!



12:17:35  <kuhnovic> Just kidding, I actually like C# better than C++



12:36:30  <tony_pixel> C# has its perks



12:36:59  <tony_pixel> I found it exceptionally convenient to work with when project contains a wide class hierarchy



12:37:16  <tony_pixel> Pattern matching is easier and more convenient than ever



12:40:00  <talltyler> I wish I had an excuse to write more C#, I’ve quite liked it in the past



12:56:55  *** Heiki has quit IRC (Quit: Bridge terminating on SIGTERM)



13:04:28  <kuhnovic> It's quite a well rounded language with powerful features without becoming too complicated



13:06:55  *** alfagamma7 has joined #openttd



13:06:55  <alfagamma7> I see



13:07:02  <alfagamma7> will need to try it out then



13:20:03  <DorpsGek> [OpenTTD/OpenTTD] cntgigadisk-creator opened issue #14728: [Crash]: Open scenario from original TTD in map editor. https://github.com/OpenTTD/OpenTTD/issues/14728



13:26:00  <DorpsGek> [OpenTTD/OpenTTD] James103 commented on issue #14728: [Crash]: Open scenario from original TTD in map editor. https://github.com/OpenTTD/OpenTTD/issues/14728



13:30:37  <DorpsGek> [OpenTTD/OpenTTD] cntgigadisk-creator commented on issue #14728: [Crash]: Open scenario from original TTD in map editor. https://github.com/OpenTTD/OpenTTD/issues/14728



13:31:56  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #14728: [Crash]: Open scenario from original TTD in map editor. https://github.com/OpenTTD/OpenTTD/issues/14728



13:34:13  <peter1138> System.FormatException: String 'true,false' was not recognized as a valid Boolean



13:34:16  <peter1138> Yeah, no kidding.



13:36:15  <jfkuayue> sei, sei jia jia, sei jing /j



13:38:32  <peter1138> I do actually have that scenario file, but mine is different and doesn't crash.



13:42:26  <LordAro> ..can TTD scenarios be distributed?



13:43:08  <DorpsGek> [OpenTTD/OpenTTD] James103 commented on issue #14728: [Crash]: Open scenario from original TTD in map editor. https://github.com/OpenTTD/OpenTTD/issues/14728



13:43:32  <peter1138> Probably shouldn't be.



13:52:58  <LordAro> take it up with Atari



13:54:18  <DorpsGek> [OpenTTD/OpenTTD] cntgigadisk-creator commented on issue #14728: [Crash]: Open scenario from original TTD in map editor. https://github.com/OpenTTD/OpenTTD/issues/14728



14:02:41  <peter1138> Well.



14:57:20  <xarick> woah, this copilot stuff is metered



14:58:10  <LordAro> crazy expensive thing to run is limited? i am shocked



14:58:13  <xarick> I can't ask copilot anymore for today



15:08:38  <peter1138> Hmm.



15:09:51  *** _glx_ has joined #openttd



15:09:51  <_glx_> I think you can unlimit if you pay



15:11:39  <dwfreed> or you are a maintainer of a non-trivial open source project



15:13:15  <_jgr_> That sounds a bit like a rug-pull waiting to happen to me...



15:23:42  <dwfreed> Absolutely, and I don't use copilot, I just know the option is there at the moment



15:23:57  <dwfreed> I am maintainer of at least one non-trivial open source project



15:33:05  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #14728: [Crash]: Open scenario from original TTD in map editor. https://github.com/OpenTTD/OpenTTD/issues/14728



15:39:26  *** Flygon has joined #openttd



16:06:36  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #14728: [Crash]: Open scenario from original TTD in map editor. https://github.com/OpenTTD/OpenTTD/issues/14728



16:08:01  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on issue #14728: [Crash]: Open scenario from original TTD in map editor. https://github.com/OpenTTD/OpenTTD/issues/14728



16:08:16  <LordAro> nice



16:08:53  <LordAro> ah, but surely OTTD should say it's a corrupted file, rather than crash? :p



16:10:38  <peter1138> "Yes"



16:10:50  <peter1138> But too many things assert instead of throwing an exception.



16:11:12  <peter1138> I had to disable asserts to get those pictures of course.



16:12:55  <LordAro> i'm impressed it got that far



16:26:09  *** Wolf01 has joined #openttd



16:29:38  <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #14729: Fix: Don't set set town index for depot tiles. https://github.com/OpenTTD/OpenTTD/pull/14729



16:29:57  <peter1138> ^ With that fix in place, loading the corrupted scenario fails without crashing.



17:23:30  *** gelignite has joined #openttd



17:40:50  <xarick> i think i did the IsTunnelBelow part



17:41:27  <xarick> it's a bit rough codewise, but terraform lower terrain is faster



17:42:16  <xarick> touches map array, requires savegame conversion...



17:42:54  <xarick> the downside now is the act of removing a tunnel



17:43:14  <xarick> it can be expensive



17:44:59  <xarick> if the bit is set, it's going to look which bits can be reseted, that involves scanning the map, looking for tunnels



17:47:19  <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1431338281433432084/image.png?ex=68fd0d27&is=68fbbba7&hm=a2af138345489a6aef79d94f8f43cc2620d879aaab231efe066d2c1a57a339c4&



17:59:12  <DorpsGek> [OpenTTD/OpenTTD] JGRennison opened pull request #14730: Fix: Do not pre-fill industry production history for unused production slots https://github.com/OpenTTD/OpenTTD/pull/14730



18:24:21  <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1431347598807138518/image.png?ex=68fd15d4&is=68fbc454&hm=647236292b18252a06a7039a9bed662ae652b7631ee93d2f520c5680c0f7a903&



18:24:21  <andythenorth> peterer would badge filters be nice in autoraplece? 🙂



18:40:41  <locosage> omg, wth, KST trains build like 5x slower with python 3.14 than 3.12



18:40:51  <LordAro> impressive



18:43:43  <andythenorth> what broke?



18:44:50  <locosage> not sure yet, judging by the timing some buffer stuff or struck packing is much slower



18:45:27  <locosage> "Writing actions: ... 1.47" vs "Writing actions: ... 119.91"



18:46:40  <andythenorth> wonder what 3.13 does



18:46:47  <andythenorth> generally the direction has been 'faster'



18:47:00  <andythenorth> is it compatible with pypy?



18:47:19  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #14730: Fix: Do not pre-fill industry production history for unused production slots https://github.com/OpenTTD/OpenTTD/pull/14730#pullrequestreview-3378379028



18:48:59  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #14729: Fix: Don't set set town index for depot tiles. https://github.com/OpenTTD/OpenTTD/pull/14729#pullrequestreview-3378387620



18:49:11  <andythenorth> lazy developers, merging PRs 😛



18:49:16  <locosage> everything but that step is slightly faster



18:50:49  <talltyler> Not sure if you’re talking about us or Python, but we do have a lot of PRs that have been approved but not merged 🙂



18:59:52  <andythenorth> is that intended?



19:00:02  <andythenorth> don't they potentially drift and need rebased?



19:00:51  <DorpsGek> [OpenTTD/OpenTTD] PeterN approved pull request #14730: Fix: Do not pre-fill industry production history for unused production slots https://github.com/OpenTTD/OpenTTD/pull/14730#pullrequestreview-3378430415



19:15:49  <talltyler> I approved most of them, I cannot (should not) also merge them.



19:16:36  <talltyler> The rest are by maintainers, who if they don’t merge something might be reconsidering the approach, so best to let them do it when they’re ready.



19:17:49  <talltyler> I also wrote most of the unreviewed PRs in the 15.0 milestone, but that is another story. 🙂



19:19:50  <DorpsGek> [OpenTTD/OpenTTD] glx22 merged pull request #14714: Doc 80666a0: Fix typo in game script changelog https://github.com/OpenTTD/OpenTTD/pull/14714



19:20:15  <LordAro> andythenorth: potentially, but it's pretty rare on the whole



19:20:44  <DorpsGek> [OpenTTD/OpenTTD] glx22 merged pull request #14691: Codechange: Add prefix to dependabot commits https://github.com/OpenTTD/OpenTTD/pull/14691



19:22:26  <locosage> talltyler: you were also pretty much the only one adding stuff into that milestone 😜



19:23:58  <DorpsGek> [OpenTTD/OpenTTD] dependabot[bot] commented on pull request #14690: Upgrade: Bump the actions group with 3 updates https://github.com/OpenTTD/OpenTTD/pull/14690#issuecomment-3444604763



19:24:01  <DorpsGek> [OpenTTD/OpenTTD] dependabot[bot] closed pull request #14690: Upgrade: Bump the actions group with 3 updates https://github.com/OpenTTD/OpenTTD/pull/14690



19:24:08  <DorpsGek> [OpenTTD/OpenTTD] dependabot[bot] opened pull request #14731: Upgrade: Bump the actions group across 1 directory with 5 updates https://github.com/OpenTTD/OpenTTD/pull/14731



19:25:26  *** michi_cc has joined #openttd



19:25:26  <michi_cc> talltyler: I don't think we ever had any rule like that. I usually don't merge PRs of people that have commit rights themself (just because sometimes there might be multiple PRs with an intended order, and I'm lazy there). But for PRs from non-commiters, if you think it's actually good, just merge and approve. 🤷



19:26:08  <_glx_> nice, we have immediate confirmation of the working dependabot change



19:30:07  <DorpsGek> [OpenTTD/OpenTTD] michicc merged pull request #14727: Codechange: Prefer string equality instead of comparison. https://github.com/OpenTTD/OpenTTD/pull/14727



19:30:12  <DorpsGek> [OpenTTD/OpenTTD] michicc commented on pull request #14727: Codechange: Prefer string equality instead of comparison. https://github.com/OpenTTD/OpenTTD/pull/14727#issuecomment-3444633474



19:30:54  <DorpsGek> [OpenTTD/OpenTTD] michicc merged pull request #14729: Fix: Don't set set town index for depot tiles. https://github.com/OpenTTD/OpenTTD/pull/14729



19:31:19  <_glx_> yeah all the `.compare()` stuff was done to replace `strcmp()`



19:33:07  <michi_cc> And at that time still targeted C++17 I think, which meant no `starts_with` anyway.



19:33:59  <DorpsGek> [OpenTTD/OpenTTD] michicc merged pull request #14380: Fix #14240: Remember previous GUI scale when toggling auto-detect https://github.com/OpenTTD/OpenTTD/pull/14380



19:34:02  <DorpsGek> [OpenTTD/OpenTTD] michicc closed issue #14240: [Bug]: Auto-detect doesn't remember the previous interface size when turned on, then back off https://github.com/OpenTTD/OpenTTD/issues/14240



19:34:39  <DorpsGek> [OpenTTD/OpenTTD] michicc merged pull request #14730: Fix: Do not pre-fill industry production history for unused production slots https://github.com/OpenTTD/OpenTTD/pull/14730



19:45:16  <locosage> oh, looks like yacc is the cause for huge slowdown in 3.14



19:45:20  <locosage> wonder if nml is affected



19:57:12  <xarick> Don't set set



19:58:25  <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1431371272972927228/Captura_de_ecra_2025-10-24_205632.png?ex=68fd2be1&is=68fbda61&hm=3bfb63211fe344b412d0d6d279018cd7f0fe93ba7a27aea48784f39a791c7377&



19:58:25  <xarick> I'm doing forbidden math operations



20:08:59  <locosage> ok, it was just silly me creating a new yacc parser for every switch 😅



20:09:14  <locosage> now it's 15x faster 🤣



20:11:05  <locosage> dunno why 3.14 got so slow but it wasn't a good code anyway



20:14:31  <xarick> should I use DistanceManhattan? I don't really need to know the manhattan distance between two tiles, only if one is further than the other



20:23:09  *** Wormnest has joined #openttd



20:25:21  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler merged pull request #14615: Feature: Game units for height. https://github.com/OpenTTD/OpenTTD/pull/14615



20:25:34  <talltyler> michi_cc: Good to know, thanks. 🙂



20:27:15  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler merged pull request #14614: Add: Height difference in bridge is too low error message. https://github.com/OpenTTD/OpenTTD/pull/14614



20:29:43  <LordAro> locosage: ;-;



20:32:12  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler merged pull request #10063: Change: Scale towns/industries by amount of land tiles. https://github.com/OpenTTD/OpenTTD/pull/10063



20:32:54  <talltyler> Okay, that’s enough merging for me 🙂



20:33:44  <talltyler> Still happy to do the work for another beta release whenever someone feels like reviewing PRs in the milestone 😄



20:41:49  *** reldred has joined #openttd



20:41:49  <reldred> Y’all been busy while I was asleep 😆



20:52:02  <LordAro> reldred: slacker



20:54:41  *** zamora1893 has joined #openttd



20:54:41  <zamora1893> https://media.discordapp.net/attachments/1134258139437469756/1431368220173926552/IMG_0462.jpg?ex=68fd2909&is=68fbd789&hm=3d7e104bcb8fd6349321f766a5b4441067a7c9265c6203d037e09956b2f9a7b6&=&format=webp&width=676&height=902



20:54:41  <zamora1893> https://media.discordapp.net/attachments/1134258139437469756/1431368220614590504/IMG_0464.png?ex=68fd2909&is=68fbd789&hm=c087243f35bbdb0d1669994535f4e6c5fbe9c034f2ef792f794a8ec0d4916576&=&format=webp&quality=lossless&width=516&height=902



20:54:41  <zamora1893> https://media.discordapp.net/attachments/1134258139437469756/1431368221205729401/IMG_0465.png?ex=68fd2909&is=68fbd789&hm=2beb3b843be82767cc62fc3e93e0050f600abbacc24455359fa382095299adae&=&format=webp&quality=lossless&width=515&height=902



20:55:36  <talltyler> Discord Moderator



20:55:52  <reldred> LordAro: Yeah 😔



20:58:17  <rito12_51026> Wow it took 3 years to develop that PR



21:01:27  *** _zephyris has joined #openttd



21:01:27  <_zephyris> Woop!



21:06:18  <xarick> I need to brute force test tunnel removal, how am I going to do this 🙁



21:15:37  *** Heiki has joined #openttd



21:19:12  <xarick> Any thing that could be improved in this implementation?<https://github.com/OpenTTD/OpenTTD/compare/master...SamuXarick:OpenTTD:Add-Helper-for-is-tunnel-below>



21:28:06  <DorpsGek> [OpenTTD/OpenTTD] michicc commented on pull request #14408: Fix: Player-placed house protection for houses should override the callback https://github.com/OpenTTD/OpenTTD/pull/14408#issuecomment-3444968481



21:50:46  *** keikoz has quit IRC (Ping timeout: 480 seconds)



21:51:07  <talltyler> Oof.



22:03:02  <_zephyris> So, Reddit would really like that gamepad PR



22:04:24  *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)



22:11:47  <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1431404837861458093/image.png?ex=68fd4b23&is=68fbf9a3&hm=3f29ccd13e41d9a19e2e37ce3d9c9b6f2e81877f43c005d0c6cabf690ae5cc28&



22:11:47  <xarick> moment of truth



22:11:59  <xarick> removing tunnels should not assert



22:13:56  *** jinks has quit IRC (Quit: ZNC - http://znc.in)



22:14:17  *** jinks has joined #openttd



22:16:09  *** Wormnest has quit IRC (Ping timeout: 480 seconds)



22:29:48  <xarick> oh interesting



22:29:57  <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1431409406523998228/image.png?ex=68fd4f64&is=68fbfde4&hm=03023792690c2fd3ec1306d2397eccdebfcb8c8a8f4fd2124d6d0a163f3089e3&



22:29:57  <xarick> building tunnels also became faster



22:35:04  *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)



22:35:11  <DorpsGek> [OpenTTD/OpenTTD] cntgigadisk-creator commented on issue #14728: [Crash]: Open scenario from original TTD in map editor. https://github.com/OpenTTD/OpenTTD/issues/14728



22:35:11  <michi_cc> talltyler: Well, you wanted reviews 😎



22:36:03  *** WormnestAndroid has joined #openttd



22:37:36  <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1431411332556521502/image.png?ex=68fd5130&is=68fbffb0&hm=59bcc97d2d98c6374dc8659e9207af7cea64e98252f4f2341b9a829bb4104829&



22:37:36  <xarick> well, this is the drawback... removing tunnels is slower



22:46:15  <xarick> no asserts occurred!



22:46:19  <xarick> very happy!



22:58:47  <talltyler> michi_cc: No, it's a good spot, thanks for that. Just oof because I've made a mess for myself. 😄



23:13:02  <michi_cc> IMHO the best way would be to ignore the NewGRF protection in the house picker fully. If the player protects the house, it's protected. If not, whatever the NewGRF says goes. I.e. the in-map bit should probably only be the user protection and nothing else.



23:22:29  <reldred> michi_cc: that's basically what all the people I've spoken to want



23:53:44  *** WormnestAndroid has quit IRC (Ping timeout: 480 seconds)