IRC logs for #openttd on OFTC at 2022-11-03
#openttd / 2022 / 11 
            

00:11:24  *** WormnestAndroid has joined #openttd



00:58:09  *** Wormnest has quit IRC (Quit: Leaving)



01:22:28  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10114: Feature: Variable interface scaling https://github.com/OpenTTD/OpenTTD/pull/10114#pullrequestreview-1166178432



01:27:00  *** Flygon has joined #openttd



01:41:36  <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #10114: Feature: Variable interface scaling https://github.com/OpenTTD/OpenTTD/pull/10114



03:55:32  *** _aD has quit IRC (Quit: leaving)



03:59:55  *** D-HUND has joined #openttd



04:03:16  *** debdog has quit IRC (Ping timeout: 480 seconds)



05:02:28  *** supermop_toil_ has joined #openttd



05:08:12  *** supermop_toil has quit IRC (Ping timeout: 480 seconds)



05:54:56  <DorpsGek> [OpenTTD/team] COOKIEMONSTRR opened issue #356: [et_EE] Translator access request https://github.com/OpenTTD/team/issues/356



06:19:32  <pickpacket> I can't find a plane that's refittable to carry new kinds of cargo. Is any base plane using cargo types yet?



06:35:55  *** Tirili has quit IRC (Quit: Leaving)



06:39:39  <EmperorJake> I think you want the "Usable default aircraft" GRF for that



07:08:26  *** sla_ro|master has joined #openttd



08:28:29  <DorpsGek> [OpenTTD/OpenTTD] niklasva opened issue #10125: OpenTTD is causing excessive syncing to iCloud drive https://github.com/OpenTTD/OpenTTD/issues/10125



08:44:06  <DorpsGek> [OpenTTD/OpenTTD] krysclarke opened pull request #10126: CodeChange: Move Sound Driver paramater name listings https://github.com/OpenTTD/OpenTTD/pull/10126



08:48:18  <pickpacket> I've tried out my tea mod a lot now, and it's hard to make it profitable. Tea leaves need to be delivered fresh in reality, so I've set them to start out at a decent cargo payment rate but decreasing rapidly. Even on a medium distance they make terribly little money, sadly. Tea -- the finished product -- starts out at about the rate of goods and



08:48:18  <pickpacket> decreases very slowly. It should theoretically be really good for long hauls, but at least on cargo ships in 1950 they're very unprofitable



08:48:54  <pickpacket> I don't know how to adjust the rates. I could set tea leaves to decrease less of course, even though that makes it a lot less realistic



08:49:19  <pickpacket> it's hard to decide how it should compare to other cargoes as well



08:49:44  <pickpacket> does it start decreasing in value when it's been loaded or when it leaves the station?



08:56:15  <DorpsGek> [OpenTTD/OpenTTD] krysclarke closed pull request #10126: Codechange: Move Sound Driver paramater name listings https://github.com/OpenTTD/OpenTTD/pull/10126



09:13:51  <DorpsGek> [OpenTTD/OpenTTD] krysclarke opened pull request #10127: Codechange: Move Sound Driver parameter name listings https://github.com/OpenTTD/OpenTTD/pull/10127



09:17:09  <petern> Hmm, is LTO enabled for Linux builds?



09:25:30  <pickpacket> EmperorJake: sounds reasonable, but would it be possible to fix the default airplanes in the game?



09:26:38  <LordAro> petern: should be



09:27:27  <EmperorJake> pickpacket: They already allowed the default road vehicles and trains to be refitted to custom cargo, not sure why they didn't for aircraft



09:28:20  <petern> I don't see "-flto" anywhere, but it might be on by default theses days, or some other option enables it.



09:31:36  <EmperorJake> EmperorJake: Actually I just tested it, it seems the default aircraft are intentionally limited to cargos marked with the Express cargo class



09:31:52  <LordAro> petern: IPO_FOUND, top level cmakelists



09:33:12  <petern> Ah, "some other option" 🙂



09:41:39  <pickpacket> EmperorJake: huh. Odd.



09:41:44  <pickpacket> but that explains it



09:43:30  <petern> How does one debug the wasm build?



09:49:48  <LordAro> by asking TB



09:52:42  <petern> 😄



10:33:40  <DorpsGek> [OpenTTD/OpenTTD] SamuXarick commented on pull request #9901: Fix: Improve performance when calculating company value https://github.com/OpenTTD/OpenTTD/pull/9901#pullrequestreview-1166638325



11:37:21  <glx[d]> petern: Check how it's done in workflows



12:14:32  *** supermop_toil has joined #openttd



12:20:58  *** supermop_toil_ has quit IRC (Ping timeout: 480 seconds)



12:25:28  *** tokai|noir has joined #openttd



12:25:29  *** ChanServ sets mode: +v tokai|noir



12:32:06  *** tokai has quit IRC (Ping timeout: 480 seconds)



12:35:16  *** wallabra_ has joined #openttd



12:36:29  *** wallabra has quit IRC (Ping timeout: 480 seconds)



