Times are UTC Toggle Colours
00:14:25 *** Lejving has quit IRC 00:22:21 *** Lejving has joined #openttd 00:48:28 <DorpsGek_III> [OpenTTD/nml] glx22 opened pull request #156: Fix: Procedure calls reference counting https://git.io/JfXTl 02:17:18 *** glx has quit IRC 02:40:58 *** Wormnest has quit IRC 02:41:52 *** D-HUND has joined #openttd 02:43:32 *** Flygon has joined #openttd 02:45:00 *** debdog has quit IRC 04:46:49 *** keoz has joined #openttd 06:03:59 *** nielsm has quit IRC 06:13:41 *** sla_ro|master has joined #openttd 06:28:31 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz opened pull request #8183: Codechange: Realign SDL driver with SDL2 driver to ease maintenance. https://git.io/JfX3D 06:30:54 <DorpsGek_III> [OpenTTD/OpenTTD] LordAro commented on pull request #8183: Codechange: Realign SDL driver with SDL2 driver to ease maintenance. https://git.io/JfX39 06:39:44 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz opened pull request #8184: Cleanup: More code comment and doxygen fixes. https://git.io/JfXst 06:41:50 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz commented on pull request #8183: Codechange: Realign SDL driver with SDL2 driver to ease maintenance. https://git.io/JfXsY 06:42:17 <DorpsGek_III> [OpenTTD/OpenTTD] LordAro commented on pull request #8184: Cleanup: More code comment and doxygen fixes. https://git.io/JfXsO 06:49:25 <DorpsGek_III> [OpenTTD/OpenTTD] danchr updated pull request #8182: Clang LTO https://git.io/JfPQ1 06:50:06 <DorpsGek_III> [OpenTTD/OpenTTD] danchr commented on pull request #8182: Clang LTO https://git.io/JfXsR 06:53:37 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz updated pull request #8183: Codechange: Realign SDL driver with SDL2 driver to ease maintenance. https://git.io/JfX3D 06:55:44 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz commented on pull request #8183: Codechange: Realign SDL driver with SDL2 driver to ease maintenance. https://git.io/JfXsP 07:00:30 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz commented on pull request #8184: Cleanup: More code comment and doxygen fixes. https://git.io/JfXsN 07:04:25 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz commented on pull request #8184: Cleanup: More code comment and doxygen fixes. https://git.io/JfXGq 07:13:55 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz updated pull request #8183: Codechange: Realign SDL driver with SDL2 driver to ease maintenance. https://git.io/JfX3D 07:42:45 *** andythenorth has joined #openttd 07:47:33 *** iSoSyS has joined #openttd 07:48:34 <DorpsGek_III> [OpenTTD/OpenTTD] LordAro approved pull request #8183: Codechange: Realign SDL driver with SDL2 driver to ease maintenance. https://git.io/JfXZV 07:53:29 <LordAro> new contributors 07:53:30 <LordAro> how lovely 07:57:21 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz commented on issue #8095: Feature suggestion: option to automatically space out vehicles in a shared order https://git.io/JfIUi 08:04:32 <DorpsGek_III> [OpenTTD/OpenTTD] andythenorth commented on pull request #8183: Codechange: Realign SDL driver with SDL2 driver to ease maintenance. https://git.io/JfXnq 08:05:24 <DorpsGek_III> [OpenTTD/OpenTTD] LordAro merged pull request #8183: Codechange: Realign SDL driver with SDL2 driver to ease maintenance. https://git.io/JfX3D 08:05:51 <DorpsGek_III> [OpenTTD/OpenTTD] LordAro approved pull request #8182: Clang LTO https://git.io/JfXnY 08:06:38 <DorpsGek_III> [OpenTTD/OpenTTD] LordAro approved pull request #8184: Cleanup: More code comment and doxygen fixes. https://git.io/JfXn3 08:16:53 *** gelignite has joined #openttd 08:41:13 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz commented on issue #7920: Purchase land tool lacks by-area https://git.io/JveBj 08:59:44 <DorpsGek_III> [OpenTTD/OpenTTD] nielsmh commented on pull request #8184: Cleanup: More code comment and doxygen fixes. https://git.io/JfXcA 09:02:17 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz commented on pull request #8183: Codechange: Realign SDL driver with SDL2 driver to ease maintenance. https://git.io/JfXCe 09:11:59 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz commented on pull request #8184: Cleanup: More code comment and doxygen fixes. https://git.io/JfXC4 09:22:06 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz dismissed a review for pull request #8184: Cleanup: More code comment and doxygen fixes. https://git.io/JfXn3 09:22:06 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz updated pull request #8184: Cleanup: More code comment and doxygen fixes. https://git.io/JfXst 09:28:20 *** iSoSyS has quit IRC 09:32:55 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz updated pull request #8184: Cleanup: More code comment and doxygen fixes. https://git.io/JfXst 09:36:26 *** cawfee has joined #openttd 09:36:29 <cawfee> hi 09:37:44 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz updated pull request #8184: Cleanup: More code comment and doxygen fixes. https://git.io/JfXst 09:40:34 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz updated pull request #8184: Cleanup: More code comment and doxygen fixes. https://git.io/JfXst 09:43:10 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz commented on pull request #8184: Cleanup: More code comment and doxygen fixes. https://git.io/JfXWQ 09:47:41 *** WormnestAndroid has quit IRC 09:48:03 *** WormnestAndroid has joined #openttd 09:51:03 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz updated pull request #8184: Cleanup: More code comment and doxygen fixes. https://git.io/JfXst 09:53:51 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz opened pull request #8185: Cleanup: StationCargoList::AreMergable doxygen comment references Veh… https://git.io/JfXlC 10:11:32 *** gelignite has quit IRC 10:11:44 *** _dp_ has quit IRC 10:13:23 *** dP has joined #openttd 10:13:23 *** dP is now known as _dp_ 10:15:54 *** dP has joined #openttd 10:15:54 *** dP is now known as _dp_ 10:16:51 *** dP has joined #openttd 10:16:51 *** dP is now known as _dp_ 10:23:21 *** johanna[m] has joined #openttd 10:27:52 *** cHawk has quit IRC 10:55:05 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz updated pull request #8185: Cleanup: StationCargoList::AreMergable doxygen comment references Veh… https://git.io/JfXlC 10:57:16 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz updated pull request #8185: Cleanup: StationCargoList::AreMergable doxygen comment references Veh… https://git.io/JfXlC 11:01:51 *** Borg has joined #openttd 11:02:05 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz updated pull request #8185: Cleanup: StationCargoList::AreMergable doxygen comment references Veh… https://git.io/JfXlC 11:02:23 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz updated pull request #8185: Cleanup: StationCargoList::AreMergable doxygen comment references Veh… https://git.io/JfXlC 11:07:55 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz updated pull request #8185: Cleanup: StationCargoList::AreMergable doxygen comment references Veh… https://git.io/JfXlC 11:25:34 <FLHerne> cawfee: Hello 11:25:41 *** cHawk has joined #openttd 11:25:58 <FLHerne> LordAro: Pity about the spam though :p 11:26:55 *** Samu has joined #openttd 11:30:24 <LordAro> FLHerne: shhh! 12:01:11 <Samu> hi 12:02:23 <cawfee> Hey FLHerne 12:06:58 *** gelignite has joined #openttd 12:20:29 <Borg> why in YAPF there is waiting for cheaper path... when more expensive one is clear to go? what is rationale for this? 12:30:24 <Guest26686> occopied track does not make the path unavailable, just slightly more expensive. Thus occupied vs detour is a trade-off which flips for a length of detour in favour of one or the other 12:31:00 <andythenorth> hmm 12:31:05 <andythenorth> can't change the GS on a running game 12:31:12 * andythenorth wonders why not 12:32:50 <Guest26686> because reasoning similar to NewGRF iirc? 12:33:39 <andythenorth> GS can't store anything in the map afaik 12:33:43 <andythenorth> unlike newgrf 12:34:09 <andythenorth> all GS data has to be serialised / deserialised on saveload 12:34:18 <andythenorth> (is why I assume that) 12:34:37 <andythenorth> seems to me that swapping the GS out should be trivial 12:34:52 <andythenorth> 'seems to me' are the dangerous words of the ill-informed :P 12:37:54 <FLHerne> Borg: A path being occupied is just a cost like the others 12:37:57 *** Guest26686 is now known as planetmaker 12:38:24 <FLHerne> It'll take a longer path if the shortest one is occupied, up to a point 12:39:01 <FLHerne> If that cost was too high, you'd have trains veering off on enormous detours instead of waiting a couple of seconds 12:40:36 <planetmaker> and the point to see there is: the path finder does not know anything about how permanent or temporary an obstacle is 12:57:01 <cawfee> can someone explain me, why i cant buy any train? new train, i see only trailer 12:58:16 <Borg> FLHerne: can I unflence that cost somehow? is there a setting? 12:59:06 <Borg> FLHerne: thats is preffered... go make detour.. instead block a whole queue.. 13:01:45 <LordAro> cawfee: what date is it in game? trains expire in later game (wagons don't because of historical reasons) 13:02:07 <LordAro> Borg: there are lots of hidden pathfinder settings in the config file 13:05:03 <cawfee> LordAro: Year 2053 :D 13:05:37 <LordAro> cawfee: yeah, that's a bit late for standard trains :) 13:05:49 <LordAro> monorail & maglev might still be available, i can't recall 13:06:44 <cawfee> Argh! yes i see now ive found the electric rails 13:07:41 <Borg> okey, lets do RTFS then again.. 13:12:45 <Borg> okey.. for stations.. it might be rail_lastred_penalty 13:15:06 <Borg> for normal tracks.. it might be rail_pbs_cross_penalty 13:15:07 <Borg> lets see 13:28:19 <Borg> grr 13:28:25 <Borg> still doing stupid thing ;) 13:29:39 <Borg> okey.. rail_pbs_cross_penlaty 1000 13:29:47 <Borg> does it job... I wonder if there will be some side effects 13:35:26 <Borg> so far. seems to works.. 13:36:27 *** nielsm has joined #openttd 13:39:05 <Borg> okey.. this is nice.. thet stopped doing stupid things.. like waiting for path to clear to empty platform.. where they have other platform empty (higher cost).. but path is clear.. 13:39:13 <Borg> now they just happily go there.. 13:45:41 <LordAro> Borg: that sounds suspiciously like you'd fiddled with your settings previously 13:45:49 <LordAro> trains definitely go to free platforms normally 13:46:16 <planetmaker> or... incompatible track types somewhere like a missing wire 13:46:42 <LordAro> planetmaker: presumably not if changing pathfinder settings changes behaviour 13:47:02 <planetmaker> you're right there :) I only read-back 2 lines :P 13:49:17 <Borg> LordAro: of course I fiddled w/ settings previously.. and im optimizing stuff even futher... 13:49:33 <Borg> I use PBS back singal as priority.. to have primary and auxialiary paths 13:49:57 <Borg> and stuff works very good... I just noticed.. that near stations trains sometimes were doing kinda stupid thing.. like waiting. 13:50:14 <LordAro> i'd highly recommend resetting your pathfinder settings to defaults then 13:50:24 <Borg> never! 13:50:26 <Borg> :D 13:50:51 <FLHerne> Borg: "waiting for path to clear to empty platform.. where they have other platform empty" sounds like it might be a different problem 13:50:52 <LordAro> right. 13:50:55 <Borg> I will play w/ new settings now.. few days. to see if I will spot some oddities.. for now.. none spotted 13:51:08 <FLHerne> https://github.com/OpenTTD/OpenTTD/issues/7941 13:51:13 <Borg> FLHerne: maybe I will do a screen shot.. to show setup ok? hold on a few secs 13:52:23 <Borg> http://node.borg.uu3.net/Industry Rails,%202122-11-19.png 13:52:28 <FLHerne> Borg: Basically, if a train has cleared the end of a platform, but is still in the block, the pathfinder sees that platform as "clear" even if it can't be signalled into it 13:52:42 <Borg> as you can see... 4th platform is shared one.. w/ PBS back signals 13:52:43 <FLHerne> s/a train/a train in front/ 13:53:55 <FLHerne> IME, that setup should work with the default pf settings 13:53:57 <Borg> now.. when first platform was clear.. and there was train on intersection moving into 2nd platform.. and I had train as well coming from 2nd track to 4th platform.. it just stopped there.. waiting for first train to clear path to 1st station.. instead moving to 4th platform.. with have cost +1500 compared to first 13:54:52 <Borg> FLHerne: it didnt.. because AFAIR.. with default settings.. PBS back signal penalty was too high.. and alt paths were basicaly never selected.. or very rary.. 13:55:03 <Borg> I use PBS back signals on tracks as well 13:56:19 <Borg> I want smooth balancing.. but if both tracks are empty.. primary needs to be always choosed... not aux.. 13:56:34 <FLHerne> That's odd, I use bidi PBS signals in similar situations all the time, and I very rarely see issues like that 13:57:05 *** nielsm has quit IRC 13:57:07 <Borg> FLHerne: on stations? or on regular paths as well? 13:57:09 <Borg> like this 13:57:16 <FLHerne> Both 13:57:41 *** cawfee has quit IRC 13:58:03 <Borg> http://node.borg.uu3.net/Industry Rails,%202122-11-19#1.png 13:58:18 <planetmaker> FLHerne, he changed the non-UI pathfinder settings... so normal experience logic might not apply at all 13:58:27 <Borg> 2 long bridges ... a bit up. there is back pbs signal 13:58:43 <FLHerne> 404 13:58:51 <Borg> hmm wtf 13:59:15 <Borg> ahh that #1 tag.. grr 13:59:37 <Borg> http://node.borg.uu3.net/Industry Rails,%202122-11-19_1.png 14:00:16 <Borg> how I can reset stuff? manualy? adjust all penalities to default? 14:00:26 <planetmaker> yes 14:00:41 <Borg> hmmm.. 14:01:20 *** Speeder_ has joined #openttd 14:01:36 <Borg> can anyone give me... p0,p1,p2, pbs_signal_back defaults? 14:01:47 <FLHerne> Or openttd -c test.cfg 14:01:52 <LordAro> just delete them from your config 14:01:55 <LordAro> and restart 14:02:02 <planetmaker> quit openttd. move openttd.cfg to a different name. start openttd and quit again 14:02:04 <Borg> arent that settings stored in save? 14:02:08 <planetmaker> now oyu have your old and a default 14:02:28 <LordAro> there you go, 3 separate solutions :) 14:02:31 <planetmaker> they are. But the log only has the changes wrt when it was created. Which is a list of length 0 14:02:36 *** iSoSyS has joined #openttd 14:02:45 <Borg> lets see if I have backups .ini.. 14:04:33 *** Speeder has quit IRC 14:07:22 <Borg> okey done 14:07:26 <Borg> lets see what happens 14:09:10 <Borg> yep... they do stupid things at station again 14:09:47 <Borg> but its ok? whats the problem? the settings there are here to tweak.. right? 14:09:57 <Borg> maybe 95% of players are happy w/ them.. 14:10:44 <Borg> im supprised tho.. that the track w/ PBS back signal is selected tho.. but maybe because I already had tweaked settings months ago. when I was starting experimenting w/ PBS back signal 14:11:40 <LordAro> i'd say 99.9% of players are happy with them 14:11:49 <LordAro> at least. 14:13:28 <Borg> okey.. back to my settings then.. 14:16:41 *** keoz has quit IRC 14:20:32 *** happpy has joined #openttd 14:21:04 <happpy> hi i got a question about the new openttd is this the writing irc channel? 14:21:30 <planetmaker> openttd does not support natively to write anywhere else than ingame chat 14:21:55 <happpy> hnn 14:21:59 <happpy> hmm 14:22:02 <LordAro> planetmaker: i think that was supposed to be 'right' 14:22:18 <LordAro> happpy: this is the channel for talking about OpenTTD things, yes 14:22:20 <LordAro> see the topic 14:22:36 <happpy> y has the new openttd not fixs the problem on mac 14:23:16 <LordAro> is this the problem on 10.12 ? 14:23:34 <happpy> yes 14:23:58 <LordAro> basically, because no one here has a machine old enough to reproduce the issue 14:24:16 <LordAro> and as yet we've not had any offers of help from people who know how to debug such a weird issue 14:24:20 *** cHawk has quit IRC 14:24:20 <planetmaker> he, ok. I totally mis-understood the question. Sorry. Yes, this is the channel to talk about OpenTTD. Of course 14:24:44 <LordAro> happpy: as far as i know, you should be able to to upgrade to 10.13 (or newer) with relative ease 14:24:47 <LordAro> which works 14:24:51 <Borg> happpy: is openttd 1.8.0 works better? compared to newer versions? 14:25:20 <FLHerne> Borg: Everything up to 1.10.0-RC1 should be fine 14:25:39 <LordAro> Borg: this is not the speed issue 14:25:42 <happpy> its not for me a player on openttdcoop discord got mac problem 14:26:53 <happpy> https://user-images.githubusercontent.com/7754310/78616142-c6a05a00-78b6-11ea-80c5-a2fd2727254a.png 14:26:55 <Borg> LordAro: I dunno what the issue is at all :) just curious.... because I saw forums posts about it.. 14:27:15 <Borg> I for example.. still on 1.8.0 and im happy with it... my choice.. 14:27:31 <happpy> chek the weblink 14:27:37 <happpy> i just dun 14:27:49 <happpy> thats the mac problem 14:28:11 <LordAro> happpy: http://github.com/OpenTTD/OpenTTD/issues/8066 14:28:22 <LordAro> see previous as to why it's not been fixed 14:28:53 <LordAro> wait, that's where you got the screenshot 14:29:06 <happpy> yeah 14:29:08 *** cHawk has joined #openttd 14:29:44 <happpy> IvanLeshev used a mac to play openttd but he can not 14:30:29 <happpy> becomes that p 14:30:31 <happpy> problem 14:30:32 <LordAro> happpy: options 1) learn deep knowledge of MacOS' internals, and fix the issue for us (we can't, and have little desire to support an out of support OS) 2) tell the user to upgrade to 10.13+ 3) Play on 1.10.0-RC1 or below 14:30:54 <LordAro> i'm not sure there's any more to discuss here 14:30:56 <andythenorth> mac support is current -2 yes/no? 14:30:59 <Borg> :> 14:31:13 <LordAro> Borg: the speed issue was fixed, and as long as you're not on 10.12, it should run as expected 14:31:23 <LordAro> andythenorth: in theory we still support 10.9 in the code 14:31:29 <LordAro> but... 14:31:51 <LordAro> nielsm's laptop is "only" 10.13 14:32:05 <Borg> LordAro: Im on Windows :) so.... its not my stuff.. I just dont get why people always want to newest... bestest... etc.. especially.. when they use outdated SW/HW.. 14:32:26 <Borg> just use whatever works.. or as you said.. learn stuff. and send patches.. 14:32:42 <LordAro> Borg: well you're just being stubborn for no reason then 14:32:45 <LordAro> (surprising no one) 14:32:48 <Borg> ;D 14:32:53 <Borg> I know I know.. its just me.. 14:33:12 *** arikover has joined #openttd 14:33:16 *** cHawk has quit IRC 14:33:38 <happpy> but if you had the bug log about the mac problem 14:33:46 <happpy> can it be fixs 14:35:58 <LordAro> happpy: probably not, the crash log hasn't been helpful in this case 14:36:11 <happpy> hmm 14:36:20 <happpy> its a sham 14:37:26 <happpy> yeah its a wird bug 14:37:34 *** happpy has left #openttd 14:37:48 *** happpy has joined #openttd 14:38:04 <happpy> ok will i hop its get fixs soon 14:38:15 <happpy> thsnks for the info 14:38:22 <happpy> have a nice day 14:38:24 <happpy> bb 14:38:27 <LordAro> you too! 14:38:34 *** happpy has left #openttd 14:39:16 *** heffer_ has quit IRC 14:47:02 *** cHawk has joined #openttd 15:00:01 *** glx has joined #openttd 15:00:01 *** ChanServ sets mode: +v glx 15:11:41 *** Wormnest has joined #openttd 15:17:28 *** heffer has joined #openttd 15:27:48 *** cHawk has quit IRC 15:28:04 *** jback has quit IRC 15:29:03 *** jback has joined #openttd 15:35:21 *** iSoSyS has quit IRC 15:49:08 *** keoz has joined #openttd 15:50:40 *** gelignite has quit IRC 16:09:53 *** virtualrandomnumber has joined #openttd 16:13:48 *** Flygon has quit IRC 16:54:04 *** happpy has joined #openttd 16:55:18 <happpy> a player on discord had a wired bug https://cdn.discordapp.com/attachments/710992141769244804/718138772063256626/unknown.png 16:56:16 <_dp_> happpy, it's not a bug, it's a debug feature 16:56:51 <happpy> ar ok 16:57:02 <_dp_> can be disabled somewhere in help menu iirc 16:57:24 <DorpsGek_III> [OpenTTD/OpenTTD] glx22 dismissed a review for pull request #7270: Introduce CMake (and removing all other project-related code) https://git.io/JfPbN 16:57:24 <DorpsGek_III> [OpenTTD/OpenTTD] glx22 updated pull request #7270: Introduce CMake (and removing all other project-related code) https://git.io/fhbqc 16:58:19 <happpy> i see 16:59:18 <happpy> will thinks of the info 16:59:20 <happpy> bb 16:59:49 <DorpsGek_III> [OpenTTD/OpenTTD] glx22 commented on pull request #7270: Introduce CMake (and removing all other project-related code) https://git.io/JfXi4 17:00:34 *** happpy has left #openttd 17:04:18 *** mindlesstux6662 has quit IRC 17:04:42 *** mindlesstux6662 has joined #openttd 17:06:09 *** Wolf01 has joined #openttd 17:09:22 *** k-man has quit IRC 17:10:01 *** k-man has joined #openttd 17:11:14 <andythenorth> hmm 17:11:16 <andythenorth> is cat? 17:13:09 *** iSoSyS has joined #openttd 17:16:27 <supermop_Home_> yo 17:21:09 <andythenorth> is curfew? 17:23:33 <Wolf01> Is depression? 17:23:43 <andythenorth> is mania? 17:23:51 <andythenorth> supermop_Home_ did you have time to break Horse yet? 17:24:17 <glx> andythenorth: #156 should fix your issue 17:29:36 *** frosch123 has joined #openttd 17:32:14 <andythenorth> glx thanks, seems to work :) 17:32:16 *** Afdal has joined #openttd 17:32:56 <Afdal> Hey guise, a thought occurs to me: has anyone ever considered making some tweaks to the company rating system to making it more interesting and competitive? 17:33:12 <Afdal> I'm a little surprised that someone hasn't made a patch 17:33:14 <Afdal> so it seems 17:33:34 <glx> use a game script 17:33:50 <DorpsGek_III> [OpenTTD/nml] andythenorth commented on pull request #156: Fix: Procedure calls reference counting https://git.io/JfXPD 17:34:24 <Afdal> would a game script mean you could simply make it as a newgrf 17:34:27 <andythenorth> glx probably needs a review from someone smarter than me? 17:35:24 <FLHerne> Afdal: Gamescripts and NewGRFs are different things 17:35:37 <FLHerne> You can rate companies with a GS, but not a grf 17:36:00 <Afdal> Why do I feel so unfamiliar with game scripts 17:36:04 <Afdal> is this a relatively new feature? 17:36:15 <FLHerne> 1.2 or something? 17:36:24 <FLHerne> So, relative to grfs, but not particularly 17:36:25 <Afdal> hmm 17:36:49 <andythenorth> so should I release new Iron Horse? 17:37:07 <FLHerne> It's basically the same Squirrel interface as AIs, but not attached to a company 17:37:25 <andythenorth> it has this mimick of var 41 with 24448 checks of var 61 per vehicle 17:37:39 <FLHerne> And it can create goals, pay companies for achieving targets, etc. 17:37:43 <andythenorth> frosch123 ^ is 25k checks of var 61 per vehicle unwise in graphics chain? 17:38:07 <supermop_Home_> andy i got distracted building a steel chain to fund my emus 17:38:08 <andythenorth> doesn't seem to have any noticeable performance impact in the mac client 17:38:51 <FLHerne> TrueBrain: https://bananas.openttd.org/en/gslibrary/ is linked from the wiki, but no longer exists 17:39:13 <glx> you can fix the wiki ;) 17:39:27 <FLHerne> Afdal: List here https://bananas.openttd.org/package/game-script 17:39:35 <FLHerne> (with links to forum threads) 17:39:36 <Afdal> Yeah I'm lookin at them now 17:39:40 <Afdal> I should use game scripts more often 17:40:10 <supermop_Home_> then i started making a screen for t shirts 17:40:15 <frosch123> andythenorth: ofc it is unwise. how many variables does a normal vehicle use? newgrf have a significant impact on performance. by making the switch more complex, you lower the total amount of usable vehicles in game 17:41:18 <andythenorth> I can reduce it by a factor of 16, if I can figure out the syntax for STORE_TEMP with var 61 result 17:41:25 <andythenorth> currently I have failed to make that work 17:42:01 <andythenorth> I can reduce it by factor 7 by rewriting my code to cache some results earlier in the chain 17:42:04 <glx> STORE_TEMP(expression, reg) should work 17:42:26 * andythenorth tries again 17:47:32 <glx> andythenorth: https://github.com/OpenTTD/nml/blob/1d67cceaf140c8b9431bb63d86f62290331ebb86/examples/object/example_object.nml#L110-L111 <-- it's even use in examples :) 17:48:17 <andythenorth> I was wearing clown shoes 17:48:30 <andythenorth> I was attempting to use STORE_TEMP([STORE_TEMP(-1, 0x10F), var[0x61, 0, 0x0000FFFF, 0xC6]], 0) 17:48:57 <andythenorth> [STORE_TEMP(-1, 0x10F), STORE_TEMP(var[0x61, 0, 0x0000FFFF, 0xC6], 0)] should be fine 17:49:16 <glx> I think both work in theory 17:49:41 <andythenorth> hmm, compiles, but wrong results 17:49:44 <andythenorth> probably my mistake 17:50:32 *** virtualrandomnumber has quit IRC 17:51:26 <glx> anyway you can always check the nfo to ensure it's correct 17:52:18 <andythenorth> that's nice trolling :) 17:52:25 <andythenorth> I haven't read that much raw nfo for some years :) 17:53:26 <glx> but nml usually doesn't put enough comments so it's not easy to check the output 17:55:16 <andythenorth> hmm 17:55:37 <glx> anyway reading an action 2 is not that hard 17:56:03 <andythenorth> does STORE_TEMP() return any value in any circumstance to an expression? (I'm not expecting it to) 17:56:56 <glx> it's a /2s so 0E s or sto [1] var7D[val2] = val1, result = val1 17:57:55 <glx> so STORE_TEMP(1,x)+1 stores 1 and return 2 17:58:02 <andythenorth> oof 17:58:56 <glx> but I think [] syntax uses r for each element 17:59:11 <glx> discarding the previous result 18:00:49 *** cHawk has joined #openttd 18:03:32 <andythenorth> I expect this to compare the value of register 0 16 times https://gist.githubusercontent.com/andythenorth/3d55388eced3314e1f4e386f11bf17b4/raw/ff50931b81346d7131ca7e9c4e07c251fd2687bc/gistfile1.txt 18:03:50 <andythenorth> and I expect register 0 to contain the value of var 61 for a specific vehicle offset (last switch) 18:04:43 * andythenorth wonders how to debug temp registers :P 18:04:50 <andythenorth> perm storage in industries is readable 18:06:15 <andythenorth> this works, but reads var 61 16 times https://gist.githubusercontent.com/andythenorth/a1930beaa10ea90dd4718eae0c49b956/raw/5191ffbf82887fe7d4115f8106a708776a87d069/gistfile1.txt 18:17:34 <supermop_Home_> andythenorth curfew isn't til 20:00 18:28:50 *** ChanServ sets mode: +o planetmaker 18:28:50 *** ChanServ sets mode: +v orudge 18:53:26 *** Borg has quit IRC 19:10:22 <glx> andythenorth: what doesn't work ? 19:11:11 <glx> because it looks like they should do the same 19:13:04 *** Progman has joined #openttd 19:15:14 <andythenorth> it does to me 19:15:21 <andythenorth> they're returning different results 19:15:34 <andythenorth> there's no trivial way to inspect the registers as far as I know? 19:15:53 <andythenorth> is there some stupid oversight in my nml? 19:16:23 <glx> well switch_alt_var_41_position_in_vehid_chain_multiple_ids_127 always return 126 in both case I think 19:16:55 <andythenorth> yes 19:17:05 <andythenorth> there's a 126 other similar switches before that :P 19:17:15 <andythenorth> all the same in both versions 19:17:50 <glx> and all the == are useless to me 19:18:09 <glx> oh no they are 19:18:15 <glx> registers 19:19:12 <andythenorth> maybe register 0 is over-written by something else 19:20:22 <andythenorth> reading a register doesn't clear it afaik 19:20:31 *** arikover` has joined #openttd 19:22:29 <andythenorth> quite puzzling :) 19:22:36 <glx> switch_alt_var_41_match_ids_15 overwrites register 0 19:22:42 <glx> when false 19:23:54 <glx> ah no I clearly don't follow your code 19:24:25 <frosch123> is there a paste with the nfo? 19:25:05 <glx> yeah the nfo would help a lot 19:25:39 <glx> because nml wise they do the same 19:25:50 *** arikover has quit IRC 19:26:36 <glx> anyway to me switch_alt_var_41_match_ids() returns 0 or 1, but the result is not used 19:27:09 <glx> same in both versions 19:29:33 <glx> so basically a waste of cpu just to return 126 anyway ;) 19:36:41 <andythenorth> I considered optimising away the last switch in the chain 19:37:23 <glx> last switch could have a single "return;" 19:37:49 <glx> but I don't see what all the checks are for 19:38:02 <andythenorth> hang on I'll paste more 19:38:05 <glx> if you just ignore the result 19:38:55 <andythenorth> https://gist.githubusercontent.com/andythenorth/2b8346f6bf9330acc947d6ecd9bba447/raw/827763d65ee1bd6248d230e4a3ce9cae48a4ce15/gistfile1.txt 19:40:12 <glx> ha yes it makes more sense :) 19:40:36 <andythenorth> to be clear also, this code works if I read var 61 16 times 19:40:47 <andythenorth> just not when I try to make it more efficient at run time :) 19:42:07 <glx> so yes nfo for switch_alt_var_41_match_ids may help 19:42:43 <glx> because that's the probable failing point if you get a different result 19:42:51 <andythenorth> oh the nfo is commented :D 19:42:58 <andythenorth> that helps 19:43:17 <andythenorth> failing version https://dev.openttdcoop.org/attachments/download/9692/iron-horse-store-var-61.nfo 19:43:25 <andythenorth> working version https://dev.openttdcoop.org/attachments/download/9693/iron-horse.nfo 19:43:27 <glx> it's not fully commented but at least you can find the switches easily :) 19:43:35 <andythenorth> switch_alt_var_41_match_ids_15 19:44:36 <andythenorth> wow I can almost read this nfo 19:44:38 <andythenorth> it's been years 19:44:50 <andythenorth> in some ways nfo is much easier than nml 19:46:55 <DorpsGek_III> [OpenTTD/OpenTTD] 2TallTyler commented on issue #7920: Purchase land tool lacks by-area https://git.io/JveBj 19:47:19 <andythenorth> why is switch_alt_var_41_match_ids_0 doing sto 19:47:39 <glx> that's nml stuff 19:47:47 <andythenorth> rad 19:50:18 <glx> it's load(1), store in 82, load(0), compare with 82 19:51:27 <glx> & 1 19:51:56 *** nielsm has joined #openttd 19:52:03 <glx> nfo seems correct 19:57:08 <andythenorth> I've tried it with different register numbers that are definitely unused elsewhere 19:57:34 <andythenorth> in case this code works, but trashes a register needed by some other switch 19:58:11 <frosch123> 1 + switch_alt_var_41_position_in_vehid_chain_multiple_ids() + switch_alt_var_41_position_in_vehid_chain_from_end_multiple_ids(); <- does the evaluation order matter there? 19:58:29 <frosch123> nml reverses them so, if the second procedure relies on registers from the first, it fails 19:59:19 <andythenorth> they _shouldn't_ rely on each other, although that fact is also worth knowing 20:00:40 <andythenorth> does LOAD_TEMP ever need a mask or shift? 20:01:00 <frosch123> i think register 82 is destroyed 20:01:03 <glx> no it just reads the value as it's stored 20:01:15 *** iSoSyS has quit IRC 20:01:28 <frosch123> the 1 + proc + proc uses it 20:01:33 <glx> reg 82 is temporary to each switch_alt_var_41_match_ids_XX 20:01:34 <frosch123> but at least one of the procs also uses it 20:02:29 <frosch123> so the result of ..._from_end... is lost? 20:02:29 <glx> and I think #156 properly notice it 20:03:05 <andythenorth> hmm I have a new interesting result 20:03:07 <andythenorth> verifying 20:03:30 <andythenorth> this might be compound issues 20:03:36 <glx> are you using #156 andythenorth ? 20:03:40 <andythenorth> yes 20:03:58 <andythenorth> ha ha ok I've got a _partial_ repro 20:04:04 <glx> then I need to fix stuff 20:04:09 <andythenorth> there are 16 registers, to hold up to 16 vehicle IDs 20:04:47 <andythenorth> to prevent unwanted ID collisions I set -1 to any spare registers that aren't actually required to hold a vehicle ID 20:05:10 <glx> I think at some point a reference is not marked as proc 20:06:07 <andythenorth> changing the -1 to a positive value gets a less broken result 20:06:11 <andythenorth> still broken, but less broken 20:06:30 <glx> the issue is reg 82 overwrite 20:06:30 <andythenorth> so comparing var 61 to -1 works 20:06:55 <andythenorth> ok so the -1 thing is just some side effect? 20:08:57 <glx> (switch_alt_var_41_position_in_vehid_chain_multiple_ids + 1) + switch_alt_var_41_position_in_vehid_chain_from_end_multiple_ids is basically (switch_alt_var_41_position_in_vehid_chain_multiple_ids + 1) + last_checked_id 20:09:10 <andythenorth> is this something that _should_ work, or am I just pushing nml too far? :) 20:09:16 <glx> because 82 is overwritten 20:09:30 <glx> it should work, but there's a bug in nml 20:09:43 <andythenorth> well this is fun 20:09:45 <glx> switch_alt_var_41_match_ids_XX should not reuse 82 20:10:03 <glx> some references are just improperly marked 20:10:45 <andythenorth> this silly feature has been quite useful for finding bugs :) 20:10:55 <andythenorth> also I can test procedure params with it 20:11:30 <glx> params are broken for now because #156 is not included yet :) 20:11:38 <andythenorth> yup :) 20:12:06 <andythenorth> procedures are proving so useful 20:12:22 <andythenorth> takes a bit of thinking about what needs putting in registers 20:12:25 <glx> but I'll try your reduced test case gist, it should show the bug 20:12:35 <DorpsGek_III> [OpenTTD/OpenTTD] nielsmh approved pull request #7270: Introduce CMake (and removing all other project-related code) https://git.io/JfX9v 20:12:54 <andythenorth> store_temp_var_61_result branch in Iron Horse also 20:13:40 <andythenorth> once params are in, I can use a stored procedure to set all the registers for a chain in one go, without repeating code per vehicle. So helpful 20:15:10 <glx> oh of course trying to plug the test case in FEAT_INDUSTRY won't work 20:15:15 <glx> silly me 20:15:18 <andythenorth> :) 20:15:26 <andythenorth> hmm if we had vehicle perm storage, I could run this switch chain once in depot and store the result 20:15:54 <andythenorth> if we had vehicle perm storage, and I could read it with var 61, the switch chain might be even simpler :P 20:18:07 <andythenorth> or I could just use bitmask_vehicle_info, how big is it? 256 unique values? 20:19:20 <glx> ok confirmed the issue, the register is reused while it should not 20:19:32 <andythenorth> \o/ 20:54:44 *** frosch123 has quit IRC 20:57:46 <TrueBrain> https://github.com/sickcodes/Docker-OSX 20:57:55 <TrueBrain> Maybe useful? 21:12:26 *** Wolf01 has quit IRC 21:16:50 *** sla_ro|master has quit IRC 21:25:59 *** gelignite has joined #openttd 21:40:42 *** Samu has quit IRC 21:40:50 *** Progman has quit IRC 21:52:52 *** nielsm has quit IRC 22:00:50 <supermop_Home_> well all my screen filler had congealed so i couldn't print a t shirt 22:01:10 <supermop_Home_> and ended up going to a protest in an aloha shirt 22:09:19 *** andythenorth has quit IRC 22:13:56 *** arikover` has quit IRC 22:18:47 *** cHawk has quit IRC 22:19:09 *** cHawk has joined #openttd 22:26:09 <DorpsGek_III> [OpenTTD/nml] glx22 updated pull request #156: Fix: Procedure calls reference counting https://git.io/JfXTl 22:37:44 *** colde has quit IRC 22:38:09 *** dvim has quit IRC 22:39:24 *** wsc has quit IRC 22:39:57 *** innocenat_ has quit IRC 22:42:48 *** dvim has joined #openttd 22:51:16 *** colde has joined #openttd 23:06:10 *** keoz has quit IRC 23:14:14 *** wsc has joined #openttd