IRC logs for #openttd on OFTC at 2026-06-22
#openttd / 2026 / 06 
            

00:21:08  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #15589: Change: Selecting Rail/Roadtype Updates build Toolbar Instead of Closing and Reopening https://github.com/OpenTTD/OpenTTD/pull/15589#pullrequestreview-4540482779



01:25:29  *** Tirili has quit IRC (Quit: Leaving)



01:49:34  *** Wormnest has quit IRC (Quit: Leaving)



02:30:10  <DorpsGek> [OpenTTD/OpenTTD] mmtunligit updated pull request #15589: Change: Selecting Rail/Roadtype Updates build Toolbar Instead of Closing and Reopening https://github.com/OpenTTD/OpenTTD/pull/15589



02:38:41  *** Flygon has quit IRC (Quit: A toaster's basically a soldering iron designed to toast bread)



02:57:26  *** Flygon has joined #openttd



03:40:05  *** Phileman has joined #openttd



03:43:30  *** Philemon has quit IRC (Ping timeout: 480 seconds)



06:19:12  <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/26e5e387612586947be266a6ff82050ccf693df4



06:19:13  <DorpsGek>   - Update: Translations from eints (by translators)



07:50:53  *** SigHunter has quit IRC (Remote host closed the connection)



07:52:54  *** SigHunter has joined #openttd



08:01:23  <DorpsGek> [OpenTTD/OpenTTD] PeterN requested changes for pull request #15589: Change: Selecting Rail/Roadtype Updates build Toolbar Instead of Closing and Reopening https://github.com/OpenTTD/OpenTTD/pull/15589#pullrequestreview-4542137019



08:03:24  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #15700: Codechange: make TransportType a scoped enum https://github.com/OpenTTD/OpenTTD/pull/15700#issuecomment-4766165949



08:34:27  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #15700: Codechange: make TransportType a scoped enum https://github.com/OpenTTD/OpenTTD/pull/15700#issuecomment-4766404482



08:40:20  <peter1138> I can't though :D



08:59:50  <DorpsGek> [OpenTTD/OpenTTD] mmtunligit updated pull request #15589: Change: Selecting Rail/Roadtype Updates build Toolbar Instead of Closing and Reopening https://github.com/OpenTTD/OpenTTD/pull/15589



09:04:58  <DorpsGek> [OpenTTD/survey-web] survey-summary[bot] pushed 1 commits to main https://github.com/OpenTTD/survey-web/commit/1e5754b07f031c2e06b7187e8ed3970d2afd80a9



09:04:59  <DorpsGek>   - Add: summary for week 25 of 2026 (by OpenTTD Survey)



09:05:31  <DorpsGek> [OpenTTD/OpenTTD] mmtunligit commented on pull request #15589: Change: Selecting Rail/Roadtype Updates build Toolbar Instead of Closing and Reopening https://github.com/OpenTTD/OpenTTD/pull/15589#pullrequestreview-4542580844



09:17:18  <DorpsGek> [OpenTTD/OpenTTD] mmtunligit updated pull request #15589: Change: Selecting Rail/Roadtype Updates build Toolbar Instead of Closing and Reopening https://github.com/OpenTTD/OpenTTD/pull/15589



09:22:52  *** WormnestAndroid has quit IRC (Remote host closed the connection)



09:23:19  *** WormnestAndroid has joined #openttd



09:35:31  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #15589: Change: Selecting Rail/Roadtype Updates build Toolbar Instead of Closing and Reopening https://github.com/OpenTTD/OpenTTD/pull/15589#pullrequestreview-4542793526



09:43:28  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #15589: Change: Selecting Rail/Roadtype Updates build Toolbar Instead of Closing and Reopening https://github.com/OpenTTD/OpenTTD/pull/15589#issuecomment-4766958920



09:45:14  *** Alkel_U3 has quit IRC (Quit: maintenance)



09:46:32  *** Alkel_U3 has joined #openttd



09:52:29  *** Flygon has quit IRC (Read error: Connection reset by peer)



09:59:27  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #15700: Codechange: make TransportType a scoped enum https://github.com/OpenTTD/OpenTTD/pull/15700



09:59:54  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #15761: Codechange: make gamelog enums scoped https://github.com/OpenTTD/OpenTTD/pull/15761



10:01:47  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #15641: Codechange: simplify LookupWithBuildOnSlopes in script_road.cpp. https://github.com/OpenTTD/OpenTTD/pull/15641



10:02:14  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #15740: Codechange: Make comments in table/sprites.h recognised by doxygen https://github.com/OpenTTD/OpenTTD/pull/15740



10:02:40  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #15574: Update: doxygen version to 1.17.0. https://github.com/OpenTTD/OpenTTD/pull/15574



10:32:02  <peter1138> Spam! :D



11:35:43  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #15762: Codechange: use proper names for SLV_20 - SLV_39 https://github.com/OpenTTD/OpenTTD/pull/15762#pullrequestreview-4543601181



11:46:50  *** talltyler has joined #openttd



11:46:50  <talltyler> Goes it release beta1 today?



11:47:35  <talltyler> Just need a changelog update from peter1138, and some website re-approvals after I change the dates on the posts. 🙂



12:16:16  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 updated pull request #15762: Codechange: use proper names for SLV_20 - SLV_39 https://github.com/OpenTTD/OpenTTD/pull/15762



12:37:26  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #15762: Codechange: use proper names for SLV_20 - SLV_39 https://github.com/OpenTTD/OpenTTD/pull/15762#pullrequestreview-4544078571



12:38:00  <_zephyris> Can do...



12:50:53  <DorpsGek> [OpenTTD/OpenTTD] mmtunligit commented on pull request #15589: Change: Selecting Rail/Roadtype Updates build Toolbar Instead of Closing and Reopening https://github.com/OpenTTD/OpenTTD/pull/15589#issuecomment-4768465779



12:52:05  *** Wormnest has joined #openttd



12:57:54  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #15762: Codechange: use proper names for SLV_20 - SLV_39 https://github.com/OpenTTD/OpenTTD/pull/15762



13:04:42  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #15763: Codechange: use proper names for SLV_40 - SLV_64 https://github.com/OpenTTD/OpenTTD/pull/15763



13:08:13  <mmtunligit> Did town limit get lifted recently? Or does json import bypass it



13:08:49  <mmtunligit> There’s some guy making a uk scenario programmatically who has way more then 5000



13:12:52  *** peter1138[d] has joined #openttd



13:12:52  <peter1138[d]> The new-game town limit only applies to towns creating during world gen. As long as there's enough space you can place as many as you like. The town importer will try to place everything that's in the file.



13:14:49  <mmtunligit> Oh huh



13:20:31  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #15763: Codechange: use proper names for SLV_40 - SLV_64 https://github.com/OpenTTD/OpenTTD/pull/15763#pullrequestreview-4544416514



13:21:18  <peter1138> FFS, 1) wrong client 2) still managed to make mistakes.