12:36:29  *** wallabra_ is now known as wallabra



12:39:54  *** D-HUND is now known as debdog



13:15:41  *** sla_ro|master has quit IRC ()



13:22:03  <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on issue #10125: OpenTTD is causing excessive syncing to iCloud drive https://github.com/OpenTTD/OpenTTD/issues/10125



13:23:10  <DorpsGek> [OpenTTD/team] glx22 commented on issue #356: [et_EE] Translator access request https://github.com/OpenTTD/team/issues/356



14:17:04  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #9901: Fix: Improve performance when calculating company value https://github.com/OpenTTD/OpenTTD/pull/9901#pullrequestreview-1166994754



14:23:37  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler commented on pull request #10058: Feature, Change: [UI] Split AI/Game Script configuration windows and add them to world gen window https://github.com/OpenTTD/OpenTTD/pull/10058#issuecomment-1302191040



14:49:43  *** wallabra_ has joined #openttd



14:51:59  *** wallabra has quit IRC (Ping timeout: 480 seconds)



14:51:59  *** wallabra_ is now known as wallabra



15:01:07  *** sla_ro|master has joined #openttd



15:03:53  *** nielsm has joined #openttd



15:15:50  <DorpsGek> [OpenTTD/OpenTTD] orudge commented on issue #10125: OpenTTD is causing excessive syncing to iCloud drive https://github.com/OpenTTD/OpenTTD/issues/10125



