Config
Log for #openttd on 4th June 2020:
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

Powered by YARRSTE version: svn-trunk