13:37:49  *** lordgarysugar has joined #openttd



13:37:50  <lordgarysugar> Is there a reason that importing towns from .json takes many times longer than generating random towns? I'm not sure if it's a bug or what. When I accidentally swapped the X and Y and all the towns were over the ocean, it placed all the signs pretty instantly. So I think it's a problem with town placement being slow rather than the actual parsing of the file



13:42:19  <talltyler> Not a bug, it’s the growth of the towns to the target population that takes time.



13:42:41  <talltyler> They get placed and then repeatedly expanded until they hit the target, or give up



13:43:02  <talltyler> What population are you using?



13:46:23  <lordgarysugar> barely any of the towns have population data so they are all set to 0



13:52:09  <lordgarysugar> I tried changing all the populations to 1 to see if the 0 was tripping it up but it's still taking just as long even though none of them would need to be expanded?



14:08:22  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #15763: Codechange: use proper names for SLV_40 - SLV_64 https://github.com/OpenTTD/OpenTTD/pull/15763



14:13:23  <talltyler> Is your heightmap really rough/hilly? Towns can only spawn on flat tiles so if the target tile isn’t flat there’s a spiral search around that tile to find a suitable spot.



14:21:39  <lordgarysugar> Not in the East of England, the 1700 villages there take about 10 minutes to process and it's almost entirely flat



