Times are UTC Toggle Colours
00:05:44 *** EmperorJake has joined #openttd 00:05:44 <EmperorJake> AXIS merges Stainless and Alloy steel 00:10:01 *** bryjen has joined #openttd 00:35:40 <DorpsGek> [OpenTTD/OpenTTD] anatolyeltsov commented on pull request #7575: Feature: Industry production graph https://github.com/OpenTTD/OpenTTD/pull/7575#issuecomment-1454938018 00:38:56 *** bryjen has quit IRC 00:44:00 *** WormnestAndroid has quit IRC 00:44:02 *** WormnestAndroid has joined #openttd 00:49:55 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #7575: Feature: Industry production graph https://github.com/OpenTTD/OpenTTD/pull/7575#issuecomment-1454941606 00:56:11 *** Tirili has joined #openttd 01:04:23 <DorpsGek> [OpenTTD/OpenTTD] glx22 updated pull request #10533: Codechange: Use SQInteger for generic numbers in script API https://github.com/OpenTTD/OpenTTD/pull/10533 01:11:16 *** Eddi|zuHause has joined #openttd 01:20:16 <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #10539: Fix: game scripts must not build with non-existing road types https://github.com/OpenTTD/OpenTTD/pull/10539#pullrequestreview-1325142422 01:30:09 <DorpsGek> [OpenTTD/BaNaNaS] glx22 commented on pull request #130: Feature: convert mentions of regions in tags to regions https://github.com/OpenTTD/BaNaNaS/pull/130#pullrequestreview-1325142814 01:50:45 *** Tirili has quit IRC 02:04:32 *** WormnestAndroid has quit IRC 02:05:18 *** WormnestAndroid has joined #openttd 02:24:15 *** lobstarooo__ has quit IRC 02:54:29 *** Wormnest has quit IRC 03:02:32 *** Vadtec has quit IRC 03:02:37 *** Vadtec has joined #openttd 03:11:42 *** bryjen has joined #openttd 03:28:27 *** D-HUND has joined #openttd 03:31:52 *** debdog has quit IRC 03:33:23 *** D-HUND is now known as debdog 04:13:54 *** TROILUS has quit IRC 04:14:17 *** TROILUS has joined #openttd 04:39:36 *** keikoz has joined #openttd 05:19:16 *** HerzogDeXtEr has quit IRC 06:17:42 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #10539: Fix: game scripts must not build with non-existing road types https://github.com/OpenTTD/OpenTTD/pull/10539 06:25:05 *** Flygon has joined #openttd 07:11:05 *** TROILUS has quit IRC 07:11:46 *** TROILUS has joined #openttd 07:25:08 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #10533: Codechange: Use SQInteger for generic numbers in script API https://github.com/OpenTTD/OpenTTD/pull/10533#pullrequestreview-1325161594 07:46:11 <petern> glx22viaGitHub: Hidden road types should not be buildable by towns either 07:47:15 <petern> I think I had a patch or pr for rhat 07:57:50 *** bryjen has quit IRC 08:14:28 <andythenorth> trying to get chatGPT to read FIRS .dot cargoflow 08:16:29 *** nielsm has joined #openttd 08:18:42 <andythenorth> hmm 08:19:02 <andythenorth> is chatGPT the best teddy bear coding partner? 08:20:23 <andythenorth> `As an AI language model, I'm not a physical teddy bear and I cannot physically interact with you like a teddy bear could. However, I can help you with a wide range of language-related tasks and answer your questions to the best of my knowledge.` 08:25:11 <dP> mayhap it's a duck 09:09:13 <DorpsGek> [OpenTTD/BaNaNaS] TrueBrain commented on pull request #130: Feature: convert mentions of regions in tags to regions https://github.com/OpenTTD/BaNaNaS/pull/130#pullrequestreview-1325176948 09:09:43 <DorpsGek> [OpenTTD/BaNaNaS] TrueBrain commented on pull request #130: Feature: convert mentions of regions in tags to regions https://github.com/OpenTTD/BaNaNaS/pull/130#pullrequestreview-1325177024 09:13:40 <TrueBrain> hmm .. according to ISO-3166, Wales doesn't exist .. 09:15:16 <TrueBrain> I do wonder how they got their TLD in that case .. 😛 09:17:25 <TrueBrain> should have `GB-WLS` assigned to it .. hmm 09:17:33 <andythenorth> I can verify that Wales does exist 09:17:36 <andythenorth> I have been there 09:19:47 *** nielsm has quit IRC 09:19:50 <TrueBrain> weird, wikipedia has a code assigned to it, but the ISO from 2022 does not 09:20:18 <TrueBrain> it does to the subparts of Wales 09:20:36 <TrueBrain> similar issues with the other countries in UK, like Scotland 09:21:06 <TrueBrain> yeah, okay, so England, Northern Ireland, Scotland and Wales are missing, in that sense 09:21:14 <TrueBrain> guess it got too complicated 😄 09:21:17 <TrueBrain> a country in a country 😛 09:22:05 <andythenorth> such 09:22:05 <TrueBrain> `on 28 September 2021, UK replaced GB as the official country code on car registration plates` 09:22:08 <TrueBrain> the UK is so weird 09:22:25 <PopeofMemes> you think thats bad? 09:22:28 <andythenorth> we're quite consistent in our inconsistency 09:22:41 <TrueBrain> if weird is bad? I leave that for the historians 09:22:45 <andythenorth> we've been doing it for thousand years or more 😛 09:22:48 <PopeofMemes> https://cdn.discordapp.com/attachments/1008473233844097104/1081869412862275664/image.png 09:22:48 <PopeofMemes> look at this 09:23:09 <TrueBrain> weird landborders we have all over the world; that is nothing weird 09:23:20 <TrueBrain> but not knowing if you want to be called GB or UK .. that is just odd 😛 09:23:20 <PopeofMemes> there are Dutch parts in Belgium, in the Netherlands, in Belgium, **In the Netherlands** 09:23:52 <PopeofMemes> imagine just going to the bathroom in Belgium and eating breakfast in the Netherlands 09:23:58 <TrueBrain> I think I will just manually add those four entries, as this is just too weird not to have 09:24:11 <andythenorth> it's very simple, United Kingdom of Great Britain and Northern Ireland 09:24:45 <andythenorth> The island of Ireland isn't Ireland 09:24:48 <TrueBrain> https://www.iso.org/obp/ui/#iso:code:3166:GB does list GB-ENG, but the full ISO 3166-2 does not 09:25:04 <TrueBrain> it is like they half-way were like: nah, we're good 09:32:05 <TrueBrain> seems GB is the only country with "this issue" 09:34:33 *** nielsm has joined #openttd 09:46:33 <TrueBrain> I am seriously considering removing the ability to name 1 version of a NewGRF different than the other .. it doesn't actually seem to be used in a way that is helpful, and it looks like it is more likely to confuse the author 🙂 09:52:20 <andythenorth> I was surprised it could be done 09:52:44 <TrueBrain> yeah, and take your GRFs for example .. the package itself is called without `(trains)`, and some versions do add it 09:52:49 <TrueBrain> doesn't seem like you actually intended that 🙂 09:53:26 <andythenorth> no 09:53:36 <andythenorth> I intended the package name to change everywhere for that package 09:53:50 <TrueBrain> yeah, and I see more of those packages 🙂 09:54:14 <andythenorth> If I want to fundamentally change the behaviour of the grf 09:54:14 <andythenorth> Such that it needs a name change for a specific version 09:54:14 <andythenorth> Then it needs a grfid bump 09:55:14 <TrueBrain> okay ... during this initial conversion, shall I also look at titles to see if I can spot regions? 09:58:30 <andythenorth> makes sense 10:00:29 <TrueBrain> the chances of misclassifying increases 🙂 10:03:00 <DorpsGek> [OpenTTD/BaNaNaS] TrueBrain updated pull request #130: Feature: convert mentions of regions in tags to regions https://github.com/OpenTTD/BaNaNaS/pull/130 10:08:48 <DorpsGek> [OpenTTD/BaNaNaS] TrueBrain updated pull request #130: Feature: convert mentions of regions in tags to regions https://github.com/OpenTTD/BaNaNaS/pull/130 10:19:27 <DorpsGek> [OpenTTD/BaNaNaS] TrueBrain updated pull request #130: Feature: convert mentions of regions in tags to regions https://github.com/OpenTTD/BaNaNaS/pull/130 10:20:07 <TrueBrain> okay, I need some extra set of eyes on this ... 10:20:07 <TrueBrain> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf?region=%28none%29 10:20:07 <TrueBrain> Is there anything in that list that trivially points to a region? 10:21:16 <andythenorth> did you engineer that 'Furry' would be first? 😛 10:21:21 <andythenorth> not judging anything or anyone 😛 10:21:31 <TrueBrain> I feel judged 10:24:08 *** Brickblock1 has joined #openttd 10:24:08 <Brickblock1> Arabian town names does not have a region 10:24:14 <TrueBrain> because that is not a region 🙂 10:24:37 <dP> Polish PKP Set 2.0 10:24:41 <TrueBrain> in the description it lists some countries, but I cannot automatically pick up on those 🙂 10:24:44 <dP> and a few serbian ones 10:25:12 <Rubidium> Alpine climate? The alps are considered a region, although there are several regions called alps... 10:25:32 <TrueBrain> different kind of "region", not one 3166 sees as a region 😄 10:26:05 <Brickblock1> some train sets also don't show up 10:26:17 <TrueBrain> "don't show up", what do you mean? 10:26:44 <Brickblock1> don't have a region is what I meant, sorry for the confusion 10:26:51 <Rubidium> you didn't add towns, right? 10:26:55 <Brickblock1> but they might not have tags 10:26:55 <TrueBrain> like? "some" is a bit too vague 😄 Sorry 🙂 10:27:09 <Brickblock1> CP-trainset and PNR trains 10:27:10 <TrueBrain> Rubidium: no, only 3166-2 at lowest resolution 10:27:20 <TrueBrain> Brickblock1: yeah, impossible for me to figure out where they belong 🙂 10:27:37 <TrueBrain> I could peek in the description, but that becomes very tricky 10:27:42 <TrueBrain> will check in a moment how bad that would be 😄 10:27:52 <Rubidium> TrueBrain: all the Auz ones, but that's just slang interfering there I guess 10:28:16 <TrueBrain> I have no clue what it actually refers to 10:28:23 <TrueBrain> there is nothing in any of the descriptions to give an indication 10:28:33 *** sla_ro|master has joined #openttd 10:28:35 <Rubidium> Auz -> Australia I'd reckon 10:28:46 <TrueBrain> but that is a guess 10:28:58 <TrueBrain> at least, I don't know 🙂 It could also be used as trademark name, and be of something different 😛 10:29:06 <Rubidium> Lombard town names -> town names of the Lombardy region, but that's like having Frisian town names 10:29:31 <TrueBrain> okay, the tt-forums post does mention Australian sometimes, but also American West .. 10:29:33 <dP> UK Town Names 10:29:39 <Rubidium> "AuzObjects (From the Land Down Under)" https://www.tt-forums.net/viewtopic.php?f=26&t=75657 10:30:12 <dP> multiple UK grfs actually 10:30:21 <TrueBrain> I rather not classify something that misclassify .. but looking at the tt-forums post, I feel a bit more empowered to classify it as Australian 😄 10:31:00 <Brickblock1> some of them are mostly generic tho 10:31:15 <TrueBrain> okay, then I will not set a region to Auz, and leave it to the author 🙂 10:31:21 <Rubidium> "Auustralian Wide Ballast Tracks" <- someone made a typo 10:31:30 <TrueBrain> so many typos 10:31:31 <TrueBrain> SO MANY 10:31:41 <Rubidium> though, he description mentions Australian 10:32:03 <Brickblock1> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/50438151 10:32:20 <dP> a lot of town names actually 10:32:31 <dP> Tibetan, Lombard, Winnipeg... 10:32:38 <Brickblock1> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/554b5201 10:33:07 <Rubidium> TrueBrain: you ditched the UN MU49? 10:33:17 <TrueBrain> Rubidium: no?! 10:33:49 <Rubidium> "Names of Texas" 10:33:49 <Brickblock1> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/474a0003 10:33:55 <Brickblock1> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/44442303 10:34:05 <TrueBrain> these kind of remarks always confuse me, as people don't state what they see, but they make a conclusion based on something they see .. doesn't help me Rubidium 😄 10:34:37 <TrueBrain> UK is annoying .. as it is too short to do generic matching on 😛 10:34:59 <TrueBrain> tnx Brickblock1 , added the bunch 10:37:50 <dP> American Interurban Set 10:38:37 <dP> nvm, filter reset xD 10:39:56 <Brickblock1> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/30f1fce0 is in the wrong spot 10:40:17 <Rubidium> TrueBrain: which one confused you? The question about UN M49, if so you previously stated "only 3166-2" so I wondered whether state names I have seen in UN M49 would be considered regions by you(r code) or not. Or was it the "Names of Texas", then that is actually the name of a NewGRF 10:40:26 <TrueBrain> Brickblock1: how that happened, lol. Let me fix 🙂 10:40:46 <Brickblock1> has Chicago in description I guess 10:41:08 <TrueBrain> Rubidium: no, I said 3166-2 was the lowest resolution 🙂 I didn't say it was the only set used 🙂 But lowest -> highest I guess .. 10:41:58 <TrueBrain> Brickblock1: it has `chicago` in the tags 10:42:13 <Brickblock1> oh 10:42:21 <Brickblock1> https://cdn.discordapp.com/attachments/1008473233844097104/1081889433109942453/image.png 10:42:21 <Brickblock1> also this 10:42:24 <TrueBrain> but that is still a misclasfficication 😛 Let me fix 🙂 10:42:33 <TrueBrain> hahaha 10:42:34 <TrueBrain> nice 😄 10:42:52 <Brickblock1> I wouldn't call those Canadian 10:43:30 <TrueBrain> `canada` in the tags 10:43:33 <TrueBrain> but .. yeah .. let's not 10:44:47 <Brickblock1> Greenlandic town names show up with Europe but I don't know how that would be changed 10:45:06 <TrueBrain> Rubidium: btw, for a complete picture, Texas is not in UN M49, as that just defines cities and regions upwards. Texas however is in 3166-2, so I mapped it now 10:46:04 <TrueBrain> Brickblock1: tags also included denmark, but let me add greenland too 🙂 10:46:23 <Brickblock1> yeah Denmark makes some sense 10:47:50 <Brickblock1> city objects being France feels wrong to me 10:48:01 <Brickblock1> but I assume it was in tags 10:48:08 <TrueBrain> newgrf id? 10:48:40 <TrueBrain> found it 10:48:58 <TrueBrain> yeah, had "new york" and "paris" in the tags 10:49:05 <TrueBrain> new york didn't make it 😛 10:49:28 <TrueBrain> reloaded the import to fix all the mentioned things 10:50:04 <FLHerne> TrueBrain: many of those have implicit regions that would be pretty hard to autodetect 10:50:06 <Brickblock1> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/0100403a 10:50:15 <TrueBrain> FLHerne: "those"? 10:52:57 <FLHerne> TrueBrain: like, NOHAB is a Swedish locomotive, all the Long Vehicles vehicles are European, the Transrapid track set is either German or Chinese 10:53:47 <FLHerne> Ikarus trucks are Hungarian 10:53:55 <TrueBrain> FLHerne: yeah, I have no intention of getting this 100% right 10:54:05 <TrueBrain> but what we can deduce, I have no problem adding .. but it has to be automated 🙂 10:54:06 <Brickblock1> that loco wasn't built for use in Sweden originally so marking Sweden is in ways weird 10:54:36 <TrueBrain> I mainly do this as many of these NewGRFs have no active authors anymore 10:54:46 <FLHerne> Brickblock1: Nordic generally then :p 10:55:01 <DorpsGek> [OpenTTD/BaNaNaS] TrueBrain updated pull request #130: Feature: convert mentions of regions in tags to regions https://github.com/OpenTTD/BaNaNaS/pull/130 10:55:01 <FLHerne> oh, I didn't realise they ran in Hungary 10:55:02 <Brickblock1> it was also used in Hungary 10:55:05 <FLHerne> ^ 10:55:55 <Brickblock1> it was also used in Belgium and Luxembourg 10:56:05 <TrueBrain> I think this will have to do .. seems like it is a fine balance between effort and result 10:56:19 <TrueBrain> as if we go any deeper, we get discussions like you guys just had 😛 10:56:57 <FLHerne> TrueBrain: I think lowering resolution beyond country codes would make sense 10:57:08 <DorpsGek> [OpenTTD/BaNaNaS] TrueBrain updated pull request #130: Feature: convert mentions of regions in tags to regions https://github.com/OpenTTD/BaNaNaS/pull/130 10:57:28 <TrueBrain> FLHerne: I honestly don't know what you try to say with that sentence, sorry 😦 10:57:39 <FLHerne> if there was a 'Europe' tag that (implicitly?) included all the European countries, and could also be put on "stuff that's clearly from /somewhere/ in Europe" 10:57:54 <TrueBrain> did you check any of the work I did? 😄 10:58:33 <FLHerne> not yet, I only read > [10:41] <TrueBrain> Rubidium: no, I said 3166-2 was the lowest resolution < 20min ago 10:58:51 <TrueBrain> yeah, and later I correct lowest -> highest, as I always confuse resolution direction 🙂 10:59:38 <TrueBrain> but clicking around gives answer to your suggestion; take for example https://sn2b8ccs-5000.euw.devtunnels.ms/package/newgrf/797a0201 🙂 10:59:45 <TrueBrain> euh, that URL doesn't work for you .. 10:59:59 <TrueBrain> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/797a0201 11:00:01 <TrueBrain> 😄 11:00:36 <FLHerne> OK, that looks neat 11:01:05 <TrueBrain> tnx all for the bunch of suggestions/corrections; we can always fix things later, but for now this seems to work just fine 🙂 11:05:52 <TrueBrain> in-game search is also rather content with this information, good 🙂 11:07:01 <DorpsGek> [OpenTTD/bananas-server] TrueBrain updated pull request #303: Feature: add support for regions, and send them as tags to the clients https://github.com/OpenTTD/bananas-server/pull/303 11:07:04 <DorpsGek> [OpenTTD/bananas-api] TrueBrain updated pull request #335: Feature: tag what region a NewGRF / Heightmap / Scenario is about https://github.com/OpenTTD/bananas-api/pull/335 11:17:48 <DorpsGek> [OpenTTD/OpenTTD] anatolyeltsov opened pull request #10541: Feature: Industry production graph https://github.com/OpenTTD/OpenTTD/pull/10541 11:22:58 <Xarick> the original NoCAB performs much better than NoNoCAB 11:23:03 <Xarick> 😦 11:26:42 *** sla_ro|master has quit IRC 11:28:41 <Xarick> let's put them face to face 11:28:50 <Xarick> with infinite money, let's see who does better 11:38:37 <TrueBrain> lol, some sets are funny .. a tag "durango" for a townname set of colorado 11:38:45 <TrueBrain> but durango is also a region in Mexico 11:40:10 <DorpsGek> [OpenTTD/BaNaNaS] TrueBrain updated pull request #130: Feature: convert mentions of regions in tags to regions https://github.com/OpenTTD/BaNaNaS/pull/130 11:52:47 <Xarick> NoCAB crashes to excessive CPU usage 😦 12:02:57 <DorpsGek> [OpenTTD/OpenTTD] Alberth289346 opened issue #10542: [Bug]: BusyBee crashes on a new map since 991a797e098e22b8b3a0adaead757d691d80fca4 https://github.com/OpenTTD/OpenTTD/issues/10542 12:03:41 <gebik> TrueBrain: its town in colorado too as I looked it up 12:06:04 *** TROILUS has quit IRC 12:06:05 *** TROILUS1 has joined #openttd 12:06:09 *** TROILUS1 is now known as TROILUS 12:10:14 <TrueBrain> The world is werid 12:10:20 <TrueBrain> Weird eveb 12:10:30 <TrueBrain> Even .. mobile phones are the worst 12:16:04 <Xarick> omg why did i install visual studio 17.5.1 12:16:10 <Xarick> i can't build yet again 12:24:43 *** HerzogDeXtEr has joined #openttd 12:25:36 <Xarick> which version of visual studio do you have installed? 12:26:30 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on issue #10542: [Bug]: BusyBee crashes on a new map since 991a797e098e22b8b3a0adaead757d691d80fca4 https://github.com/OpenTTD/OpenTTD/issues/10542 12:28:22 <DorpsGek> [OpenTTD/OpenTTD] Kuhnovic opened pull request #10543: Feature: Region-based pathfinder for ships https://github.com/OpenTTD/OpenTTD/pull/10543 12:29:17 <glx[d]> Xarick: I use 17.5.1 12:29:36 <Xarick> i wonder if i can revert to 17.4.5 12:30:38 <klote[d]> Hi, i managed to get opettd server running but its using version 12.1 12:30:48 <klote[d]> how do i update it to the newest version? 12:30:58 <klote[d]> using linux 12:39:57 <TallTyler> TrueBrain: Hmm that might be my NewGRF. Durango is the location of one of America’s best-known tourist railroads. No idea why I added that as a tag though. 12:43:52 *** poglet has joined #openttd 12:43:52 <poglet> klote[d]: You need to manually download and extract the tar fine to a directory. For example: wget https://cdn.openttd.org/openttd-releases/13.0/openttd-13.0-linux-generic-amd64.tar.xz 12:43:52 <poglet> Then extract it using: tar -xf openttd-13.0-linux-generic-amd64.tar.xz 12:44:53 <poglet> Some older instructions here you just need you update the file url: https://www.vultr.com/docs/how-to-install-an-openttd-server-on-ubuntu-20-04/ 12:45:10 <klote[d]> ok 12:46:29 <klote[d]> lol shit 12:46:38 <klote[d]> downloaded it to the wrong folder i think 12:46:50 <TrueBrain> TallTyler: It is indeed yours 😉 12:51:00 <DorpsGek> [OpenTTD/OpenTTD] J0anJosep commented on pull request #9577: Feature: Multi-tile depots https://github.com/OpenTTD/OpenTTD/pull/9577#issuecomment-1455082979 12:54:31 <DorpsGek> [OpenTTD/OpenTTD] Kuhnovic updated pull request #10543: Feature: Region-based pathfinder for ships https://github.com/OpenTTD/OpenTTD/pull/10543 12:55:34 <klote[d]> poglet: Hi thanks, this is kind of confusing. 12:55:51 <klote[d]> am i supposed to just swap the openttd program? 12:56:21 *** Kuhnovic has joined #openttd 12:56:21 <Kuhnovic> KuhnovicviaGitHub: Can one of the lead devs slap a Preview label on my PR? This really needs to be play tested 😉 12:56:48 <klote[d]> because i dont want to lose config info 12:58:04 <klote[d]> i used apt install openttd and its running fine 12:58:10 <klote[d]> i just want to upgrade 12:58:21 <klote[d]> to 13 13:00:12 <poglet> Yeah sorry when i checked about 2 weeks ago apt didn't have the latest version so i had to go though these steps to get it working. Maybe someone here knows how long it takes for apt to get updated, not sure what the process is for that. 13:00:39 <klote[d]> ah 13:02:45 <FLHerne> poglet / klote[d]: Ubuntu doesn't update the major version of its OpenTTD packages except with new Ubuntu releases 13:02:55 <FLHerne> they only backport bugfixes 13:03:33 <FLHerne> (which is silly in my opinion but typical policy for non-rolling-release distros) 13:03:34 <klote[d]> tOk 13:03:45 <klote[d]> So i tried to install the depencys 13:04:06 <klote[d]> but they are also not all availible :x 13:05:15 <klote[d]> libfluidsynth2 13:05:45 <FLHerne> This guide you're trying to follow is outdated and a bad idea generally 13:05:57 <klote[d]> T-T 13:06:04 <klote[d]> then what am i supposed to do... 13:06:28 <klote[d]> how do i upgrade the 12.1 to 13... 13:06:40 <FLHerne> download 'Linux Generic Binaries' from https://www.openttd.org/downloads/openttd-releases/latest 13:06:46 <FLHerne> unpack it anywhere you like 13:06:55 <klote[d]> thats what i did 13:06:58 <FLHerne> execute the `openttd` binary in the resulting folder 13:08:01 <FLHerne> it can coexist with the distro package, or you could uninstall that to avoid confusion 13:08:51 <klote[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1081926299808976926/image.png 13:09:01 <klote[d]> what am i doing wrong here? 13:09:03 <FLHerne> the config is normally in ~/.openttd or ~/.local/share/openttd and won't be lost if you uninstall the distro package https://github.com/OpenTTD/OpenTTD/blob/master/docs/directory_structure.md 13:09:40 <FLHerne> klote[d]: ./openttd 13:10:59 <FLHerne> ^ enter ./openttd (explicitly "the one in this directory") rather than relying on $PATH 13:11:08 <klote[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1081926875556876298/image.png 13:11:17 <FLHerne> it might be interesting to see what `which openttd` returns if you run that 13:11:26 <FLHerne> hrm 13:11:32 <FLHerne> now that isn't supposed to happen 13:11:47 <klote[d]> is probably because i have missing depencys? 13:12:19 <klote[d]> or does it need command -D behind it? 13:12:32 <klote[d]> to run server terminal? 13:12:33 <FLHerne> the generic binary is supposed to be built in a way that minimises the need for such things 13:12:46 <FLHerne> please upload the crash.log it printed the path for 13:13:03 <Xarick> reverted VS to 17.4.5 13:13:31 <klote[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1081927475707260960/image.png 13:13:34 <FLHerne> you could try with -D, but I expect it will still crash 13:13:45 <klote[d]> it gave specific error now 13:15:59 <FLHerne> hm 13:17:47 <FLHerne> download https://cdn.openttd.org/opengfx-releases/7.1/opengfx-7.1-all.zip and put it in ~/.openttd/baseset/ 13:17:49 <FLHerne> probably 13:18:28 <klote[d]> extract it first? 13:19:56 <FLHerne> yes, right 13:20:08 <FLHerne> (the zip, but don't unpack the tar inside that) 13:22:41 <klote[d]> nice it worked 13:22:45 <klote[d]> its running 13:23:31 <klote[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1081929989689192519/image.png 13:25:08 <klote[d]> its not responding to any commands. 13:25:47 <klote[d]> https://cdn.discordapp.com/attachments/1008473233844097104/1081930562429784114/image.png 13:28:41 <klote[d]> none of the rcon commands are working 13:40:09 <TrueBrain> lol, one minor issue with 3166 .. something like Aruba is part of NL, so it is part of Europe, it says 13:40:15 <TrueBrain> that is not exactly true 😛 13:42:16 <andythenorth> administrative regions vs. geographical regions 😛 13:42:22 <andythenorth> welcome to my day job 🙂 13:42:47 <TrueBrain> always makes me giggle how bad those datasets actually are 🙂 13:47:50 <klote[d]> ok so ingame rcon commands dont work 😒 13:51:38 <Rubidium> TrueBrain: what's wrong with the dataset? Does the data set declare Aruba as part of Europe, or did you assume transitivity? 13:52:17 <TrueBrain> that is not what I said; I never claimed anything wrong with the dataset. I just said there was an issue 🙂 13:52:42 <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #10531: Codechange: Cache ScriptConfig for the slot in constructor https://github.com/OpenTTD/OpenTTD/pull/10531#pullrequestreview-1325220550 13:53:44 <TrueBrain> and andy summarized the issue very nicely 🙂 13:55:06 <TrueBrain> some heightmaps tagged 20 regions .. lol 13:55:14 <TrueBrain> I will be using the region that is above that instead 😛 13:55:19 <TrueBrain> it won't fit in network packets otherwise 😄 13:58:40 <andythenorth> so that sea-regions-pathfinder patch 😛 13:58:52 <andythenorth> a tile will know which region it is in? 14:01:14 <DorpsGek> [OpenTTD/OpenTTD] Alberth289346 commented on issue #10542: [Bug]: BusyBee crashes on a new map since 991a797e098e22b8b3a0adaead757d691d80fca4 https://github.com/OpenTTD/OpenTTD/issues/10542 14:01:39 <DorpsGek> [OpenTTD/BaNaNaS] TrueBrain updated pull request #130: Feature: convert mentions of regions in tags to regions https://github.com/OpenTTD/BaNaNaS/pull/130 14:02:27 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1081939788757745674/image.png 14:02:27 <andythenorth> kinds of want to stop this happening on map gen 😛 14:02:28 <TrueBrain> wauw, Albert still plays OpenTTD? 😄 14:02:49 <andythenorth> come back Alberth! 14:03:14 <DorpsGek> [OpenTTD/OpenTTD] andythenorth commented on issue #10542: [Bug]: BusyBee crashes on a new map since 991a797e098e22b8b3a0adaead757d691d80fca4 https://github.com/OpenTTD/OpenTTD/issues/10542 14:04:07 <andythenorth> hmm I already adjust some industry placement with GS after map gen 14:04:16 <andythenorth> maybe I can do more of that 14:04:47 <andythenorth> I'm not sure it's fair to the channel though, I will have a lot of stupid questions / rants 14:04:59 <TrueBrain> Similar question as earlier, but this time for heightmaps and scenarios .. 14:04:59 <TrueBrain> https://7c60-80-61-152-235.eu.ngrok.io/package/scenario?region=%28none%29 14:04:59 <TrueBrain> https://7c60-80-61-152-235.eu.ngrok.io/package/heightmap?region=%28none%29 14:04:59 <TrueBrain> Did I miss anything obvious? 14:05:31 <andythenorth> what was the thing people asked? We ping some content authors when we do this stuff? 14:05:37 <TallTyler> I can make you a Stage channel, "Andy complains about GS" 😛 14:05:41 <andythenorth> not sure how to ping, who is mod? 14:05:58 <andythenorth> "who is in our consultee database / contact list?" 14:06:11 <TrueBrain> I don't really need anyone specifically to look over this, just humans 😛 14:06:13 <andythenorth> TallTyler: I can just complain to chatGPT now 14:06:31 <TallTyler> I think the consensus was, make a thread in Discord channel #development-forum and ping the `@NewGRF Developer` role (which OpenTTD Developers now have permission to do) 14:06:58 <andythenorth> thanks...well next time we can do that 🙂 14:07:01 <petern> So much chat 14:07:36 <TallTyler> Firefox logo heightmap? 14:07:39 <petern> I think my legs are done for 14:08:22 <andythenorth> how many miles? 14:08:48 <TallTyler> Where is the region "Lakes"? 14:08:56 <TrueBrain> lol, let me put it on the ignore list 😄 14:09:00 <petern> 45 MTB miles 14:09:18 <andythenorth> quite a lot 14:09:37 <TrueBrain> TallTyler: There are regions called "West", and "Rivers", which tend to match on tags in the wrong way 😛 14:09:47 <TrueBrain> I already added many on the ignore list .. Lakes is a new one 😄 14:10:38 <TrueBrain> seems that was the last one 😛 14:10:44 <TallTyler> A lot of scenarios in None are real places that didn't get categorized 14:11:07 <TrueBrain> example of one? 14:11:29 <andythenorth> are there also regions in a fictional world? 14:11:30 <TrueBrain> "a lot" sadly isn't matching my filters 😄 14:11:42 <andythenorth> do we need a separate namespace for 'Italo Calvino type regions'? 14:11:57 <andythenorth> https://en.wikipedia.org/wiki/Invisible_Cities 14:12:03 <TallTyler> La Pampa 512x512 is Argentina according to Google 14:12:21 <TallTyler> Same for Santa Cruz 512x512 14:12:38 <TallTyler> Fallout OpenTTD is the United States 😛 14:12:52 <andythenorth> not _the_ United States though 14:12:59 <andythenorth> _a_ United States 14:13:05 <TallTyler> Tierra del Fuego is also Argentina 14:13:32 <andythenorth> first one to link Borges wins a prize 😛 14:13:34 <TrueBrain> those scenarios had no tags at all 14:13:48 <TrueBrain> hmm .. would require some more fuzzy matching on the name .. 14:13:53 <TallTyler> And then RT3 Go West! is the northeast US: New York and most of the New England states 14:14:10 <TallTyler> I made several versions of that though, and the FIRS one isn't in "None" 14:14:18 <TallTyler> Acadia 1750 is Nova Scotia, Canada 14:14:21 <petern> Victor Borges? 14:14:47 <TrueBrain> TallTyler: slow down 😉 We only set the region for content that already had an indicator for region 14:14:56 <TallTyler> Malaysia is presumably the country of the same name 😛 14:15:05 <TrueBrain> nothing for RT3 Go West had any indications in its tags nor name 14:15:08 <TrueBrain> so they are not tagged 14:15:13 <TallTyler> There are also two Fiji scenarios 14:15:34 <TallTyler> Not complaining, just making a list for you 🙂 14:15:35 <TrueBrain> anyway, it is not useful to list them like this; but yes, all content with a name matching a region and no tags, aren't matched atm .. will see what I can do there 14:15:38 <TrueBrain> is tricky 🙂 14:15:56 <TallTyler> I have to run an errand now but will be back in about 30 minutes 14:16:16 <TrueBrain> there are just too many darn regions to accidentially match 😄 14:19:10 <klote[d]> Hmm 14:19:14 <klote[d]> i need some help 14:19:16 <andythenorth> petern: the other one 😛 14:19:47 <andythenorth> TrueBrain: TrueBrain seems like it might be a human curation shaped problem 😛 14:19:53 <andythenorth> can the bot do it? 14:19:58 <klote[d]> Why is it when i download and run openttd in linux the list_dirs is pointing towards Windows path structure... 14:19:58 <Xarick> wow, thinking of pathfinding 14:20:39 <TrueBrain> andythenorth: huh? That did not compute 14:21:06 <andythenorth> I'm not sure it computed when I wrote it 14:21:13 <andythenorth> it made sense 2 mins ago 14:21:16 <andythenorth> now it does not 14:21:17 <klote[d]> derppp 14:21:27 <klote[d]> need to use rcon command first 14:22:25 <TrueBrain> andythenorth: lol; anyway, this is a bot doing it, which is the problem 14:22:57 <Xarick> it's been 3 years 14:23:18 <Xarick> I have a PR about finding closest ship depot <https://github.com/OpenTTD/OpenTTD/issues/8022> 14:23:33 <Xarick> shall i submit it? 14:23:34 <petern> Christ, I just flung my phone down the stairs 😦 14:23:52 <TrueBrain> Noooooooooooo 14:24:00 <Xarick> ok :/ 14:24:08 <TrueBrain> did it survive? 14:25:33 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on issue #10542: [Bug]: BusyBee crashes on a new map since 991a797e098e22b8b3a0adaead757d691d80fca4 https://github.com/OpenTTD/OpenTTD/issues/10542 14:26:46 <TrueBrain> There are 3 `Mayotte` .. eeuuhh .. which to pick 😄 Lol 14:27:36 <pickpacket> is it possible to reduce the speed of the game below 'normal'? 14:31:19 <TrueBrain> wow, also many Santa Cruz .. you near something this way about the world 😛 14:31:21 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #10533: Codechange: Use SQInteger for generic numbers in script API https://github.com/OpenTTD/OpenTTD/pull/10533#pullrequestreview-1325226428 14:36:55 <TrueBrain> TallTyler: okay, fixed a bunch of entries based on their name; I hope I didn't introduce false positives 😛 14:39:58 <andythenorth> shall I try and teach GS about newgrf industry layouts? 😛 14:40:07 <andythenorth> and how to terraform land for them? 14:40:41 <frosch> TrueBrain: do you use "world" as key word to classify stuff? 14:40:44 <DorpsGek> [OpenTTD/bananas-server] TrueBrain updated pull request #303: Feature: add support for regions, and send them as tags to the clients https://github.com/OpenTTD/bananas-server/pull/303 14:40:55 <frosch> i am looking at "africa", and 3 out of 6 items look wrong 14:41:17 <TrueBrain> frosch: "world" is on a blacklist, and does not participate in the region selection 14:41:31 <TrueBrain> frosch: people keep giving me these vague remarks .. I need a bit more specific information 😄 14:41:46 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf?region=Africa 14:41:51 <TrueBrain> I don't even know what content-type you were looking at 😛 14:42:14 <TrueBrain> did NML CAT made it into the list again? It is such an annoying one .. 14:42:29 <frosch> no idea how "world flags" "world train set" and "nml cat" ended up there 14:42:29 <TrueBrain> owh, this one is spelled differently, great 14:42:30 <TrueBrain> lol 14:42:36 <andythenorth> hmm how can I find a GS block of connected sea tiles? 14:42:39 <andythenorth> of minimum size 😛 14:42:52 <TrueBrain> frosch: I do; they had tags like that 14:43:03 <frosch> ah, wolrd train set is fine, it lists all the continents in the description 14:43:14 <TrueBrain> basically, they tagged themselves as "Africa", "America", "Asia", "Australia", "Europa" 14:44:22 <TrueBrain> and I will fix the NML CAT .. it matches on "Lakes", which happens to be a region in Africa 😛 14:44:32 <frosch> lol 14:44:39 <TrueBrain> frosch: similar for World Flags btw, so I guess it is fine 14:45:31 *** WormnestAndroid has quit IRC 14:46:32 <TrueBrain> pretty sure they meant to target americas, but they wrote america .. 😛 14:46:46 <frosch> how odd... i assumed "kiwi..." to be new-zealand stuff, but it is korean... 14:46:56 <TrueBrain> yeah, I had to double check that one too 😄 14:47:03 <TrueBrain> also if it was south or north .. 14:48:50 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/454e3701 <- that one is unfortunate 14:49:31 <TrueBrain> had the tag `Australian` 14:49:48 <TrueBrain> and DPRK 14:49:49 <TrueBrain> seems fine? 14:49:53 <TrueBrain> what is unfortunate about it? 14:50:36 <TrueBrain> next import it will also add JP; original tag was "japan set" .. I did not have that in my list yet 😄 14:50:42 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/50430307 <- balkan is in asia? 14:51:18 <TrueBrain> there is a region called `Balkan` in `Turkmenistan` 14:51:51 <TrueBrain> will add it to the ignore list 14:52:05 <TrueBrain> frosch: still no clue what you meant with this 🙂 14:52:26 <andythenorth> it's not really regional, it's just indicating other grfs it is designed to work with? 14:52:35 * andythenorth guesing 14:52:49 <frosch> TrueBrain: i see how it got classified as australian, but i think it was not meant to be 14:53:00 <TrueBrain> the author explicitly did 14:53:05 <TrueBrain> there is Auz in the list 14:54:17 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/454e0e01 <- same with that, it's mostly "generic" but for some reason lists a "random" selection of other grfs 14:54:25 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick commented on pull request #10533: Codechange: Use SQInteger for generic numbers in script API https://github.com/OpenTTD/OpenTTD/pull/10533#pullrequestreview-1325230336 14:55:11 <TrueBrain> frosch: that is up to the author ofc .. not much we can do about that, I guess 🙂 14:56:33 <Xarick> omg the triggle A Hog bankrupted 14:56:41 <Xarick> infrastructure maintenance 14:56:55 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/54530102 <- haha, belgium annexed luxembourg? 14:57:27 <klote[d]> Hi, my server is not showing up in the server list... i can only see it lan 14:57:51 <TrueBrain> frosch: many of these regions exist in both countries 🙂 But I will make a priority fall here, and assume Luxembourg is meant 😉 14:57:56 <klote[d]> Is this a firewall thing? 14:58:00 <klote[d]> port forward? 15:00:07 <glx[d]> no it's the `server_game_type` setting 15:00:33 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/454e0202 <- how did that one get "berlin"? 15:00:55 <TrueBrain> frosch: explicit tag by the author 15:01:12 <klote[d]> glx[d]: right 15:01:14 <klote[d]> thanks 15:01:55 <glx[d]> port forwarding is a nice thing to do, but not required anymore 15:02:44 <TrueBrain> frosch: later on you find a similar one with also "uk" on the list 15:02:46 <frosch> oh, the tags are still part of global.yaml 15:03:00 <frosch> i only checked the diff that removed them from version.yaml 15:03:18 <TrueBrain> I kinda ignored version.yaml tags .. purely based it on the ones in global 😛 15:03:26 <TrueBrain> I should make a run with the old tags in versions there too 15:03:53 <frosch> yeah, way too many people write stuff into version-specific entries 15:04:04 <frosch> the gui does not hide them well enough 15:04:15 <TrueBrain> I think I will remove `name` for sure 15:04:21 <TrueBrain> `description` I can see a use-case for 15:04:29 <TrueBrain> but honestly, we just need to redo the UI to give a better experience 15:04:34 <TrueBrain> andy keeps saying he is going to do it 😛 15:04:51 <frosch> currently we have the package text as placeholder text 15:05:07 <frosch> maybe we need a checkbox instead, which defaults to "use package info" 15:05:27 <TrueBrain> something like that, and that unchecking shows the textbox 15:05:32 <TrueBrain> but .. in the current UI this is just .. ugh 15:06:11 <Xarick> I'm scared of posting a PR that was rejected in the past 😦 15:06:16 <Xarick> should I try? 15:06:40 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/434b4fb1 <- it's so generic, it's just californian 🙂 15:06:42 <TrueBrain> what did Einstein say ... trying the same thing twice expecting a different outcome is the definition of stupidity? 15:07:07 <Xarick> <https://github.com/OpenTTD/OpenTTD/issues/8022> 15:07:10 <TrueBrain> frosch: ghehe 😄 Yeah, using the tags as regions isn't perfect 🙂 15:07:22 <Xarick> it was rejected in 2019 15:08:06 <Xarick> OPF was still a thing bakc then 15:08:33 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/576d5201 <- lol, the tags reveal the true nature of the set, the name was lying 🙂 15:09:17 <frosch> creating a random town name set, by using RL names from one specific region, and adding random prefixes and suffixes 🙂 15:10:53 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/54540801 <- wrong "charlotte", i guess? 15:11:22 <TrueBrain> there is only one region by that name 😄 To the ignore list! 15:11:37 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf?region=Central+America <- i like how all entries in the "central america" region are named "north america" 🙂 15:11:49 <klote[d]> what does reload_cfg setting do 15:12:12 <klote[d]> wiki doesnt show explenation 15:12:17 <TrueBrain> frosch: yeah .. they all also do something in Mexico, they claim 😛 15:12:43 <TrueBrain> was funny at first, as it preferred Mexico City over Mexico .. 15:14:18 <frosch> there are so many sets on bananas, which my geometry skills would never have put into some region. now i can find them 🙂 15:14:29 <frosch> *geography even 15:14:56 <frosch> my geometry skills are okay, maybe that's my issue with geography 15:15:23 <TrueBrain> 😄 15:15:28 <TrueBrain> I learnt a lot of new things today 🙂 15:16:32 <michi_cc[d]> Hmm, looks like there's some release constipation again. 15:16:40 <frosch> i really don't understand the geography spam in some tags 15:17:01 <TrueBrain> I hope this new system also reduces it in some places .. 15:17:06 <TrueBrain> but I sided with the authors for now 15:17:25 <TrueBrain> if you disagree with any, do let me know 15:17:43 <andythenorth> michi_cc[d]: ? 🙂 15:18:10 <michi_cc[d]> I thought I read some talk of 13.1 in here, but nothing seems to have happened so far. 15:19:14 <frosch> TrueBrain: can you detect which sets list regions from 3 or more continents? 15:19:23 <TrueBrain> sure, with some effort 15:19:33 <TrueBrain> what you want to do with that list? 15:19:55 <frosch> maybe they are just "generic" and should have no region 15:22:52 <Xarick> inb4 closed 15:23:37 <TrueBrain> frosch: 4 hits in newgrfs .. now to find their grfid 15:23:41 <TrueBrain> but not many, at least 🙂 15:23:47 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/504381b1 <- someone put a break in the tags between "french" and "canada" 🙂 15:24:11 <TrueBrain> these kind of errors happens a lot ... 😛 15:24:20 <TrueBrain> but I can fix this one 🙂 15:25:20 <frosch> "future reality inspired maglevs" <- some names ... 15:25:44 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick opened pull request #10544: Fix #5713: Use pathfinder to find closest ship depot https://github.com/OpenTTD/OpenTTD/pull/10544 15:27:16 <TrueBrain> 797a0201, 57545453, 4a448807, 5f5f0200 have 3 or more continents frosch 15:27:37 <Xarick> No common parent found for this merge commit (max-depth of 256 reached) what is this 15:27:55 <TrueBrain> 2 of them you already knew about 15:28:11 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #10533: Codechange: Use SQInteger for generic numbers in script API https://github.com/OpenTTD/OpenTTD/pull/10533#pullrequestreview-1325235604 15:28:46 <DorpsGek> [OpenTTD/bananas-frontend-web] TrueBrain updated pull request #165: Feature: add support for regions (add, edit, filter) https://github.com/OpenTTD/bananas-frontend-web/pull/165 15:29:27 <DorpsGek> [OpenTTD/bananas-frontend-web] TrueBrain commented on pull request #165: Feature: add support for regions (add, edit, filter) https://github.com/OpenTTD/bananas-frontend-web/pull/165#pullrequestreview-1325235775 15:30:50 <DorpsGek> [OpenTTD/bananas-api] TrueBrain updated pull request #335: Feature: tag what region a NewGRF / Heightmap / Scenario is about https://github.com/OpenTTD/bananas-api/pull/335 15:30:52 <frosch> i think JF Ratt Roads (5f5f0200) is the only valid one of them 15:30:58 <frosch> the other 3 are just region spam 15:31:07 <TrueBrain> okay, now I just need to fix regression tests .. 15:31:13 <glx[d]> trying to improve ScriptText validation error messages, I had a test idea that didn't end well <https://gist.github.com/glx22/14d3e99050978cde1ddcbab23138b7e5> 15:31:21 <glx[d]> looks like more validation is needed 15:31:40 <TrueBrain> glx[d]: nice list 😄 15:32:00 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #10544: Fix #5713: Use pathfinder to find closest ship depot https://github.com/OpenTTD/OpenTTD/pull/10544 15:32:57 *** D-HUND has joined #openttd 15:33:12 <Xarick> needed a rebase 15:33:35 <glx[d]> updated the gist with the test code 15:33:42 <glx[d]> (silly test, but hey) 15:36:24 <DorpsGek> [OpenTTD/bananas-server] TrueBrain updated pull request #303: Feature: add support for regions, and send them as tags to the clients https://github.com/OpenTTD/bananas-server/pull/303 15:38:26 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/79210502 <- same thing with "brazilian" "portuguese" 15:38:53 <TrueBrain> explicit requested by author in this case 15:38:53 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/47430101 <- wrong "alba" 15:38:58 <TrueBrain> should we take action on it? 15:39:32 <TrueBrain> will fix alba 15:40:08 <frosch> if it is not too much work, we should fix the french canadian and brazilian portoguese stuff 15:40:11 <TrueBrain> okay, BR / PT is this one set, I can fix that 15:40:17 <TrueBrain> already did the canadian one 15:40:23 <andythenorth> hmm which will be less work? 15:40:36 <andythenorth> teaching the game whether an industry is buildable at a specific location? 15:40:46 <andythenorth> or watching me try to figure it out in GS? 15:41:17 <frosch> it's ridiculous how many regions only exist in the dropdown, because world train set tags them 15:41:40 <TrueBrain> and you can argue two sides of the coin there .. either that is a good thing, or a bad thing 😄 15:42:30 <gebik> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/50438161 15:42:55 <gebik> Found this 15:43:14 <TrueBrain> lol .. prussia -> russia 😄 15:43:20 <gebik> lol 15:45:04 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/newgrf/4b535831 <- new england is not england, i guess 🙂 15:46:34 <TrueBrain> "New England" is nothing 😛 It is a collection of things .. hmmmmm 15:48:30 <frosch> hmm, what info do you use for classification? only the old tags, or also the description? 15:48:37 <TrueBrain> tags and name 15:48:45 <TrueBrain> I don't want to touch descriptions, as they contain a lot of bloat 😛 15:49:04 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/heightmap/00000785 <- ok ,so "crete" is not known 15:49:17 <TrueBrain> nope 15:49:54 <frosch> do you feel like adding more tags? or rather "no thanks"? 🙂 15:50:26 <TrueBrain> so far, I only did things automated, with a very big script 15:50:45 <petern> 'nsfw' for george's LV4 (are they even on there) 15:50:45 <TrueBrain> what you suggest is close to curation, I guess; not a bad thing, but I kept away from that currently 🙂 15:51:16 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/heightmap/00000109 <- wrong georgia 15:51:16 <TrueBrain> frosch: called "kriti", so it doesn't match 15:53:11 <Eddi|zuHause> you can probably safely exclude tags that are used only once 15:53:26 <frosch> lol, no 15:53:46 <frosch> plenty of very specific townname sets, which are classified correctly 15:54:15 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/heightmap/00001562 <- i guess this is the definition of region spam 15:54:45 <TrueBrain> it is 15:54:51 <Eddi|zuHause> whenever someone says "wrong georgia" i need to think of that tiny island near antarctica :p 15:56:23 <andythenorth> uuf can I be bothered to rewrite the industry placement entirely in GS? 😛 15:57:19 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/heightmap/00000132 <- wrong "midlands" :p 15:58:13 <frosch> how can a region have such a generic name as "midlands" though 15:58:25 <TrueBrain> similar for West / East / etc 15:58:26 <TrueBrain> it is funny 15:58:55 <Eddi|zuHause> how many cities can be called "new city"? or "red castle"? :p 16:01:32 <TrueBrain> okay, locally fixed all the links given so far; keep them coming 🙂 16:02:36 <andythenorth> so currently the GS doesn't init until after map gen AFAIK 16:02:48 <andythenorth> and the grf has no idea whether a GS is present 16:03:13 <andythenorth> it seems pointless to place a lot of industries at map gen, then have GS delete them all 16:06:49 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/heightmap/00000105 <- wrong guinea 16:08:22 <TrueBrain> restarting the service to load in the new database 16:08:44 <TrueBrain> done 16:09:38 <DorpsGek> [OpenTTD/OpenTTD] glx22 opened pull request #10545: Improve ScriptText validation error messages and prevent circular references https://github.com/OpenTTD/OpenTTD/pull/10545 16:10:16 <Xarick> hmm i could provide the real pathfinder cost as distance 16:10:33 <Xarick> i guess it's what you wanted 16:10:50 <Xarick> it's just that the distance isn't effectively used 16:10:59 <Xarick> it's just passed around for nothing 16:11:17 <frosch> shall i report all the world map scenarios, which spam all regions? 16:11:27 <TrueBrain> well, I wasn't planning on fixing them 16:11:30 <TrueBrain> so that sounds weird 😛 16:11:44 <frosch> good, then i can skip reporting them 🙂 16:12:07 <TrueBrain> frosch: hmm .. this one is a bit odd .. Papua New Guinea is part of Melanesia, it seems, but the description suggests it is part of Indonesia .. 16:12:18 <TrueBrain> some political thing? 16:12:30 <TrueBrain> owh, no, I am reading it wrong 16:12:37 <frosch> no, geography/political discrepancy 🙂 16:12:38 <TrueBrain> it contains both PNG and Indonesia 16:12:38 <TrueBrain> fine 16:12:50 <TrueBrain> okay, so the new info is correct 16:12:59 <frosch> one island, half indonesia, half new guinea 16:13:00 *** Wormnest has joined #openttd 16:13:03 <frosch> but the african guinea is wrong 16:13:13 *** WormnestAndroid has joined #openttd 16:13:48 <frosch> well, "was" wrong 16:14:13 <TrueBrain> yeah, fixed it in last reload 🙂 But was still wondering about the remaining two, but those are fine 🙂 16:14:22 <TrueBrain> as for the "world" maps, I really don't know what a good solution is .. 16:14:39 <TrueBrain> we can introduce "world" or "earth" (as we also have "mars") 16:14:50 <TrueBrain> but .. I am sure if we don't curate it, people will do what is done currently anyway 16:14:59 <TrueBrain> reason I limited the regions you can enter to 10, which I think is still a lot 😛 16:17:09 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/scenario/00001941 https://7c60-80-61-152-235.eu.ngrok.io/package/scenario/000001c5 <- everything is identical except "author" 16:17:15 <frosch> if in doubt, blame the author? 16:17:38 <TrueBrain> existing issue 16:18:08 <frosch> oh, probably same author, just ottd vs gh account 16:20:45 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/scenario/000011a9 <- wrong victoria 16:21:06 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/scenario/0000194d <- same 16:21:44 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/scenario/00000f90 <- lol 16:23:04 <andythenorth> hmm how do I prospect within a fixed region of the map? 16:23:30 <andythenorth> I guess I draw a rect of tiles or something and monte carlo them 16:23:56 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/scenario/00001942 <- wrong santa catarina 16:24:04 <andythenorth> how do I undo terraforming in GS? 16:24:18 <andythenorth> keep the tile height and contents? 16:24:27 <andythenorth> then rebuild it? 16:24:42 <andythenorth> I need to terraform tile for a specific industry layout 16:24:54 <andythenorth> I can only go tile-by-tile, and the terraforming might fail 16:25:00 <andythenorth> so I need to be able to revert 16:25:18 <andythenorth> GS can place houses and stuff I assume 16:25:33 <TrueBrain> frosch: I don't get what Victoria it should be 😛 16:26:01 <frosch> both refer to lake victoria in africa 16:26:13 <TrueBrain> ah, not known in 3166, fine, will mute it 😛 16:28:24 <TrueBrain> all 4 are fixed now 16:33:40 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/scenario/00001919 <- why is that listed in the dropdown for "jamaica" when it is not classified as such? 16:34:08 <TrueBrain> good question 16:34:10 *** sla_ro|master has joined #openttd 16:34:10 <TrueBrain> will investigate 😄 16:40:13 <frosch> https://7c60-80-61-152-235.eu.ngrok.io/package/scenario/00000e76 https://7c60-80-61-152-235.eu.ngrok.io/package/scenario/00000fa7 <- 2x wrong florida 16:43:15 <TrueBrain> you will find 2 more; fixed 16:43:24 <frosch> that's all, looks awesome 🙂 16:43:52 <TrueBrain> cheers, and tnx 16:44:02 <TrueBrain> also for checking at this level that you did 😄 16:44:38 <frosch> there are plenty of scenarios and heightmaps with no tags, which could have some, but author's fault 16:44:47 <TrueBrain> yeah .. so much we can do automated 16:47:12 <frosch> lol, did i just spend 2 hours with this? :p 16:47:22 <frosch> well, time to cook then 16:47:26 <TrueBrain> I spend two days with this ... 😛 16:48:06 <TrueBrain> ah, found why Merrimack Valley went wrong; the version has other tags than the package 16:48:14 <TrueBrain> https://7c60-80-61-152-235.eu.ngrok.io/package/scenario/00001919/2021-02-09T04:37:10+00:00 16:49:10 <TrueBrain> so .... I guess we really should remove "regions" from version too 16:49:18 <TrueBrain> but that would be another PR, to collapse name and region 16:53:20 <DorpsGek> [OpenTTD/OpenTTD] glx22 updated pull request #10545: Improve ScriptText validation error messages and prevent circular references https://github.com/OpenTTD/OpenTTD/pull/10545 16:55:05 <TrueBrain> okay, I can already do that .. there are no tags worth talking about per version; the globals are always a superset, basically 17:00:01 <DorpsGek> [OpenTTD/bananas-api] TrueBrain updated pull request #335: Feature: tag what region a NewGRF / Heightmap / Scenario is about https://github.com/OpenTTD/bananas-api/pull/335 17:02:42 <DorpsGek> [OpenTTD/BaNaNaS] TrueBrain updated pull request #130: Feature: convert mentions of regions in tags to regions https://github.com/OpenTTD/BaNaNaS/pull/130 17:02:58 <TrueBrain> need to rebase that commit on the latest .. but first, dinner 😄 17:05:00 *** WormnestAndroid has quit IRC 17:05:09 *** WormnestAndroid has joined #openttd 17:07:42 <frosch> i keep wondering about some form of "fictional" tag, which people can use for their comic-based or alternate-reality stuff 17:07:59 <frosch> nothing you could auto-detect, but may still be useful 17:09:17 <frosch> but "fictional" is so arbitrary... it's may be something a single person created by putting japan and ireland onto the same heightmap next to each other (yes, that exists), or some popular culture dc-universe stuff 17:16:47 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #10545: Improve ScriptText validation error messages and prevent circular references https://github.com/OpenTTD/OpenTTD/pull/10545#pullrequestreview-1325251004 17:23:32 *** tokai has joined #openttd 17:23:32 *** ChanServ sets mode: +v tokai 17:30:01 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1081992027115376762/image.png 17:30:02 <andythenorth> oof 17:30:22 *** tokai|noir has quit IRC 17:30:39 <andythenorth> I have no idea 😛 17:31:35 <andythenorth> lol 17:31:37 <andythenorth> `foreach (industry_spec in firs["industries"]) { 17:31:37 <andythenorth> local map_curator = industry_spec.economy_variations[firs.get_economy_id()].map_curator; 17:31:37 <andythenorth> firs.map_curators.industry_types.append(map_curator) 17:31:37 <andythenorth> } 17:31:37 <andythenorth> ` 17:31:49 <andythenorth> oh maybe it's the fingerprinting 17:32:30 <glx[d]> there's an op limit for loading 17:35:11 <TrueBrain> frosch: I like the idea 17:36:53 <TrueBrain> Just the name is tricky .. maybe something for a later PR 🙂 17:39:16 *** Flygon has quit IRC 17:43:15 <frosch> i stopped liking the idea :p it's so subjective 17:43:21 *** audigexJon has joined #openttd 17:43:21 <audigexJon> frosch: Possibly because of "Kingston"? 17:45:43 <TrueBrain> Owh, right, I was going to remove the ability to set region per version .. Will be global-only :p 17:46:10 <TrueBrain> Avoids the confusion as with this package 🙂 17:48:33 *** D-HUND has quit IRC 17:50:18 <DorpsGek> [OpenTTD/OpenTTD] glx22 updated pull request #10545: Improve ScriptText validation error messages and prevent circular references https://github.com/OpenTTD/OpenTTD/pull/10545 17:50:36 <Xarick> I don't understand yapf at times 17:51:26 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #10545: Improve ScriptText validation error messages and prevent circular references https://github.com/OpenTTD/OpenTTD/pull/10545#pullrequestreview-1325254839 17:55:28 <TrueBrain> why would it be that on one page the CSP does not trigger, and on another it does .. hmm 17:58:42 <TrueBrain> ah ... I set that header for only that one page 😄 Lol 17:58:59 <glx[d]> TallTyler: I marked #10492 so #10545 should be marked too 😉 18:00:02 <glx[d]> and I think circular reference issue happens in 13.0 too 18:00:03 <TallTyler> Huh, I didn't remove that label 18:00:19 <glx[d]> maybe we set labels at the same time 18:00:28 <TallTyler> I added the Script API label and we must have hit save at the same time 18:00:29 <TallTyler> Yes 18:01:10 <DorpsGek> [OpenTTD/bananas-frontend-web] TrueBrain updated pull request #165: Feature: add support for regions (add, edit, filter) https://github.com/OpenTTD/bananas-frontend-web/pull/165 18:01:13 <TallTyler> I'd never remove your labels intentionally 🙂 18:01:42 <glx[d]> unless I use the wrong one 😉 18:01:52 <TallTyler> I suppose 18:02:18 <TrueBrain> okay, region support is now ready for review 🙂 18:02:19 <TallTyler> I don't know if all the labels are useful, but I try to keep up with them on issues and PRs as one small way I can contribute 🙂 18:02:53 <TrueBrain> and if people could stop making BaNaNaS changes, I can update that PR 😛 (I am not serious; we will find a moment to make the change happen 🙂 ) 18:05:02 <DorpsGek> [OpenTTD/BaNaNaS] TrueBrain updated pull request #130: Feature: convert mentions of regions in tags to regions https://github.com/OpenTTD/BaNaNaS/pull/130 18:08:05 <TrueBrain> https://c01a-80-61-152-235.eu.ngrok.io/package/heightmap?resolution=high&terrainType=mountainous®ion=Americas 18:08:05 <TrueBrain> Hihi, that now works .. 😄 18:09:28 <Rubidium> glx[d]: regarding #10545, there are way more throws in that function to consider 18:10:16 <glx[d]> oh right 18:10:54 <DorpsGek> [OpenTTD/bananas-api] TrueBrain updated pull request #335: Feature: tag what region a NewGRF / Heightmap / Scenario is about https://github.com/OpenTTD/bananas-api/pull/335 18:11:03 <DorpsGek> [OpenTTD/bananas-frontend-web] TrueBrain updated pull request #165: Feature: add support for regions (add, edit, filter) https://github.com/OpenTTD/bananas-frontend-web/pull/165 18:11:12 <DorpsGek> [OpenTTD/bananas-server] TrueBrain updated pull request #303: Feature: add support for regions, and send them as tags to the clients https://github.com/OpenTTD/bananas-server/pull/303 18:12:12 <TrueBrain> too bad I cannot demo how the editor looks like, as it is somewhat weird but best I can do atm .. but logging in doesn't work, as ngrok only allows forwarding a single port 18:12:15 <TrueBrain> and I need 3 😦 18:17:11 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #10544: Fix #5713: Use pathfinder to find closest ship depot https://github.com/OpenTTD/OpenTTD/pull/10544 18:20:42 <Xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1082004778441248808/screenshot17.png 18:20:42 <Xarick> this is what autoreplace/autorenew does to my vehicles... 18:25:39 <andythenorth> FML 18:26:07 <andythenorth> python list `foo = ["cabbage", "dogs"]` 18:26:39 <andythenorth> chameleon rendering of that in template using `${foo}` is `['cabbage', 'dogs']` 18:27:17 <andythenorth> squirrel wants `["cabbage", "dogs"]` 18:27:58 <Xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1082006609552752821/image.png 18:27:58 <Xarick> what is this warning? should I be worried? 18:28:01 <DorpsGek> [OpenTTD/OpenTTD] glx22 updated pull request #10545: Improve ScriptText validation error messages and prevent circular references https://github.com/OpenTTD/OpenTTD/pull/10545 18:34:11 <glx[d]> of course only MSVC likes what I wrote 18:38:24 *** JGR has joined #openttd 18:38:24 <JGR> In general you'd be better off with values and std::move for most of these string parameters 18:41:29 <Xarick> I find the max_penalty value to be quite short for ships 18:42:36 <Xarick> the default of 2000 18:42:52 <Xarick> works fine for roads and rails 18:44:10 <DorpsGek> [OpenTTD/OpenTTD] DorpsGek pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/e60632bce655b4eb7be881e1f513ed1362e7e578 18:44:11 <DorpsGek> - Update: Translations from eints (by translators) 18:46:19 <Rubidium> glx[d]: isn't it way easier to just pass the vector as parameter? And potentially statically allocate it in GetEncodedText and clear it there for each call? 18:46:51 *** sla_ro|master has quit IRC 18:47:44 <glx[d]> that's a solution too 18:54:21 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #10544: Fix #5713: Use pathfinder to find closest ship depot https://github.com/OpenTTD/OpenTTD/pull/10544 18:56:33 *** gelignite has joined #openttd 19:05:25 <DorpsGek> [OpenTTD/OpenTTD] J0anJosep updated pull request #8480: Feature: Extended depots https://github.com/OpenTTD/OpenTTD/pull/8480 19:05:28 <DorpsGek> [OpenTTD/OpenTTD] J0anJosep updated pull request #9577: Feature: Multi-tile depots https://github.com/OpenTTD/OpenTTD/pull/9577 19:11:02 <DorpsGek> [OpenTTD/OpenTTD] J0anJosep updated pull request #9577: Feature: Multi-tile depots https://github.com/OpenTTD/OpenTTD/pull/9577 19:11:08 <DorpsGek> [OpenTTD/OpenTTD] J0anJosep updated pull request #8480: Feature: Extended depots https://github.com/OpenTTD/OpenTTD/pull/8480 19:13:24 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #10546: Fix 30ae072: when a road type is hidden, towns may not build them even when that flag is set https://github.com/OpenTTD/OpenTTD/pull/10546 19:15:08 <DorpsGek> [OpenTTD/bananas-api] frosch123 commented on pull request #335: Feature: tag what region a NewGRF / Heightmap / Scenario is about https://github.com/OpenTTD/bananas-api/pull/335#pullrequestreview-1325264569 19:20:30 <DorpsGek> [OpenTTD/bananas-frontend-web] frosch123 approved pull request #165: Feature: add support for regions (add, edit, filter) https://github.com/OpenTTD/bananas-frontend-web/pull/165#pullrequestreview-1325265154 19:26:44 *** lobstarooo__ has joined #openttd 19:28:00 *** lobstarooo has joined #openttd 19:34:47 *** lobstarooo__ has quit IRC 19:34:51 *** lobstarooo is now known as lobstarooo__ 19:37:28 <DorpsGek> [OpenTTD/bananas-api] TrueBrain commented on pull request #335: Feature: tag what region a NewGRF / Heightmap / Scenario is about https://github.com/OpenTTD/bananas-api/pull/335#pullrequestreview-1325267060 19:39:24 <DorpsGek> [OpenTTD/bananas-api] frosch123 approved pull request #335: Feature: tag what region a NewGRF / Heightmap / Scenario is about https://github.com/OpenTTD/bananas-api/pull/335#pullrequestreview-1325267294 19:39:36 <DorpsGek> [OpenTTD/bananas-server] frosch123 approved pull request #303: Feature: add support for regions, and send them as tags to the clients https://github.com/OpenTTD/bananas-server/pull/303#pullrequestreview-1325267327 19:39:56 <TrueBrain> so this is happening? 😄 19:40:02 <TrueBrain> tnx for the reviews frosch 🙂 19:40:35 *** lobstarooo__ has quit IRC 19:40:39 <DorpsGek> [OpenTTD/bananas-api] TrueBrain merged pull request #335: Feature: tag what region a NewGRF / Heightmap / Scenario is about https://github.com/OpenTTD/bananas-api/pull/335 19:40:45 <DorpsGek> [OpenTTD/bananas-frontend-web] TrueBrain merged pull request #165: Feature: add support for regions (add, edit, filter) https://github.com/OpenTTD/bananas-frontend-web/pull/165 19:40:50 <DorpsGek> [OpenTTD/bananas-server] TrueBrain merged pull request #303: Feature: add support for regions, and send them as tags to the clients https://github.com/OpenTTD/bananas-server/pull/303 19:40:51 *** lobstarooo__ has joined #openttd 19:41:34 <DorpsGek> [OpenTTD/OpenTTD] github-code-scanning[bot] commented on pull request #9577: Feature: Multi-tile depots https://github.com/OpenTTD/OpenTTD/pull/9577#pullrequestreview-1325267516 19:42:17 <TrueBrain> ugh, exactly when the CI is busy, ofc 😛 19:43:35 <TrueBrain> https://github.com/OpenTTD/BaNaNaS/pull/130 now passes CI and ready for review 🙂 19:43:54 <TrueBrain> 1669 files changed, lol 19:44:02 <TrueBrain> +2000, -10000 lines 19:44:14 <LordAro> :D 19:44:27 <TrueBrain> tags are bloat! 😛 19:48:22 <LordAro> though i wonder if tags should be kept temporarily, just in case, things need to be reverted 19:48:38 <LordAro> may be easier if the db doesn't need reverting as well 19:48:58 <TrueBrain> it is git 19:49:26 <TrueBrain> so I rather have we remove it in the assumption we no longer needed it, then keep it and completely forget about it 😄 19:49:41 <LordAro> i guess 19:50:03 <andythenorth> oof such GS templating with python 🙂 19:50:09 <andythenorth> keeps me quiet I guess 19:50:13 <andythenorth> writing actual working code 19:50:17 <DorpsGek> [OpenTTD/OpenTTD] github-code-scanning[bot] commented on pull request #8480: Feature: Extended depots https://github.com/OpenTTD/OpenTTD/pull/8480#pullrequestreview-1325268457 19:52:03 <DorpsGek> [OpenTTD/OpenTTD] Kuhnovic updated pull request #10543: Feature: Region-based pathfinder for ships https://github.com/OpenTTD/OpenTTD/pull/10543 19:53:37 <TrueBrain> okay, staging shows it is all working as expected .. so just need an approval on BaNaNaS to bring this live 😄 \o/ 🥳 19:55:47 <klote[d]> Hi, im tryin to load a specific map during startup of the server 19:56:10 <DorpsGek> [OpenTTD/BaNaNaS] frosch123 approved pull request #130: Feature: convert mentions of regions in tags to regions https://github.com/OpenTTD/BaNaNaS/pull/130#pullrequestreview-1325269047 19:57:04 <frosch> some items had an "archived:" at the end, which was reordered alphabetical in that PR 19:57:12 <frosch> i assume we added those manually before 19:57:17 <klote[d]> i added the line load 2 in the on_server.scr document 19:57:17 <TrueBrain> frosch: yup; I assumed the same 🙂 19:57:22 <klote[d]> but it doesnt seem to work 19:57:37 <TrueBrain> frosch: we really should make that a button in the interface 😛 19:57:43 <klote[d]> but when i directly type load 2 in the rcon it works just fine 19:57:49 <klote[d]> even in the server terminal 20:00:05 <DorpsGek> [OpenTTD/bananas-server] TrueBrain created new tag: v1.7.0 https://github.com/OpenTTD/bananas-server/releases/tag/v1.7.0 20:00:24 <DorpsGek> [OpenTTD/bananas-frontend-web] TrueBrain created new tag: 1.3.0 https://github.com/OpenTTD/bananas-frontend-web/releases/tag/1.3.0 20:00:31 <klote[d]> what scr document am i supposed to load command line to start a specific save game when server starts? 20:00:59 <DorpsGek> [OpenTTD/bananas-api] TrueBrain created new tag: 1.7.0 https://github.com/OpenTTD/bananas-api/releases/tag/1.7.0 20:02:29 <LordAro> klote[d]: not trying to disuade you or anything, but since you're on discord, some of the other help channels may prove more... helpful 20:02:37 <LordAro> this tends to be very dev focused 20:02:44 <klote[d]> which one? 20:02:57 <LordAro> no idea, i'm not on it :p 20:03:00 <LordAro> but i know they exist 20:03:02 <klote[d]> most help so far was from this openttd. 20:03:04 <klote[d]> channel 20:03:15 <TrueBrain> Discord channel #openttd-help: gives the suggestion they want to help 😄 20:08:14 <DorpsGek> [OpenTTD/BaNaNaS] TrueBrain merged pull request #130: Feature: convert mentions of regions in tags to regions https://github.com/OpenTTD/BaNaNaS/pull/130 20:08:26 <TrueBrain> this will disrupt everything for a bit, as the system wasn't designed to process so many mutations at once 😄 20:15:37 <TallTyler> I didn't realize that this backend improvement to Bananas would be implemented immediately, without having to release a new OpenTTD. Very nice! 🙂 20:18:13 <DorpsGek> [OpenTTD/OpenTTD] glx22 updated pull request #10545: Improve ScriptText validation error messages and prevent circular references https://github.com/OpenTTD/OpenTTD/pull/10545 20:19:38 <FLHerne> klote[d]: You can run `openttd -g blah.sav` to load a particular save immediately 20:19:54 <FLHerne> (add -D as usual for dedicated server) 20:19:56 <frosch> TallTyler: a new release would involve in-game dropdowns 20:20:00 <frosch> noone implemented that yet 20:20:02 <TrueBrain> TallTyler: I will give an example (referring to a discussion we had the other day :P) 20:20:26 <klote[d]> FLHerne: Yeah but when i restart server it wont load that specific map 20:20:26 <TrueBrain> TallTyler: I like deploying things without downtime 🙂 20:20:54 <klote[d]> I want it to always load the savegame when game restarts 20:21:58 <FLHerne> then yeah, you need rcon/adminport 20:21:58 <Xarick> what does inline do? explain me like i'm 4 years old 20:22:10 <Xarick> when should I use it 20:22:25 <glx[d]> depending on when the scr is executed, it might happen to do the load command while generating the map 20:22:36 <klote[d]> ah 20:22:47 <FLHerne> klote[d]: actually, if you use `restart_game_date` it should restart the same map, I think? 20:23:02 <DorpsGek> [OpenTTD/OpenTTD] github-code-scanning[bot] commented on pull request #10543: Feature: Region-based pathfinder for ships https://github.com/OpenTTD/OpenTTD/pull/10543#pullrequestreview-1325273448 20:23:11 <glx[d]> I know for instance it's not possible do add commands after load in a script 20:23:27 <glx[d]> because they are then ran during the load 20:23:34 <glx[d]> and not after it 20:23:52 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #10545: Improve ScriptText validation error messages and prevent circular references https://github.com/OpenTTD/OpenTTD/pull/10545#pullrequestreview-1325273545 20:23:57 <Xarick> Yapf code sometimes have inline, sometimes doesn't 20:24:03 <Xarick> i never know the difference 20:24:28 <glx[d]> compiler knows (inline is a hint for compiler and it may not follow it) 20:25:30 <JGR> inline is more than a hint when it comes to headers, one definition rule, etc 20:25:35 <Xarick> https://github.com/OpenTTD/OpenTTD/blob/master/src/pathfinder/yapf/yapf_ship.cpp#L50 this has inline 20:26:02 <Xarick> <https://github.com/OpenTTD/OpenTTD/blob/master/src/pathfinder/yapf/yapf_ship.cpp#L257> this doesn't 20:26:13 <Xarick> so I never know why or when to use it 20:26:29 <klote[d]> Yeah 20:26:39 <klote[d]> tried adding it to other documents doesnt work lol 20:27:09 <klote[d]> what about autoexec.scr - This Script is executed on gamestart [all - use this for custom aliases per ex.] ? 20:27:43 <klote[d]> allot of servers have ingame commands 20:28:00 <Rubidium> Xarick: I think compilers have become smart enough to not have to add inline everywhere, except when it comes to headers like JGR already said 20:28:03 <klote[d]> is that some sort of bot? 20:29:13 <Rubidium> Xarick: any given the trivialness of those two functions, I reckon both are compiled exactly the same by most compilers 20:30:38 *** sla_ro|master has joined #openttd 20:31:39 <Kuhnovic> I finally got my PR to build the preview but it keeps crashing. I thought it was because of my changes but I tried previews from different PR and I get the same. I build two docks, send a ship to go back and forth between the two, and after some time I get a crash. I tried both Chrome and Edge, same results. Has anybody else had this issue? 20:31:51 <glx[d]> it's autosave 20:32:12 <glx[d]> saving crashes for map >= 256x256 tiles 20:33:21 <Kuhnovic> My maps were 64x64 20:33:53 <glx[d]> could still be related to autosave 20:34:53 <glx[d]> oh maybe something with lzma 20:35:57 <Kuhnovic> Turned auto save off, lets give it another go 20:37:09 <glx[d]> as we patch lzma source to build emscripten version 20:37:25 <klote[d]> this wiki is pointing towards a bot called soap but i cant seem to find a download. 20:37:31 <klote[d]> http://wiki.openttdcoop.org/Soap 20:38:34 <glx[d]> yeah unfortunately it's on openttdcoop 20:41:15 <klote[d]> hmm 20:42:02 <TrueBrain> owh bah, I don't always filter empty regions .. so now it is committing `regions: []` 20:42:04 <TrueBrain> not that it matters 20:42:06 <TrueBrain> it just annoys me 😛 20:42:14 <TrueBrain> it did the same with tags, which annoyed me equally 20:43:36 <klote[d]> libottdadmin <- people using this to automate things on servers? 20:44:00 <nielsm> Xarick: in C++ the "inline" keyword does not mean the function should be inlined, it means the function is allowed to be defined multiple times and not cause an error. You need to use "inline" on free functions defined in header files , i.e. functions that have their function body in the .h file, and are not inside a class. For any other cases, the "inline" keyword does nothing. 20:47:29 <glx[d]> it's a lib to connect to server admin port 20:48:12 <Kuhnovic> https://cdn.discordapp.com/attachments/1008473233844097104/1082041898044231721/image.png 20:48:12 <Kuhnovic> glx[d]: disabling autosave did the trick. Look mum, no buoys! 20:48:23 <frosch> oh dear.. i expected something like this to happen... you can only have 25 discussion categories :p 20:48:29 <frosch> we have 64 translations 20:48:33 <glx[d]> oups 20:49:03 <Xarick> glx[d]: ,it crashes for me for 256x256 20:49:29 <TrueBrain> frosch: ... lol 20:49:44 <TrueBrain> so, time for a new organization called OpenTTD-translators, and a repository per language? 🙂 20:49:54 <TrueBrain> just for everyone to have a discussion in their own language? 20:50:04 <TrueBrain> such a pro-move of GitHub, removing discussions from teams 😛 20:50:26 <glx[d]> or tag discussion with a [ISO_CODE] 20:50:58 <glx[d]> and hope translators will be smart 20:51:10 <frosch> the nice thing about categories was, that you had unique urls for them, like https://github.com/OpenTTD/team/discussions/categories/fr_fr 20:51:39 <TallTyler> TrueBrain: I took a stab at adding control codes to make real-time units optional: https://github.com/2TallTyler/OpenTTD/commit/18e373896c3803f0321c16278268d81fd7018cb4 20:51:56 <TallTyler> It works, although I haven't committed any changes that use the new codes yet 20:52:05 <TrueBrain> not bad 😄 20:52:17 <TrueBrain> and sorry, I wanted to give it a spin of what was in my head, but BaNaNaS blabla took prio 😛 20:53:02 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on discussion #10509: Removal of old tags hinders finding new content https://github.com/OpenTTD/OpenTTD/discussions/10509 20:53:11 <TallTyler> I think I can continue to run the economy using `_economy_date`, but outside real-time mode, keep that in sync with `_date` in `OnNewDay()` instead of updating them separately 20:53:36 <frosch> ok, back-up plan: check which teams actually discussed anything in the past, and only give them categories :p 20:54:23 <TrueBrain> TallTyler: sounds like you have a plan 😄 20:54:35 <TallTyler> Yes! 😄 20:55:02 <TallTyler> It's a lot to slog through, but these codes were a big mental hurdle to overcome, so hopefully I can make faster progress 20:55:12 <TrueBrain> ghehe 20:55:14 <andythenorth> oof do I want to write a recursive python->squirrel list converter? 😛 20:55:18 <TrueBrain> welcome to development 😄 20:55:20 <andythenorth> I do not like recursive things 😛 20:56:14 <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #10546: Fix 30ae072: when a road type is hidden, towns may not build them even when that flag is set https://github.com/OpenTTD/OpenTTD/pull/10546#pullrequestreview-1325277559 20:56:46 <frosch> shall i include those language, where a poor soul posted 1 post, and noone replied? 🙂 20:58:08 <TrueBrain> frosch: nah; do the important ones first 😛 20:59:47 <nielsm> TallTyler: I don't know the full context of it, but in that text stuff with choice between calendar/realtime units, be careful if those stand-alone STR_TIME_MONTH_SINGULAR/STR_TIME_MONTHS_PLURAL can be used in a context together with concrete numbers since the plurals can be much more complex then 20:59:56 <TallTyler> I'm getting a weird crash when closing OpenTTD. Am I reading this right that `AI::Uninitialize` is causing the crash, as it's at the top of the stack trace? 20:59:56 <TallTyler> `[00] openttd 0x00007FF786EB9BFB AI::Uninitialize + 427 (C:\Users\tyler\Documents\GitHub\OpenTTD\src\ai\ai_core.cpp:195)` 21:00:18 <TallTyler> (if I'm right, maybe it's not something I broke :P) 21:01:57 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #10546: Fix 30ae072: when a road type is hidden, towns may not build them even when that flag is set https://github.com/OpenTTD/OpenTTD/pull/10546 21:03:03 <glx[d]> I have `delete _settings_newgame.ai_config[c];` for this line 21:03:13 <glx[d]> and it's guarded 21:03:27 <TallTyler> nielsm: The standalone units are mostly used in setting names and helptexts where either no number exists, or the number is hardcoded into the string. I can't replace the latter (as far as I'm aware) because I can't SetDParam()s for these strings, as they're only referenced in the setting .ini file 21:04:33 <frosch> 6 teams used the discussions more than once, in 6 teams a single person made a post without nay reply, in 3 teams the only post is from me for some admin stuff :p 21:04:45 <klote[d]> Can any one point me towards a openttd server bot that allows ingame commands and automated messages 21:05:03 <klote[d]> or are people making this themselves? 21:05:33 <glx[d]> I think most write admin bots tailored to their server 21:05:39 <klote[d]> holy cow 21:05:40 <TallTyler> glx[d]: Same, and I should note that I can't reliably reproduce the crash 21:06:30 <TallTyler> So far it's only when exiting the game from the intro menu, but not every time 21:06:31 <nielsm> TallTyler: yeah that's what I thought too, it just feels dangerous to put those strings as standalone... I wonder if it would be better to introduce a formatting code that selects between two strings based on the realtime on/off setting. that will allow translators to put whatever form is appropriate within strings (like grammatical case might differ?) 21:07:32 <TallTyler> I don't quite understand your proposed solution. Could you write some psuedocode for what the strings might look like? 21:07:48 <nielsm> also that could solve issues that might occur if the words sometimes need to occur at the start of a sentence and would need to be capitalized, instead of having to need even more variations of those static strings 21:08:22 <DorpsGek> [OpenTTD/OpenTTD] J0anJosep updated pull request #9577: Feature: Multi-tile depots https://github.com/OpenTTD/OpenTTD/pull/9577 21:09:41 <nielsm> like a graph legend text: "Payment for one unit of cargo delivered in number of {RTS "days" "seconds"}" 21:09:45 <frosch> on the plus side: the translation that used the discussions are all from stable countries, so I can use flags again 21:10:27 <nielsm> "{RTS "Yearly" "Period-wise"} check enabled" 21:10:54 <nielsm> RTS = Real-Time Setting, in this idea 21:11:11 <TallTyler> Hmm, that does look significantly better 21:11:19 <TallTyler> Good suggestion 21:11:43 <TallTyler> Do we have any string codes that take parameters like that? 21:12:00 <nielsm> well the plural and case codes? 21:12:41 <glx[d]> better use plural as reference 😉 21:13:08 <glx[d]> just ignore the index ref 21:13:08 <JGR> Isn't that likely to result in needing to nest plural/case codes inside your {RTS} block? 21:13:14 <JGR> That could get complicated 21:13:35 <TallTyler> Oh duh, plurals 😛 21:14:55 <nielsm> JGR: I think this form would only be used for situations that are numeral-less, so nested plurals shouldn't be needed, for the cases where you do have numerals you use the inserts for "{NUM} <timeunit>" 21:16:02 <TallTyler> The unit control codes do that already, to handle unit conversion like 1 day = 2 seconds 21:16:10 <nielsm> yes 21:17:16 <TallTyler> It’s a bit silly to do that with months = minutes and years = periods since they are both 1:1, but I liked the consistency 21:23:30 *** gelignite has quit IRC 21:24:13 <DorpsGek> [OpenTTD/OpenTTD] glx22 merged pull request #10545: Improve ScriptText validation error messages and prevent circular references https://github.com/OpenTTD/OpenTTD/pull/10545 21:24:16 <DorpsGek> [OpenTTD/OpenTTD] glx22 closed issue #10542: [Bug]: BusyBee crashes on a new map since 991a797e098e22b8b3a0adaead757d691d80fca4 https://github.com/OpenTTD/OpenTTD/issues/10542 21:27:17 *** nielsm has quit IRC 21:31:21 *** sla_ro|master has quit IRC 21:31:52 <frosch> https://cdn.discordapp.com/attachments/1008473233844097104/1082052887166845059/image.png 21:31:52 <frosch> <- anythingmissing? 21:41:14 <Xarick> I need to come up with a strategy to deal with autoreplace on my AI. Need a way to not care, or not keep track of all vehicles in a route, but still know that the route has vehicles. I don't want to invoke vehicle lists, because they're going to be cpu hogs over time with the amount of vehicles increasing. 21:41:43 <Xarick> I still need to know which vehicles are not profitable, and that's the complicated part 21:42:39 *** keikoz has quit IRC 21:43:30 <TrueBrain> frosch: Nope, seems good! 21:45:02 <TallTyler> The “first come first serve” policy makes sense to me 21:45:56 <frosch> newsletter sent, let's see whether at least team notifications work, and people show up :p 21:50:01 <TallTyler> I got the notification 21:50:29 <TallTyler> en_US translators represent, maybe the most useless team! 😛 21:50:49 <frosch> i think on devzone we had someone sign up for en_GB 21:51:03 <frosch> we did not offer that this time :p 21:51:19 <Xarick> <https://github.com/OpenTTD/OpenTTD/blob/master/src/script/api/script_vehiclelist.cpp#L92-L99> this one looks promising 21:51:54 <Xarick> getting a list of vehicles of a route via list of vehicles sharing orders 21:52:41 <Xarick> what I really wanted though was ScriptVehicleList_Group to be similar in design as ScriptVehicleList_SharedOrders 21:54:21 <Xarick> i could use ScriptVehicleList_SharedOrders if ships were smart enough to reach nearest depots 21:54:42 <Xarick> *cough* merge 10544 21:57:49 <Xarick> or I could look into openttd code, and make groups keep track of vehicles in it 22:03:35 <Xarick> if I alter an order in a shared order list to send to depot and stop, all vehicles would go stop, and I only wanted to act on 1 vehicle. If I unshare the vehicle, the vehicle becomes "out of the loop", ScriptVehicleList_SharedOrders becomes ineffective to keep track of vehicles 22:05:51 <DorpsGek> [OpenTTD/team] frosch123 opened pull request #406: Change: Team discussions are dead. Long live team discussions. https://github.com/OpenTTD/team/pull/406 22:08:44 <frosch> hmm, how are those scripts deployed? 22:09:19 <DorpsGek> [OpenTTD/team] 2TallTyler approved pull request #406: Change: Team discussions are dead. Long live team discussions. https://github.com/OpenTTD/team/pull/406#pullrequestreview-1325287753 22:10:48 <Xarick> The way I have my AI working right now is to never use autoreplace. It currently creates a list of vehicles per group, and manually do replacements on vehicles of that group. If the vehicles I keep track are suddenly crashed, or belong to another route (because I build more vehicles reusing the id I was tracking, via horrible checks), I remove them from the list to keep it as fresh as possible 22:11:11 <frosch> looks like no deployment needed. stuff is directly triggered from main branch 22:12:53 <Xarick> this way I don't lose track of any vehicle I create, but at the same time, it makes vehicle maintenance, route maintenance slow... not automated, meaning my AI performance is not at the top. 22:13:21 <Xarick> it becomes a big problem when i deal with hundreds of routes 22:13:38 <Xarick> vehicles end up waiting in depots for a replacement for years 22:14:34 <TrueBrain> frosch: Yup 22:14:37 <Xarick> if I could find a way to shift this burden to autoreplace and autorenew, my AI would be faster, wouldn't have to deal with these tasks 22:15:24 <TrueBrain> Does seem the repo needs some glx[d] love 😄 22:16:21 <Xarick> it would compete better, have more time for pathfinding and so on 22:16:47 <frosch> is glx master of check-annotations? 22:17:10 <TrueBrain> He became a master, after fixing so many repos 22:17:45 <glx[d]> another repo needs action bumps ? 22:18:26 <frosch> https://github.com/OpenTTD/team/pull/406 <- that one 22:24:08 <DorpsGek> [OpenTTD/OpenTTD] glx22 updated pull request #10533: Codechange: Use SQInteger for generic numbers in script API https://github.com/OpenTTD/OpenTTD/pull/10533 22:34:41 <Xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1082068696299745321/screenshot18.png 22:34:41 <Xarick> TallTyler: found a river at sea level 22:45:51 *** lobstarooo__ has quit IRC 22:46:03 <DorpsGek> [OpenTTD/team] glx22 opened pull request #407: Update: [Actions] Bump used actions versions https://github.com/OpenTTD/team/pull/407 22:46:06 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #10533: Codechange: Use SQInteger for generic numbers in script API https://github.com/OpenTTD/OpenTTD/pull/10533#pullrequestreview-1325292625 22:47:59 <DorpsGek> [OpenTTD/team] frosch123 commented on pull request #407: Update: [Actions] Bump used actions versions https://github.com/OpenTTD/team/pull/407#issuecomment-1455232205 22:48:35 <frosch> and master is 7 commits behind main 22:49:18 <TrueBrain> Please don't use master/main 22:49:22 <TrueBrain> Use a version tag 22:49:38 <DorpsGek> [OpenTTD/OpenTTD] glx22 updated pull request #10533: Codechange: Use SQInteger for generic numbers in script API https://github.com/OpenTTD/OpenTTD/pull/10533 22:50:00 <glx[d]> I copied nml workflow for this one 22:50:04 <TrueBrain> v2 in this case 22:50:13 <TrueBrain> Means we should fix nml too 😄 22:50:53 <TrueBrain> The idea behind it is, that majors always stay compatible 22:50:54 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #10533: Codechange: Use SQInteger for generic numbers in script API https://github.com/OpenTTD/OpenTTD/pull/10533#pullrequestreview-1325294227 22:51:08 <TrueBrain> Main ofc can break :p 22:52:34 <DorpsGek> [OpenTTD/team] glx22 updated pull request #407: Update: [Actions] Bump used actions versions https://github.com/OpenTTD/team/pull/407 22:54:44 <DorpsGek> [OpenTTD/team] frosch123 approved pull request #407: Update: [Actions] Bump used actions versions https://github.com/OpenTTD/team/pull/407#pullrequestreview-1325294807 23:00:16 <DorpsGek> [OpenTTD/nml] glx22 opened pull request #280: Update: [Actions] Bump used actions versions https://github.com/OpenTTD/nml/pull/280 23:00:34 <DorpsGek> [OpenTTD/nml] frosch123 approved pull request #280: Update: [Actions] Bump used actions versions https://github.com/OpenTTD/nml/pull/280#pullrequestreview-1325295803 23:00:37 <DorpsGek> [OpenTTD/team] glx22 merged pull request #407: Update: [Actions] Bump used actions versions https://github.com/OpenTTD/team/pull/407 23:02:15 <andythenorth> such bedtime 23:03:24 <DorpsGek> [OpenTTD/team] frosch123 updated pull request #406: Change: Team discussions are dead. Long live team discussions. https://github.com/OpenTTD/team/pull/406 23:04:12 <DorpsGek> [OpenTTD/team] frosch123 merged pull request #406: Change: Team discussions are dead. Long live team discussions. https://github.com/OpenTTD/team/pull/406 23:05:59 <DorpsGek> [OpenTTD/nml] glx22 merged pull request #280: Update: [Actions] Bump used actions versions https://github.com/OpenTTD/nml/pull/280 23:19:29 *** Wormnest has quit IRC 23:21:48 *** WormnestAndroid has quit IRC 23:32:16 <DorpsGek> [OpenTTD/OpenTTD] glx22 merged pull request #10533: Codechange: Use SQInteger for generic numbers in script API https://github.com/OpenTTD/OpenTTD/pull/10533 23:33:28 *** WormnestAndroid has joined #openttd 23:46:59 *** HerzogDeXtEr has quit IRC