16:18:15  *** Flygon has quit IRC (Quit: A toaster's basically a soldering iron designed to toast bread)



16:24:43  *** HerzogDeXtEr has joined #openttd



16:40:14  *** gelignite has joined #openttd



17:12:55  *** Wormnest has joined #openttd



17:27:55  <pickpacket> I'm looking at https://wiki.openttd.org/en/Manual/Game%20Mechanics/#delivery-payment-rates but there's something that's unclear to me: when is the delivery time count starting? Is it when cargo is dropped off at the source station, when it's loaded onto the transport, or when the transport leaves the station?



17:28:35  <TallTyler> When it’s loaded, I believe



18:00:16  <pickpacket> oh... that explains a lot 



18:01:00  <pickpacket> standing at the station waiting for a full load can be pretty costly then



18:05:07  <glx[d]> you need to adapt capacity so you can always have a vehicle loading, but not taking to long to fully load



18:15:34  <frosch> iirc cargo only ages when inside vehicles



18:15:43  <frosch> so it ages while the train is waiting for full load



18:15:53  <frosch> but it does not age when no vehicle is loading



18:18:02  <Merni> weird



18:18:39  <Merni> so would it be better in that sense to have a very infrequent service so that there is always a full load when the train comes around



18:18:49  <Merni> I guess that wouldn't make the station rating happy though



18:22:47  <frosch> yes, no train waiting is bad for rating



18:23:01  <frosch> but having a way too long train that takes longer to load than to drive, that is bad as well



18:23:51  <frosch> but ofc. if you play on 4kx4k, then the cargo age is maxed out anyway, and it makes no difference anymore



18:25:40  <petern> 256 forever



18:38:04  <glx[d]> I do all my tests on 64 (because it's faster to generate)



18:45:52  *** geli has joined #openttd



18:52:14  *** gelignite has quit IRC (Ping timeout: 480 seconds)



18:55:42  <DorpsGek> [OpenTTD/OpenTTD] DorpsGek pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/accbfd502e109267be90307258615dec2d844b36



18:55:43  <DorpsGek>   - Update: Translations from eints (by translators)



18:57:32  *** Wolf01 has joined #openttd



18:59:45  <pickpacket> I was testing out my tea mod last night on a pretty big map with some others (joint company). Since tea leaves age quickly we set up shop in an area where several tea farms were close to a factory (where they deliver tea leaves). Those short routes were okay in terms of profitability, but when we loaded a ship full of tea to take on a longer



18:59:45  <pickpacket> trip… that was just a massive money/time sink



19:00:05  <pickpacket> even though Tea Tea Deluxe ages better than coal



19:01:17  <pickpacket> Basically I don't know whether I should tweak the cargo values or my transportation strategy :P



19:04:19  *** wallabra_ has joined #openttd



19:05:00  <petern> > Using pointer to local variable 'extra' that is out of scope.CppCheck (c-cpp-flylint)(invalidLifetime)



19:05:04  <petern> Possibly found...



19:09:59  *** wallabra has quit IRC (Ping timeout: 480 seconds)



19:09:59  *** wallabra_ is now known as wallabra



19:13:44  <petern> Also somehow we sometimes we provide nullptr to a reference.



19:13:47  <pickpacket> of course the cargo ship in question only goes 25km/h....



19:15:26  <petern> Ah no, we dereference it, without checking it's not null.



19:17:09  <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #10114: Feature: Variable interface scaling https://github.com/OpenTTD/OpenTTD/pull/10114



19:21:30  <petern> Suspicious pointer subtraction. Did you intend to write '->'?



19:21:32  <petern> Nice



19:23:28  *** _aD has joined #openttd



19:25:45  <Eddi|zuHause> how dare a compiler assert the right to guess what the programmer wants!!! :p



19:29:48  <petern> It's not the compiler, it's from cppcheck, so it's kinda the point of it.



19:30:42  <petern> Perfect, solved the wasm build \o/



19:30:50  <petern> So yeah, useful tool.



20:00:11  <DorpsGek> [OpenTTD/OpenTTD] stormcone commented on pull request #10114: Feature: Variable interface scaling https://github.com/OpenTTD/OpenTTD/pull/10114#issuecomment-1302601655



20:04:24  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10114: Feature: Variable interface scaling https://github.com/OpenTTD/OpenTTD/pull/10114#issuecomment-1302606473



20:58:34  <DorpsGek> [OpenTTD/OpenTTD] LC-Zorg commented on pull request #10114: Feature: Variable interface scaling https://github.com/OpenTTD/OpenTTD/pull/10114#issuecomment-1302655462



21:33:42  *** geli has quit IRC (Quit: Stay safe!)



21:43:44  *** nielsm has quit IRC (Ping timeout: 480 seconds)



22:16:16  *** m3henry has joined #openttd



22:16:20  <DorpsGek> [OpenTTD/OpenTTD] M3Henry updated pull request #10123: Feature: Display power-to-weight ratio in ground vehicle details GUI https://github.com/OpenTTD/OpenTTD/pull/10123



22:21:17  <DorpsGek> [OpenTTD/OpenTTD] PeterN opened pull request #10128: Fix: Ensure 31-bit shifts are unsigned. https://github.com/OpenTTD/OpenTTD/pull/10128



22:25:46  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler requested changes for pull request #10123: Feature: Display power-to-weight ratio in ground vehicle details GUI https://github.com/OpenTTD/OpenTTD/pull/10123#pullrequestreview-1167737678



22:27:15  <m3henry> Indeed



22:30:24  *** sla_ro|master has quit IRC ()



22:31:02  *** HerzogDeXtEr has quit IRC (Read error: Connection reset by peer)



22:32:51  <DorpsGek> [OpenTTD/OpenTTD] M3Henry updated pull request #10123: Feature: Display power-to-weight ratio in ground vehicle details GUI https://github.com/OpenTTD/OpenTTD/pull/10123



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



22:35:16  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler dismissed a review for pull request #10123: Feature: Display power-to-weight ratio in ground vehicle details GUI https://github.com/OpenTTD/OpenTTD/pull/10123#pullrequestreview-1167737678



22:44:02  <DorpsGek> [OpenTTD/OpenTTD] michicc approved pull request #10128: Fix: Ensure 31-bit shifts are unsigned. https://github.com/OpenTTD/OpenTTD/pull/10128#pullrequestreview-1167751779



23:01:08  <DorpsGek> [OpenTTD/OpenTTD] 2TallTyler approved pull request #10123: Feature: Display power-to-weight ratio in ground vehicle details GUI https://github.com/OpenTTD/OpenTTD/pull/10123#pullrequestreview-1167762630



23:03:37  *** wallabra has quit IRC (Ping timeout: 480 seconds)



23:04:09  <DorpsGek> [OpenTTD/OpenTTD] JGRennison opened issue #10129: [Bug]: Use after free via SetDParamStr for STR_CONFIG_ERROR_INVALID_GRF https://github.com/OpenTTD/OpenTTD/issues/10129



23:05:55  *** wallabra has joined #openttd



23:14:10  <DorpsGek> [OpenTTD/OpenTTD] M3Henry dismissed a review for pull request #10123: Feature: Display power-to-weight ratio in ground vehicle details GUI https://github.com/OpenTTD/OpenTTD/pull/10123#pullrequestreview-1167762630



23:14:13  <DorpsGek> [OpenTTD/OpenTTD] M3Henry updated pull request #10123: Feature: Display power-to-weight ratio in ground vehicle details GUI https://github.com/OpenTTD/OpenTTD/pull/10123



23:14:53  <m3henry> Just some additional comments



23:15:33  <petern> Did you fix the compile warnings?



23:21:08  <m3henry> Those didn't seem related, but I'll check again



23:23:39  <DorpsGek> [OpenTTD/OpenTTD] PeterN dismissed a review for pull request #10128: Fix: Ensure 31-bit shifts are unsigned. https://github.com/OpenTTD/OpenTTD/pull/10128#pullrequestreview-1167751779



23:23:42  <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #10128: Fix: Ensure 31-bit shifts are unsigned. https://github.com/OpenTTD/OpenTTD/pull/10128



23:24:04  <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #10128: Fix: Ensure 31-bit shifts are unsigned. https://github.com/OpenTTD/OpenTTD/pull/10128#issuecomment-1302784794



23:25:03  <m3henry> Oh I see, it's because I added the override specifier



23:25:19  <m3henry> I'll deal with that in the morning



23:25:54  <TallTyler> Request a re-review when you're ready 🙂