14:29:17  <LordAro> 10 minutes feels excessive



14:29:33  <LordAro> also, can confirm, East of England is all villages and it's almost entirely flat



14:35:53  <talltyler> Hmm, send me the JSON file if you want and I’ll take a look at what’s happening



14:40:07  <lordgarysugar> dm or just paste it here?



14:40:29  <LordAro> oh yeah, because pasting a huge json file into a text channel is a great idea



14:40:32  <LordAro> can you imagine



14:40:45  <LordAro> this is sarcasm, DO NOT



14:40:50  <lordgarysugar> I can upload it as a file 😭



14:41:04  <LordAro> that would be acceptable :p



14:41:25  <lordgarysugar> the same thing happens with a flat map so I don't think the heightmap/terrain has anything to do with the issue



14:43:12  <lordgarysugar> https://cdn.discordapp.com/attachments/1008473233844097104/1518627424722489446/EastEng_Vill_pop_1.json?ex=6a3a9b80&is=6a394a00&hm=497707101b01c26915971ef67c36b5f328cb1a03ebd65c84cf6e3799acd12f67&



14:43:12  <lordgarysugar> here is a file of 1861 villages, I'm loading them on a 16kx16k map if that makes a difference



14:52:39  <LordAro> well that's a ridiculous map size



14:52:45  <LordAro> but probably, yeah



14:52:51  <LordAro> also makes it jgr's problem ;)



15:01:18  <lordgarysugar> hmm, I think it is a map size problem



15:01:45  <lordgarysugar> loading 139 towns on a 4k flat map takes 4 seconds, loading the same on a 16k map takes 45 seconds



15:04:37  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #15764: Codechange: use proper names for SLV_65 - SLV_89 https://github.com/OpenTTD/OpenTTD/pull/15764



15:07:43  <mmtunligit> Not suprising that multiplying map size by 16 increases processing time by a similar amount



15:08:37  <lordgarysugar> I don't see why it should, it's the same number of towns and knows where to put them



15:14:29  <talltyler> That’s a JGRPP bug then 🙂



15:19:42  <lordgarysugar> Well, I'd rather it be a bug than a performance issue



15:20:33  *** _jgr_ has joined #openttd



15:20:33  <_jgr_> When I am finished with work I will look at it



15:23:14  <lordgarysugar> you can look at importing signs too if you fancy it 😂



16:05:04  <will_marshall_> If I had a fiver for every time my promise to fix a bug was met with a feature request I would be in the same position I'm in now but I'd be less sober because I would have spent it all on drink.



16:09:00  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #15589: Change: Selecting Rail/Roadtype Updates build Toolbar Instead of Closing and Reopening https://github.com/OpenTTD/OpenTTD/pull/15589#issuecomment-4770382432



16:15:26  <_jgr_> The cuplrit appears to be `UpdateNearestTownForRoadTiles`



16:19:57  <peter1138> Iterating the entire map twice for every town seems like a "good" idea :)



16:20:22  <peter1138> 16kx16k is going to be slower than 4kx4k, inevitably.



16:25:37  <lordgarysugar> will_marshall_: Forgive me but it is annoyingly almost possible because you do get signs if a town can't be placed but there's no way to cause town placement to fail for an entire list



16:26:13  <talltyler> A proper sign importing feature would also let you pick the colour of the sign 🙂



16:26:28  <will_marshall_> Well if you're doing an import of xx towns at least you can place all the towns and then update all the roads at the end?



16:27:08  <talltyler> Rather than trying to use a debugging tool 🙂



