β΄ go to previous day02:20:37 *** Wormnest has quit IRC (Quit: Leaving)02:47:56 *** gnu_jj_ has joined #openttd02:51:10 *** geizeskrank has quit IRC (Ping timeout: 480 seconds)02:51:17 *** gnu_jj has quit IRC (Ping timeout: 480 seconds)02:54:42 *** geizeskrank has joined #openttd04:09:05 *** keikoz has joined #openttd04:19:48 *** WormnestAndroid has quit IRC (Remote host closed the connection)04:19:58 *** WormnestAndroid has joined #openttd04:35:22 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)04:35:24 *** WormnestAndroid has joined #openttd04:35:31 *** WormnestAndroid has quit IRC (Read error: Connection reset by peer)04:40:57 *** WormnestAndroid has joined #openttd04:50:04 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #13958: Codechange: Manage window viewport with std::unique_ptr. https://github.com/OpenTTD/OpenTTD/pull/13958#pullrequestreview-273846471904:51:07 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #13942: Revert f719fa678f: [CI] The fix is now implemented at vcpkg level https://github.com/OpenTTD/OpenTTD/pull/13942#pullrequestreview-273848736905:14:50 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 commented on pull request #13953: Add Utf8View to simplify iterating over UTF-8 data with bounds checking https://github.com/OpenTTD/OpenTTD/pull/13953#pullrequestreview-273850285205:27:43 <andythenorth> well05:28:01 <andythenorth> coffee?05:30:06 *** keikoz has quit IRC (Ping timeout: 480 seconds)06:44:23 <kuhnovic> YES06:57:49 *** Flygon has quit IRC (Quit: A toaster's basically a soldering iron designed to toast bread)07:23:12 <_zephyris> β Second cup, going down now.07:28:46 <_zephyris> So, `alternative_sprites` in `if` blocks are allowed, but don't work in `if` blocks - they just get included/used whatever.07:39:54 <peter1138> What are you actually trying to do?07:41:57 <peter1138> The binary NewGRF file format doesn't allow for conditional alternative sprites.07:45:32 <jfkuayue> Haggis!07:51:12 *** debdog has joined #openttd07:52:12 <_zephyris> I'm not trying to do conditional alternative per se07:52:37 <_zephyris> Trying to work out if it would be better to have an error instead of a warning if alternative sprites get redefined07:53:20 <_zephyris> (and testing conditional alternate handling as a good example of why allowing redefinition gives unexpected behaviour)08:02:00 <peter1138> So are you talking about modifying NML to give an error?08:03:30 <_zephyris> IMO yes08:04:12 <_zephyris> Both for alternative sprites which are isolated from their parent in a conditional block, and for redefinition of an existing combination of zoom/bit depth08:06:52 <_zephyris> First because it won't behave as expected, second because I can't see a useful reason to use for it08:07:37 <peter1138> See, I asked what you were trying to do, and you told me what you are not trying to do ;-)08:08:46 <_zephyris> Welll, I'm not sure I'm _able_ to do what you asked π08:09:52 <_zephyris> Currently it's schrodingers what am I actually trying to do08:37:22 <peter1138> Huh.08:37:28 <peter1138> "Why is this feature not working?"08:37:36 <peter1138> Well, being on the right branch would have helped me.08:49:38 <DorpsGek> [OpenTTD/nml] zephyris opened issue #371: Bug: `alternative_sprites` in conditional blocks away from the parent behave inconsistently https://github.com/OpenTTD/nml/issues/37108:55:25 <xarick> hi08:58:02 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1357277956074115093/image.png?ex=67ef9f19&is=67ee4d99&hm=7b252829c43955f8789eec51ca8b765fdcc2e51bad8fca39fca1aa4d2dbf085c&08:58:02 <xarick> I can't wait any longer08:59:09 <DorpsGek> [OpenTTD/nml] zephyris opened issue #372: Bug(?): Re-definition of alternative sprites is not treated as an error but has no productive use https://github.com/OpenTTD/nml/issues/37208:59:18 <kuhnovic> Sure you can, you're not running at 0.01x yet09:00:20 <xarick> i wish things would go faster09:00:30 <xarick> AI is not efficient enough09:00:50 *** mindlesstux has quit IRC (Quit: The Lounge - https://thelounge.chat)09:01:46 *** mindlesstux has joined #openttd09:10:33 *** akimoto has joined #openttd09:17:51 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1357282944720437410/screenshot78.png?ex=67efa3bf&is=67ee523f&hm=a6e327534029b3788858ddfd2064250f33ead65cfa0c38fe57731f5a7f8bd90d&09:28:17 <kuhnovic> The good ol' 4K map again...09:31:06 <peter1138> You wouldn't have these performance issues if you didn't increase all the limits that are there for reasons.09:31:28 <peter1138> (He says, not loading Wentbourne)09:31:51 <peter1138> ((Although that's probably using increased limits too.))09:37:52 <exceptik> peter1138: good ol hiding problems under carpet method?10:35:53 <michi_cc> No, more like "Why does my head hurt if I hit it with a hammer?"10:39:33 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1357303503667728454/image.png?ex=67efb6e4&is=67ee6564&hm=bcd3892cf8859670e9c1082a214f01e1e8ebe5ad972acaa18f01ab83ca5c317d&10:39:33 <xarick> looks like my other game with supersimpleais has reached the orderlist limit or something10:44:59 <xarick> let me confirm something10:55:00 <DorpsGek> [OpenTTD/OpenTTD] glx22 merged pull request #13942: Revert f719fa678f: [CI] The fix is now implemented at vcpkg level https://github.com/OpenTTD/OpenTTD/pull/1394211:17:03 <DorpsGek> [OpenTTD/nml] zephyris commented on issue #371: Bug: `alternative_sprites` in conditional blocks away from the parent behave inconsistently https://github.com/OpenTTD/nml/issues/37111:32:49 <xarick> no limits11:32:57 <xarick> it's the ai failing11:33:47 *** SigHunter_ has quit IRC ()11:39:42 *** SigHunter has joined #openttd11:40:20 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #13946: Fix: Reference to the correct section of the README if a graphics or β¦ https://github.com/OpenTTD/OpenTTD/pull/1394611:48:05 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #13906: Fix: [Script] Reset instance when changing running scripts in scenario editor https://github.com/OpenTTD/OpenTTD/pull/13906#pullrequestreview-273956832911:51:30 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #13896: Fix #10179: [Script] Only run the Game::GameLoop() in-game https://github.com/OpenTTD/OpenTTD/pull/13896#issuecomment-277550912111:51:57 <DorpsGek> [OpenTTD/OpenTTD] PeterN updated pull request #13940: Codechange: Store loaded GRFFiles in vector directly. https://github.com/OpenTTD/OpenTTD/pull/1394011:55:44 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #13949: Fix: MayHaveRoad claimed rail station tiles had road, so the custom stationspec index would be read as roadtype https://github.com/OpenTTD/OpenTTD/pull/13949#issuecomment-277551863711:56:54 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #13950: Codefix: Conversion from char to char32_t requires cast to unsigned first https://github.com/OpenTTD/OpenTTD/pull/1395011:57:33 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #13951: Codefix dccc6185: Incorrect encoding of empty parameters in ScriptText. https://github.com/OpenTTD/OpenTTD/pull/1395111:58:11 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #13952: Simplify Czech townname generation https://github.com/OpenTTD/OpenTTD/pull/1395211:59:18 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #13906: Fix: [Script] Reset instance when changing running scripts in scenario editor https://github.com/OpenTTD/OpenTTD/pull/13906#pullrequestreview-273959492012:02:29 *** nebulabc has quit IRC (Remote host closed the connection)12:02:29 *** asymptotically2 has quit IRC (Remote host closed the connection)12:02:50 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #13953: Add Utf8View to simplify iterating over UTF-8 data with bounds checking https://github.com/OpenTTD/OpenTTD/pull/13953#pullrequestreview-273960498412:06:25 <xarick> the instance must exist for GS too, I think12:17:54 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #13896: Fix #10179: [Script] Only run the Game::GameLoop() in-game https://github.com/OpenTTD/OpenTTD/pull/13896#issuecomment-277560557612:19:43 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #13906: Fix: [Script] Reset instance when changing running scripts in scenario editor https://github.com/OpenTTD/OpenTTD/pull/13906#pullrequestreview-273964758612:20:54 <_glx_> xarick: No, GS instance can safely be nullptr, it's properly checked.12:21:06 *** asymptotically2 has joined #openttd12:21:15 <peter1138> Yeah, there shouldn't be running scripts in scenario editor.12:21:48 <peter1138> They are incorrectly started after loading an existing scenario (or perhaps it needs to be a converted savegame.)12:22:52 <_glx_> Starting and running are different things. But for converted savegame it needs to be started to not discard saved data.12:23:43 <xarick> it won't crash in the debug window?12:23:49 <_glx_> I guess a saved scenario will also start GS on reload.12:24:09 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #13957: Fix #13955: Fixed graphs to respect RTL https://github.com/OpenTTD/OpenTTD/pull/13957#pullrequestreview-273965011612:24:38 <_glx_> xarick: No crash in debug window if it's invalidated, which my PR does.12:26:01 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #13956: Fix #13954: Plotting graphs with limited data to the right in case of wallclock mode https://github.com/OpenTTD/OpenTTD/pull/13956#pullrequestreview-273966405612:29:09 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #13906: Fix: [Script] Reset instance when changing running scripts in scenario editor https://github.com/OpenTTD/OpenTTD/pull/13906#pullrequestreview-273967300312:30:52 *** nebulabc has joined #openttd12:36:09 <DorpsGek> [OpenTTD/OpenTTD] xJayMorex updated pull request #13957: Fix #13955: Fixed graphs to respect RTL https://github.com/OpenTTD/OpenTTD/pull/1395712:39:47 <DorpsGek> [OpenTTD/OpenTTD] PeterN commented on pull request #13958: Codechange: Manage window viewport with std::unique_ptr. https://github.com/OpenTTD/OpenTTD/pull/13958#pullrequestreview-273970210612:42:39 <peter1138> Annoying. I have a headless pi that is no longer listening for SSH.12:51:39 <kuhnovic> Can somebody look at 13885 ? It might be nice to have it in the upcoming beta / release12:56:58 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #13953: Add Utf8View to simplify iterating over UTF-8 data with bounds checking https://github.com/OpenTTD/OpenTTD/pull/1395312:57:06 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #13953: Add Utf8View to simplify iterating over UTF-8 data with bounds checking https://github.com/OpenTTD/OpenTTD/pull/13953#pullrequestreview-273975334412:59:18 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #13953: Add Utf8View to simplify iterating over UTF-8 data with bounds checking https://github.com/OpenTTD/OpenTTD/pull/13953#pullrequestreview-273975964012:59:29 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #13953: Add Utf8View to simplify iterating over UTF-8 data with bounds checking https://github.com/OpenTTD/OpenTTD/pull/13953#pullrequestreview-273976024212:59:32 <DorpsGek> [OpenTTD/OpenTTD] xJayMorex updated pull request #13956: Fix #13954: Plotting graphs with limited data to the right in case of wallclock mode https://github.com/OpenTTD/OpenTTD/pull/1395613:06:59 <frosch123> Hmm, why are there no translations in emscripten deployments? I can only select English13:13:17 <frosch123> Found the explanation in the CMakeLists13:13:17 *** herms0 has joined #openttd13:15:46 *** TinoDidriksen has quit IRC (Remote host closed the connection)13:15:46 *** herms has quit IRC (Quit: Ping timeout (120 seconds))13:15:46 *** herms0 is now known as herms13:17:01 *** TinoDidriksen has joined #openttd13:17:41 *** TinoDidriksen is now known as Guest1284013:18:38 *** Guest12840 is now known as TinoDidriksen13:20:42 *** Webster has quit IRC (Ping timeout: 480 seconds)13:49:18 <xarick> wow... array insert is slo14:00:28 <andythenorth> peter1138: I assume you were talking to me there, about Horse compile times? π14:00:33 <andythenorth> π΄14:03:18 *** akimoto has quit IRC (Ping timeout: 480 seconds)14:05:45 *** Webster has joined #openttd14:06:32 <peter1138> Well.14:07:50 <xarick> this.random_numbers_array.insert(x, number); this is doing lots of resizes14:08:54 <truebrain> frosch123: Size π14:10:17 <frosch123> And missing fontconfig14:10:17 <frosch123> So I can't test RTL stuff anyway14:13:16 <truebrain> Nope14:25:13 <xarick> beta2 coming now?14:30:10 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #13957: Fix #13955: Fixed graphs to respect RTL https://github.com/OpenTTD/OpenTTD/pull/1395714:32:07 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #13957: Fix #13955: Fixed graphs to respect RTL https://github.com/OpenTTD/OpenTTD/pull/13957#pullrequestreview-274012047814:32:37 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #13957: Fix #13955: Fixed graphs to respect RTL https://github.com/OpenTTD/OpenTTD/pull/13957#pullrequestreview-274012265314:40:58 <xarick> AILists don't take many ops to save or load? interesting14:41:22 <DorpsGek> [OpenTTD/OpenTTD] frosch123 opened pull request #13959: Fix: Numbers were left-aligned for RTL languages in several windows. https://github.com/OpenTTD/OpenTTD/pull/1395914:58:02 *** keikoz has joined #openttd14:59:13 <peter1138> > Also, if I might make a suggestion, can you please disable passengers in cargo vehicles?14:59:16 <peter1138> Hh14:59:24 <peter1138> So there are NewGRFs that just allow anything :)14:59:39 <peter1138> > This was designed for players who like eyecandy,14:59:40 <peter1138> Uh huh.15:03:35 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #13953: Add Utf8View to simplify iterating over UTF-8 data with bounds checking https://github.com/OpenTTD/OpenTTD/pull/1395315:05:26 *** Wormnest has joined #openttd15:13:13 *** nielsm has joined #openttd15:20:43 <DorpsGek> [OpenTTD/OpenTTD] frosch123 approved pull request #13885: Change: Treat already built errors as success when building long road or rail https://github.com/OpenTTD/OpenTTD/pull/13885#pullrequestreview-274030516715:28:49 <_glx_> xarick: Not a reason to misuse them15:50:34 <xarick> could some of the squirrel functions be optimized somehow?15:50:49 <xarick> i know it's 3rd party code π¦15:51:02 <xarick> code style is different16:01:01 <_glx_> opcodes and squirrel source functions are unrelated16:03:12 <_glx_> the only thing responsible for opcode use is the code in .nut16:06:58 <xarick> can table keys be something complex...16:07:16 <xarick> probably not16:08:50 <_glx_> keys can be anything IIRC16:08:58 <_glx_> except null maybe16:09:25 <exceptik> Inb4 hashable null16:09:41 <xarick> `table.rawset([nice_array], [another_array]);`16:10:46 <_glx_> "values can be any possible type and keys any type except 'null'"16:11:05 <xarick> very interesting, an array as key16:11:22 <xarick> let's see if it works the way I think16:14:58 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #13953: Add Utf8View to simplify iterating over UTF-8 data with bounds checking https://github.com/OpenTTD/OpenTTD/pull/13953#pullrequestreview-274046679116:15:09 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1357387963738095626/image.png?ex=67f0058d&is=67eeb40d&hm=724103f0e12da0db246ea1897289a2895e66e4c2e45ace6ce928ad21b6b254f9&16:15:09 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1357387963985690855/image.png?ex=67f0058d&is=67eeb40d&hm=81318c11dc03290afa1326e9a5689e6caf7c2be312aca01b8dd3efe8ce7a44e8&16:15:09 <xarick> nope16:15:17 <xarick> how disappointing16:18:41 <_glx_> it's not the same array16:18:48 <_glx_> so not the same key16:19:17 <andythenorth> naptime?16:22:53 <xarick> how many cargo classes vs how many cargo types in the game16:23:17 <Rubidium> too16:24:39 <xarick> CargoType = uint8_t16:25:25 <xarick> CargoClass : uint8_t16:29:34 <_glx_> 16 cargo classes (it's 16bit bit mask)16:31:13 <_glx_> and 64 cargo types <https://newgrf-specs.tt-wiki.net/wiki/Action0/Cargos>16:39:05 *** Flygon has joined #openttd16:43:45 <DorpsGek> [OpenTTD/OpenTTD] frosch123 merged pull request #13953: Add Utf8View to simplify iterating over UTF-8 data with bounds checking https://github.com/OpenTTD/OpenTTD/pull/1395316:53:00 <xarick> TableListToAIList16:54:11 <xarick> Undoing this <https://github.com/SamuXarick/LuDiAI-AfterFix/blob/master/Utils.nut#L79-L91>17:02:41 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #13943: Remove C-style string parsing and composing https://github.com/OpenTTD/OpenTTD/pull/1394317:06:41 <andythenorth> hmm after much 'optimisation' Iron Horse 'refactor' branch compile time is now....17:06:47 <andythenorth> ....the same as the last tag on main branch17:06:48 <andythenorth> lol17:06:55 <andythenorth> "side quest"17:07:14 <andythenorth> and it has fewer vehicles17:11:56 <andythenorth> no I tested wrong somehow17:13:22 <andythenorth> 19 seconds vs 36 seconds from clean17:13:26 <andythenorth> less lol17:15:19 <xarick> shouldn't I be getting errors about CargoID or BridgeID?17:15:48 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1357403226458296391/image.png?ex=67f013c4&is=67eec244&hm=f5ca46073f8860e2d39aff0a39e6ee90dad2c8375421f9b8dfd0ad4232612d5b&17:15:48 <xarick> nvm spoke too early17:47:33 *** D-HUND has joined #openttd17:49:07 <andythenorth> hmm run out of optimisation ideas17:51:16 <andythenorth> nml generation is 4 s, nmlc step is 11 s, grfcodec is 3 s, process overhead is 1 s17:51:46 <andythenorth> there's some jitter, sometimes nml generation is only 2s and nmlc is only 10 s17:52:33 <andythenorth> I can save 0.3 s in nml generation by caching some infrequently changed nml files and using #include to bring them17:52:44 <andythenorth> but the file io might eat that saving17:54:17 <andythenorth> with a primed cache, nmlc -> grf is 9.1 s17:54:29 <andythenorth> but that's much slower than grfcodec when sprites have changed17:54:29 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1357412960380653748/image.png?ex=67f01cd5&is=67eecb55&hm=8aadc374c59efd463043a31ebc11c713d0f3a3b045c48c8b2087969d82156c6a&17:54:29 <xarick> AIList save and load working correctly!17:54:46 <xarick> used cities Pass / Mail use AIList now17:55:45 <andythenorth> hmm I could generate both grf and nfo, and use the nfo route when only sprites are changed17:56:10 <andythenorth> but that will be more complex17:59:17 <andythenorth> _glx_: could we cache the parse (and AST?) from a #include> π17:59:30 <andythenorth> "pickle is slow" though17:59:52 <_glx_> but how to handle the #defines ?18:00:29 <DorpsGek> [OpenTTD/OpenTTD] Kuhnovic commented on pull request #13885: Change: Treat already built errors as success when building long road or rail https://github.com/OpenTTD/OpenTTD/pull/13885#issuecomment-277654950618:00:54 <andythenorth> ah yes, defines π18:01:19 <andythenorth> no caching if define is used π18:02:04 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1357414870211104838/image.png?ex=67f01e9c&is=67eecd1c&hm=b34ca73182f083d42c163f739326560988d06311d431e014e265e79b6e15bd8c&18:02:04 <andythenorth> the parser and preprocessing for current Horse:18:03:22 <andythenorth> unrelated: the grfcodec build of iron-horse.grf is 38 MB, the nmlc build is 45 MB18:05:46 <andythenorth> also wonder why grfcodec is so slow π18:18:49 <Rubidium> maybe grfcodec tries harder to compress the images?18:19:16 <Rubidium> there are different strategies that can be used, so it might be trying all and chose the most efficient one18:24:05 * andythenorth looking what flags I use18:25:27 <andythenorth> might be because grfcodec has -c set in flags18:32:35 <andythenorth> looks like grfcodec is arm native18:32:41 <andythenorth> I guess it can't be multithreaded18:38:21 <frosch123> Nml and grfcodec compression is identical, they use the same code18:39:07 *** kuka_lie has joined #openttd18:39:21 <frosch123> Making sprite compression in nml multithreaded is easy, I had a patch for that 10 years ago18:42:40 *** tokai has joined #openttd18:42:40 *** ChanServ sets mode: +v tokai18:45:29 <_zephyris> Would help with extra zoom baseset build times...18:49:27 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)18:52:06 *** yiffgirl has joined #openttd18:52:06 <yiffgirl> is there anything i can do to help speed https://github.com/OpenTTD/nml/pull/358 along? it'd be a major quality of life upgrade18:58:05 <_glx_> andythenorth: Yeah if grfcodec crops but nml doesn't it can explain the difference19:02:30 <_zephyris> _glx_: Any thoughts or feelings about handling alternative sprites?19:02:50 <_zephyris> I've listed them as bugs, but that's kinda up for debate...19:03:40 <andythenorth> frosch123: was it an mp pool?19:05:50 <frosch123> No. Threadpool in single process. All the png loading and compression runs in c modules without GIL19:07:27 <andythenorth> π19:10:57 <jfkuayue> Why do all gay people type without capitalisations?19:10:57 <andythenorth> wonder why Horse is not creating .nmlcache19:12:39 <_glx_> should I <https://github.com/OpenTTD/OpenTTD/compare/master...glx22:OpenTTD:GS_strings> ?19:14:50 <DorpsGek> [OpenTTD/OpenTTD] glx22 merged pull request #13906: Fix: [Script] Reset instance when changing running scripts in scenario editor https://github.com/OpenTTD/OpenTTD/pull/1390619:15:44 <andythenorth> maybe .nmlcache is only created if --nfo isn't used19:16:41 <_glx_> the cache is only used when actually using the images, so for --grf19:17:37 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 merged pull request #13957: Fix #13955: Make graphs respect RTL https://github.com/OpenTTD/OpenTTD/pull/1395719:17:40 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 closed issue #13955: [Bug]: Plots do not respect right-to-left text direction https://github.com/OpenTTD/OpenTTD/issues/1395519:17:43 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 closed issue #13955: [Bug]: Plots do not respect right-to-left text direction https://github.com/OpenTTD/OpenTTD/issues/1395519:18:13 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #13959: Fix: Numbers were left-aligned for RTL languages in several windows. https://github.com/OpenTTD/OpenTTD/pull/13959#pullrequestreview-274091381319:20:24 <andythenorth> jfkuayue: you're going to get kicked19:20:39 <andythenorth> or you're going to have to explain you left your device unlocked19:30:02 *** Wolf01 has joined #openttd19:37:05 <jfkuayue> This does not mean the reverseβ¦19:49:25 <DorpsGek> [OpenTTD/OpenTTD] frosch123 opened pull request #13960: Codechange: Remove broken and unused StrMakeValidInPlace overload. https://github.com/OpenTTD/OpenTTD/pull/1396019:50:47 <yiffgirl> andythenorth: as a gay person this is extremely funny to me20:00:22 <DorpsGek> [OpenTTD/OpenTTD] frosch123 commented on pull request #13956: Fix #13954: Plotting graphs with limited data to the right in case of wallclock mode https://github.com/OpenTTD/OpenTTD/pull/13956#issuecomment-277679369920:01:12 <DorpsGek> [OpenTTD/OpenTTD] frosch123 updated pull request #13960: Codechange: Remove broken and unused StrMakeValidInPlace overload. https://github.com/OpenTTD/OpenTTD/pull/1396020:02:08 *** WormnestAndroid has quit IRC (Remote host closed the connection)20:02:08 *** WormnestAndroid has joined #openttd20:02:08 <xarick> TownManager is still a mess. I really wanted to make this fast20:05:20 <andythenorth> yiffgirl: itβs big lol eh π20:08:01 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 approved pull request #13960: Codechange: Remove broken and unused StrMakeValidInPlace overload. https://github.com/OpenTTD/OpenTTD/pull/13960#pullrequestreview-274101850120:16:16 <DorpsGek> [OpenTTD/OpenTTD] xJayMorex commented on pull request #13956: Fix #13954: Plotting graphs with limited data to the right in case of wallclock mode https://github.com/OpenTTD/OpenTTD/pull/13956#issuecomment-277682332420:22:17 <frosch123> Any opinions on #13954?20:22:17 <frosch123> Should time charts start on the left and put "now" somewhere in the middle? Or should they right-align, so "now" is at the border?20:23:24 <_zephyris> Are you worried about incomplete data, ie. early in the game?20:23:40 <frosch123> Xjay disagreed with me at first, then implemented it anyway, but only for wall clock mode. Now he disagrees again for calendar mode20:24:20 <frosch123> _zephyris: Yes, it only affects start of game.20:24:46 <frosch123> "Who cares"?20:25:08 <_zephyris> Yeah20:26:08 <_zephyris> FWIW I'd right align for time prior to now, and left align for time on an absolute date scale.20:27:33 *** kuka_lie has quit IRC (Quit: Lost terminal)20:27:40 <_zephyris> The conceptual different being 'relative to now' vs. 'here's an absolute date time scale which is invariant, and I'm plotting my data next to that'20:31:27 <_zephyris> Hmm, just saw the industry production vs. other graph inconsistency... That'll bug me20:31:50 <_zephyris> Mark me as conflicted20:37:42 *** nielsm has quit IRC (Ping timeout: 480 seconds)20:41:46 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1357455056777248988/screenshot81.png?ex=67f04409&is=67eef289&hm=d02f4bca364c2ec47c25b215050377b0d7f7baa13b490f966412f5652b8102fb&20:41:46 <xarick> https://cdn.discordapp.com/attachments/1008473233844097104/1357455057162993734/screenshot82.png?ex=67f04409&is=67eef289&hm=1f2f68d492c4a36bde3a236f8ca72a392c1994591e7d199db9b24c600709489f&20:43:03 <xarick> industry production should start at left20:46:32 <xarick> or just wait enough time and everything will look alright21:06:29 <xarick> Need to get rid of this array to something faster <https://github.com/SamuXarick/LuDiAI-AfterFix/blob/master/TownManager.nut#L40>21:06:34 <xarick> must think21:10:00 <DorpsGek> [OpenTTD/OpenTTD] glx22 opened pull request #13961: Change: [Script] Start GS (but don't run it) when generating world in scenario editor https://github.com/OpenTTD/OpenTTD/pull/1396121:27:06 *** keikoz has quit IRC (Ping timeout: 480 seconds)21:41:34 *** Wolf01 has quit IRC (Quit: Once again the world is quick to bury me.)22:05:49 *** peter1138[d] has quit IRC (Quit: User went offline on Discord a while ago)22:18:06 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick updated pull request #13665: Codechange: Code style changes to AI/GS GUI files https://github.com/OpenTTD/OpenTTD/pull/1366522:29:38 <__abigail> jfkuayue: Good morning! I hope you're having a wonderful day. I just wanted to inform you that some of us are, in fact, capable of capitalisation.23:29:07 <reldred> It's a bit of a generalization, yes.23:55:57 <jfkuayue> Sorry, I was just confusedcontinue to next day β΅