16:27:16  <will_marshall_> Adapting town importing to importing other features would be a good "first PR" job.



16:27:40  <lordgarysugar> talltyler: Yes I noticed that new? feature yesterday so I can have suburb labels be different colours from peak labels or water body labels



16:28:08  <will_marshall_> Signs have colour because companies have colour.



16:28:42  <talltyler> No, it’s a new feature that lets unowned signs have text colour. Not sign colour! 🙂



16:28:52  <will_marshall_> Oh, neat.



16:28:52  <talltyler> Scenario Editor only



16:29:25  <talltyler> Kuhnovic wanted to label things, forget if it was rivers or mountains 🙂



16:29:39  <lordgarysugar> Yeah, they're only text without a block background which actually means a lot of the sign colours have poor legibility over green tiles



16:31:23  <_zephyris> talltyler: I'm around for proof-reading and approvals tonight btw



16:33:00  <_jgr_> will_marshall_: Yes, that is what is done for the usual way of bulk adding random towns in the scenario editor



16:33:20  <_jgr_> Should not be too difficult to do the same here



16:35:22  <will_marshall_> i guess town founding during gameplay is so rare (gamescripts maybe?) that talking about any other optimizations for that function are totally moot.



17:14:39  <peter1138> Oh, Well.



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



18:22:09  <_jgr_> Should be sorted now



18:45:02  *** MinchinWeb[m] has quit IRC (Ping timeout: 480 seconds)



18:45:07  *** MinchinWeb[m] has joined #openttd



18:46:36  *** kuhnovic has joined #openttd



18:46:36  <kuhnovic> talltyler: Both 🙂 I wanted a nice way to label landmarks for my Southern Norway map. And I still need to do that...



19:02:12  *** MinchinWeb[m] has quit IRC (Read error: Connection reset by peer)



19:11:19  *** MinchinWeb[m] has joined #openttd



19:46:25  *** gelignite has quit IRC (Ping timeout: 480 seconds)



19:53:08  <peter1138> https://www.twitch.tv/theromero Bit of Quake



20:08:45  <DorpsGek> [OpenTTD/nml] Rito13 opened pull request #434: Codefix: Index equal to the list length is also invalid https://github.com/OpenTTD/nml/pull/434



20:10:24  <DorpsGek> [OpenTTD/nml] LordAro commented on pull request #434: Codefix: Index equal to the list length is also invalid https://github.com/OpenTTD/nml/pull/434#issuecomment-4772585996



20:19:07  <DorpsGek> [OpenTTD/nml] Rito13 updated pull request #434: Codefix: Index equal to the list length is also invalid https://github.com/OpenTTD/nml/pull/434



20:32:59  *** WormnestAndroid has quit IRC (Remote host closed the connection)



20:33:03  *** WormnestAndroid has joined #openttd



20:38:49  *** MinchinWeb[m] has quit IRC (Read error: Connection reset by peer)



20:39:05  *** MinchinWeb[m] has joined #openttd



20:43:08  *** Flygon has joined #openttd



22:12:39  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #15765: Codechange: use proper names for SLV_90 - SLV_114 https://github.com/OpenTTD/OpenTTD/pull/15765



22:13:27  <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #15766: Codechange: use proper names for SLV_115- SLV_144 https://github.com/OpenTTD/OpenTTD/pull/15766



23:09:00  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #15764: Codechange: use proper names for SLV_65 - SLV_89 https://github.com/OpenTTD/OpenTTD/pull/15764#pullrequestreview-4548775237



23:09:05  *** Flygon has quit IRC (Read error: Connection reset by peer)



23:13:41  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #15765: Codechange: use proper names for SLV_90 - SLV_114 https://github.com/OpenTTD/OpenTTD/pull/15765#pullrequestreview-4548790610



23:19:10  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #15766: Codechange: use proper names for SLV_115 - SLV_144 https://github.com/OpenTTD/OpenTTD/pull/15766#pullrequestreview-4548803100