Back to Home / #openttd / 2016 / 05 / Prev Day | Next Day
#openttd IRC Logs for 2016-05-26

---Logopened Thu May 26 00:00:31 2016
00:01-!-ToneKnee [~quassel@host86-135-232-26.range86-135.btcentralplus.com] has joined #openttd
00:03<ST2>btw Eddi|zuHause: about OpenTTD, and what we've talked: can you say where you make testings?
00:03<ST2>because our test servers are public and real players can touch it and TEST
00:06<ST2>** that's how DistCargo appeared - no replies = it's good" **
00:08-!-ToneKnee_ [~quassel@host86-135-237-142.range86-135.btcentralplus.com] has quit [Ping timeout: 480 seconds]
00:14-!-Snail [~jacopocol@cpe-98-14-130-227.nyc.res.rr.com] has quit [Quit: Snail]
00:21-!-supermop_ [~supermop@pool-100-37-203-161.nycmny.fios.verizon.net] has quit [Ping timeout: 481 seconds]
00:32-!-Quinch [~oftc-webi@d205-206-102-151.abhsia.telus.net] has quit [Ping timeout: 480 seconds]
01:20<Xal>ST2: I think what Eddi|zuHause was trying to tell you was that the burden of proof lies on the accuser, and claiming that the majority of players play MP requires data to back it up. Because there's no data on the total number of player (offline) there's no way to back up your claim
01:30-!-debdog [~debdog@2a02:8070:4584:4200:7a24:afff:fe8a:d04d] has quit [Remote host closed the connection]
01:53-!-Xal [~xal@S0106f0f2490b0073.vw.shawcable.net] has quit [Quit: bye]
02:00-!-Biolunar [Biolunar@x4d0839f4.dyn.telefonica.de] has joined #openttd
02:08-!-Progman [~progman@p57A18A69.dip0.t-ipconnect.de] has joined #openttd
03:10-!-efess [~Efess@c-24-61-64-170.hsd1.ct.comcast.net] has quit [Ping timeout: 480 seconds]
03:12-!-JezK_ [~jez@2407:7800:400:107f:3db5:daca:8457:e66a] has quit [Quit: :q!]
03:20-!-Wolf01 [~wolf01@0001288e.user.oftc.net] has joined #openttd
03:20<Wolf01>o/
03:25-!-Arveen [~Arveen@ip-95-223-75-47.hsi16.unitymediagroup.de] has joined #openttd
03:41-!-Hiddenfunstuff [~Geth@y32.ip1.anvianet.fi] has joined #openttd
04:13-!-Supercheese [~Superchee@cpe-76-178-137-142.natnow.res.rr.com] has quit [Quit: Valete omnes]
04:15-!-Arveen [~Arveen@ip-95-223-75-47.hsi16.unitymediagroup.de] has quit [Quit: Nettalk6 - www.ntalk.de]
04:49-!-JacobD88 [~Thunderbi@cpc80661-stap13-2-0-cust817.12-2.cable.virginm.net] has joined #openttd
05:04-!-efess [~Efess@c-24-61-64-170.hsd1.ct.comcast.net] has joined #openttd
05:10-!-ConductCat [~Conductor@pool-108-56-0-223.washdc.btas.verizon.net] has joined #openttd
05:16-!-Myhorta [~Myhorta@00018fad.user.oftc.net] has joined #openttd
05:16-!-ConductorCat [~Conductor@pool-108-56-0-223.washdc.btas.verizon.net] has quit [Ping timeout: 480 seconds]
05:19<Wolf01>ha... Project Ara (the modular smartphone from LG/Google) lost the full customisation... it will end with just the customisable cover and the ability to replace the battery
05:20<Wolf01>http://www.brickshelf.com/cgi-bin/gallery.cgi?f=564397 also, the guy uploaded the photos of the last exhibition
05:28-!-ConductorCat [~Conductor@pool-108-56-0-223.washdc.btas.verizon.net] has joined #openttd
05:34-!-ToneKnee [~quassel@host86-135-232-26.range86-135.btcentralplus.com] has quit [Ping timeout: 480 seconds]
05:35-!-ConductCat [~Conductor@pool-108-56-0-223.washdc.btas.verizon.net] has quit [Ping timeout: 480 seconds]
05:40-!-debdog [~debdog@2a02:8070:4584:4200:7a24:afff:fe8a:d04d] has joined #openttd
05:47-!-gelignite [~gelignite@x4e3192cb.dyn.telefonica.de] has joined #openttd
06:00<Ethereal_Whisper>mfw I split lines and then merged then immediately because I wasn't paying attention
06:00<Ethereal_Whisper>http://i.imgur.com/RA3YGb8.png GOOD JOB TRICIA
06:01<Wolf01>:)
06:05<Ethereal_Whisper>I'm gonna leave the construction in place since it's fully balanced and it would take more work to delete it than it's worth lol
06:28<V453000>0.13 HYPE
06:28<Wolf01>/¯¯¯¯||¯¯¯¯||¯¯¯¯||¯¯¯¯||¯¯¯¯\
06:28<Wolf01>all aboard the hype train
06:28<V453000>TRAINZ
06:28<V453000>BRAINZ
06:38-!-TheMask96 [martijn@envy.vhost.ne2000.nl] has quit [Ping timeout: 480 seconds]
06:43-!-TheMask96 [martijn@wrath.vhost.ne2000.nl] has joined #openttd
07:22-!-roidal [~roland@194-152-171-20.adsl.highway.telekom.at] has joined #openttd
07:30-!-Arveen [~Arveen@ip-95-223-75-47.hsi16.unitymediagroup.de] has joined #openttd
07:34-!-debdog [~debdog@2a02:8070:4584:4200:7a24:afff:fe8a:d04d] has quit [Remote host closed the connection]
07:38-!-debdog [~debdog@2a02:8070:4584:4200:7a24:afff:fe8a:d04d] has joined #openttd
08:14-!-debdog [~debdog@2a02:8070:4584:4200:7a24:afff:fe8a:d04d] has quit [Remote host closed the connection]
08:15-!-Samu [~oftc-webi@po-217-129-255-23.netvisao.pt] has joined #openttd
08:15-!-debdog [~debdog@2a02:8070:4584:4200:7a24:afff:fe8a:d04d] has joined #openttd
08:26-!-Snail [~jacopocol@cpe-98-14-130-227.nyc.res.rr.com] has joined #openttd
08:34-!-sim-al2 is now known as Guest2188
08:34-!-sim-al2 [~sim-al2@108-221-157-231.lightspeed.mmphtn.sbcglobal.net] has joined #openttd
08:35<Samu>hi
08:35<Samu>
08:38-!-Guest2188 [~sim-al2@108-221-157-231.lightspeed.mmphtn.sbcglobal.net] has quit [Ping timeout: 480 seconds]
08:44-!-Wolf03 [~wolf01@host206-62-dynamic.25-79-r.retail.telecomitalia.it] has joined #openttd
08:44-!-Wolf01 is now known as Guest2190
08:44-!-Wolf03 is now known as Wolf01
08:46-!-Guest2190 [~wolf01@0001288e.user.oftc.net] has quit [Read error: Connection reset by peer]
08:50-!-Snail [~jacopocol@cpe-98-14-130-227.nyc.res.rr.com] has quit [Quit: Snail]
08:52-!-Snail [~jacopocol@cpe-98-14-130-227.nyc.res.rr.com] has joined #openttd
08:52-!-Snail [~jacopocol@cpe-98-14-130-227.nyc.res.rr.com] has quit []
08:57-!-JacobD88 [~Thunderbi@cpc80661-stap13-2-0-cust817.12-2.cable.virginm.net] has quit [Quit: JacobD88]
09:12-!-andythenorth [~Andy@194.168.185.226] has joined #openttd
09:14<Samu>i stumbled upon a log with st2 arguing
09:14<Samu>with zu
09:16<Samu>i'm sorry st2, but you started the "war"
09:16<Samu>that's my view on it
09:18-!-andythenorth [~Andy@194.168.185.226] has left #openttd []
09:20<Samu>for me, CargoDist is an interesting change of pace, but I agree, it's confusing at first
09:22<supermop>not sure why some people find it offensive
09:26<Samu>the rest of the discussion is... uhm.... "irrelevant" to the topic at hand: CargoDist
09:33<Samu>CargoDist in multiplayer, might work better on those servers without goal, sandbox style
09:38<Samu>I don't think people hate CargoDist because they dislike it, but rather because they don't know how it works. I didn't like it at first, mainly because I didn't know what was happening
09:38<Samu>there is a learning curve that players have to go thru
09:38<Samu>but I guess most don't want to
09:46<Wolf01>I enable it and connect everything, then I'll upgrade some lines or others depending on the needs
09:46<Wolf01>I don't know how it works and I don't want to
09:47<Wolf01>it's just another mean to give some movement to the game, like industry closures and production changes
09:50-!-Alberth [~alberth@2001:981:c6c5:1:be5f:f4ff:feac:e11] has joined #openttd
09:50-!-mode/#openttd [+o Alberth] by ChanServ
09:50<Wolf01>o/
09:50<@Alberth>hi hi
09:53<Samu>yes Wolf01 - i also find CargoDist more tailored towards long lasting games
09:54<Samu>btpro servers are mostly competitive games that don't tend to last too long.
09:54<Samu>i see why it doesn't become popular
09:55<supermop>no reason you can't have a 'competitive' game with CD, it's just that the existing competitive servers and GS use criteria that don't take advantage of it at all
09:59<V453000>every time we tried cargodist, it would start doing weird shit with a big network, and not giving any gameplay benefit at all
10:01<@Alberth>solving shit isn't the fun part? :)
10:03<V453000>it was shit you can't solve
10:03<V453000>I don't remember specifically, but it happens every single time
10:03<_dp_>CD is much like a GS on its own, conflicts with almost any other gs/goal.
10:04<V453000>once like 95% of farm supplies prefered 1 farm cluster in FIRS, another time some passengers decided it is a good thing to wait at the station for wtf reason
10:04<V453000>we didn't use any GS
10:04<V453000>it's just trash
10:04<V453000>IF it had a nice gameplay-enhancing concept, I could see the point
10:05<Samu>why would passengers want to go to a farm
10:05<Samu>farm accepts passengers?
10:05<Wolf01>V453000, maybe the other farm paid more, and the passengers were doing something
10:06<V453000>yeah Wolf01 , nope
10:06<@Alberth>disabled preference for local delivery, I hope?
10:06<V453000>I don't really know anymore Alberth
10:06<@Alberth>but yeah, it starts working when you connect things, so biggest trouble arises when you connect everything(tm)
10:07<V453000>but most likely our settings were proper, most likely I discussed the issue when it was hot in this channel
10:07<@Alberth>:)
10:08<Wolf01>the only time I found a weird situation was when I changed the direction of the traffic because I removed an old loop around a lake replacing it with a more suitable (and fast) double track with a bridge, but I had to connect it on the other side of a station
10:09<Wolf01>while before it was A-B-C-D, it bacame A-C-B-C-D and the network exploded
10:09<Wolf01>*became
10:09<V453000>it probably always has a chance do do some weird shit, but since we build huge networks, that chance usually (always so far) gets triggered
10:19<Samu>https://www.openttd.org/en/server/98619 - I joined my server to see how long it would take to fully synchronize
10:19<Samu>5 minutes 40 seconds
10:22<Samu>2 min 30 secs to download, then 3 min 10 secs to catch-up
10:22<Samu>there's more than 15k vehicles in it
10:23<Wolf01>http://img-9gag-fun.9cache.com/photo/awxwvW8_460sv_v1.mp4 could we have these?
10:24<Wolf01>oh wait... there aren't cars in ottd :(
10:24<Wolf01>Anybody who wants to add cars so we can have these? :D
10:25<Samu>15965 vehicles
10:29<Samu>server is so slow that sometimes it disappears from the master server list
10:29<Samu>then re-appears
10:29<supermop>i never have had 'weird' problems with CD, even with every town on the map connected in one network, and local trams or buses within those towns
10:30<supermop>i do get lines or nodes that become stressed over capacity, but i see that as part of the point of CD, to selectively determine which trunks need more bandwidth
10:31<supermop>rather than a point to point mesh that eventually is just homogeneous over the whole map
10:32<Wolf01>play mini metro, that's paxdest on steroids
10:46<supermop>Wolf01: actually i find homogeneous mesh is a valid strategey in mini metro
10:46<Wolf01>I still fail with every strategy
10:47<Wolf01>tries with a mesh, trunks and branches, triangles, circles, squares, clouds, dogs and dicks
10:47-!-ConductCat [~Conductor@pool-108-56-0-223.washdc.btas.verizon.net] has joined #openttd
10:48<supermop>Wolf01: i dont think it is possible to not fail
10:49<Wolf01>with "fail" I mean that I transport <1000 passengers before failing hard
10:49<Wolf01>usually I'm on 480-650
10:53-!-ConductorCat [~Conductor@pool-108-56-0-223.washdc.btas.verizon.net] has quit [Ping timeout: 480 seconds]
11:06<Samu>i have a problem to solve, it requires savegame conversion
11:06<Samu>"- Implemented a fix that corrects the name of AIs that are started via startai command in the console when the parameter also included the version. Saving a game will thus, store the correct name of the AI from now on. Loading previous savegames that have stored the incorrect name will still fail to recognize the intended name of the AI."
11:09<Samu>i wanted to convert the name that is incorrectly stored on old saves into the correct name
11:10<Samu>problem is that how would it distinguish which name is right from which name is wrong
11:11<Samu>something Wolf01 said to me yesterday
11:11<Samu>there is no real way to tell, is it?
11:16<Wolf01>I said that was the wrong approach for the problem
11:17<Wolf01>maybe you should find why it pass the folder name to the function instead of the display name, and not trying to fix the string at the end
11:18-!-gelignite [~gelignite@x4e3192cb.dyn.telefonica.de] has quit [Quit: http://bit.ly/1kso8Ta]
11:20<Samu>it doesn't pass folder name
11:20<Samu>i type in console: startai wormai.4
11:21<Samu>it takes the 'wormai.4' part to search for a wormai with version 4, it does this part correctly
11:21<Samu>however, it forgets to put the correct name once it finds it
11:21<Samu>it leaves wormai.4 as the name of the AI
11:22<Wolf01>then you need to get the real name when the search function finds it
11:22<Samu>my fix was actually that
11:23<Wolf01>but you shouldn't edit the strings directly (eg. removing the .4)
11:24<Samu>i put this after line 25 of script_config.cpp
11:24<Samu>this->name = (info == NULL) ? NULL : stredup(GetInfo()->GetName());
11:24<Samu>info == NULL would mean that it did not find any wormai with version 4
11:26<Samu>if there is INFO, it changes this->name into the intended name
11:27<Samu>this->name goes from wormai.4 to WormAI
11:30<Samu>I am, however, wondering if this fix breaks something else
11:30<Samu>i am assuming that it doesn't
11:30<Wolf01>you need to find where is used this->name
11:31<Samu>it is used on ai_sl.cpp for example
11:31<Samu>both saving and loading
11:31<Wolf01>then you might want to add this->realname and use that, leaving this->name untouched
11:44<Samu>you're right, i can't do it like this
11:46-!-TheMask96 [martijn@wrath.vhost.ne2000.nl] has quit [Ping timeout: 480 seconds]
11:51-!-TheMask96 [martijn@sloth.vhost.ne2000.nl] has joined #openttd
11:57<Samu>if (this->info != NULL) this->name = (GetInfo()->GetName());
11:57<Samu>pff :(
11:58<Wolf01>it's almost the same as before
11:58<Samu>doesn't change "none" to NULL anymore, but... it's the wrong approach
11:59<Wolf01>also, I need to sleep
12:01<Samu>woah, it really is searching for a scriptI by the name none
12:01<Samu>what would happen if someone actually creates an AI with this name
12:03-!-tokai [~tokai@00012860.user.oftc.net] has joined #openttd
12:03-!-mode/#openttd [+v tokai] by ChanServ
12:05<Samu>i got to fix it at the console_cmds.cpp
12:09-!-tokai|noir [~tokai@00012860.user.oftc.net] has quit [Ping timeout: 480 seconds]
12:18-!-frosch123 [~frosch@00013ce7.user.oftc.net] has joined #openttd
12:26<Samu> config->Change(config->GetInfo()->GetName()); - pointer to incomplete class type is not allowed
12:26<Samu>:(
12:29<Samu>3>..\src\console_cmds.cpp(1175): error C2027: use of undefined type 'AIInfo'
12:29<Samu>3> d:\openttd\trunk\src\company_base.h(118): note: see declaration of 'AIInfo'
12:29<Samu>3>..\src\console_cmds.cpp(1175): error C2227: left of '->GetName' must point to class/struct/union/generic type
12:31<Samu>company_base.h, line 118: class AIInfo *ai_info;
12:32<Samu>I have no idea what I'm looking at :(
12:33<Samu>these errors are chinese to me, no idea what's undefined, what's declaration, what's class/struct/union... :(
12:39<TrueBrain>buy any C++ book?
12:39<TrueBrain>or .. any programming book, if class/struct/union is unknown to you
12:41<@Alberth>nah, that would be too easy
12:42<TrueBrain>I am afraid the word is: too hard :P
12:42<@Alberth>also a valid option :)
12:43<@Alberth>quak
12:43<TrueBrain>I am still considering frosch only boots up till someone said quak ..
12:43<@Alberth>he reads the log, looking for quaks :)
12:43<frosch123>hoi mammals and gastropodes :)
12:44<TrueBrain>I am gassy today, that is for sure
13:03-!-Gja [~Martin@93-167-84-102-static.dk.customer.tdc.net] has joined #openttd
13:03-!-glx [~glx@000128ec.user.oftc.net] has joined #openttd
13:03-!-mode/#openttd [+v glx] by ChanServ
13:06<Samu>omg I think I did it
13:08<Samu>https://paste.openttdcoop.org/puz8c3l3n
13:08<@Alberth>all problems solved \o/
13:08<Samu>I hope it is correcting the name now
13:08<Samu>gonna make sure
13:10<Samu>hmm, i guess not
13:10<Samu>nop, grrr
13:11<Samu>I only want to change the name, not the whole script
13:12<Samu>removed
13:15<Samu>https://paste.openttdcoop.org/pqbqzlwvt - reverted to this fix, it does it better, but at the risk of not being compatible with the so many other functions that make use of this
13:51-!-HerzogDeXtEr [~farci@i59F6B6FD.versanet.de] has joined #openttd
14:02-!-Quatroking [~Quatrokin@ip226-139-211-87.adsl2.static.versatel.nl] has joined #openttd
14:06<Wolf01>you don't even need the else, just put the code between the 2 ifs
14:09-!-ToneKnee [~quassel@host109-148-29-203.range109-148.btcentralplus.com] has joined #openttd
14:12-!-Tirili [~Unknown@HSI-KBW-082-212-030-207.hsi.kabelbw.de] has joined #openttd
14:12<Samu>Change(stuff, stuff, stuff, stuff) - does a lot of stuff than just changing the name :(
14:13<Samu>void ScriptConfig::Change(const char *name, int version, bool force_exact_match, bool is_random)
14:16<Samu>AIInfo *AIScannerInfo::FindInfo(const char *nameParam, int versionParam, bool force_exact_match)
14:16<Samu>this is the real information retriever
14:16-!-andythenorth [~Andy@cpc87201-aztw31-2-0-cust156.18-1.cable.virginm.net] has joined #openttd
14:16-!-rahtgaz [~rahtgaz@00021442.user.oftc.net] has joined #openttd
14:17<Samu>there's a GameInfo version too
14:19<Samu>feels like a network of functions
14:21<Samu>but what I wanted to say was that, https://paste.openttdcoop.org/puz8c3l3n - line 19 is doing it wrong
14:22<Samu>it changes the name, yes, but then also searches the system for an AI with the changed name, and if I had told to start version 4 of wormai, now it is going to load the most up-date version, so I can't do it like that
14:23<Samu>it was going to load version 5
14:25-!-Wormnest [~Wormnest@s5596abd2.adsl.online.nl] has joined #openttd
14:30<Samu>hi w
14:30<Samu>Wormnest:
14:30<Wormnest>hi :)
14:31<Samu>i made a fix, but it's far from perfect https://paste.openttdcoop.org/pqbqzlwvt?/pqbqzlwvt
14:32<Samu>there are a lot of functions that depend on Change function
14:33<Wormnest>Nice, does it work for all cases now?
14:33<Samu>I don't know how to answer that
14:33<Samu>old saves won't work
14:33<Samu>they stored the wrong name
14:34<Samu>loading them back will still give the same error you were getting
14:34<Wormnest>Well I´m not too sure that´s correct
14:34<Samu>new saves, however, will store the correct name
14:34<Samu>loading back will not give that error, but will still load the latest available version of WormAI
14:34<Wormnest>I´ve been thinking about it and I think storing wormai.4 instead of wormai was on purpose
14:35<Samu>oh really? :(
14:35<Wormnest>It was done to differentiate with a normally loaded ai
14:36<Wormnest>This also means I think that the purpose was to be able to load back the explicitly set version of the ai
14:36<andythenorth>is cat?
14:36<Samu>you're refering to the 2nd savegame? it has it stored as WormAI
14:37<Samu>how did you start it?
14:37<Samu>the bug is originated from within the console :(
14:37<Wormnest>no the first
14:38<Samu>1st savegame has it stored as wormai.4 - this was what you had input in the console
14:38<Wormnest>starting as startai wormai.4 stores name in savegame as wormai.4
14:38<Wormnest>but starting normal saves as wormai
14:39<Samu>it's a console command bug in my opinion, what led me to check what was is really doing
14:39<Wormnest>I think it´s on purpose but I might be wrong
14:40<Wormnest>If you add a command to load an explicit version of an ai instead of the latest version
14:40<Wormnest>It also makes sense that you would want to load that version back from a savegame
14:40<Wormnest>Thus you cannot save it as wormai and store version 4, but you need to add something thus wormai.4
14:41<Samu>it stores both, name and version
14:41<Samu>name was being stored as wormai.4, version was stored as 4
14:41<Wormnest>Yes
14:41<Samu>uhm yes? so hmm I'm confused
14:42<Wormnest>But if it stored name as wormai then you could not distinguis between wormai loaded normally
14:42<Wormnest>and an older wormai version loaded explicitly when a newer version was already available
14:43<Samu>min_loadable_version for both version 4 and version 5 is 1
14:43<Wormnest>that´s why it stores it as wormai.4 meaning ok we know this isn´t the latest version but that´s the one we want
14:44<Samu>hmm let me test again with my fix
14:44<Wormnest>but ofcourse I´m only guessing I´m not the one who wrote the code :)
14:45<Samu>there's more checkings done other than just the name
14:45<Samu>it checks version if it finds an ai with that name
14:45<Samu>since it was working with the wrong name, it wasn't even going to check the version
14:45<Wormnest>Yea that was clear from the console messages
14:46<Samu>let me find the piece of code that works with that information
14:46<Samu>if (strcasecmp(ai_name, i->GetName()) == 0 && i->CanLoadFromVersion(versionParam) && (version == -1 || i->GetVersion() > version)) {
14:47<Wormnest>By the way the documentation claims that the intention is to always load the version of the ai it was saved with:
14:47<Wormnest>http://noai.openttd.org/docs/trunk/classAIController.html#_details
14:48<Samu>strcasecmp(ai_name, i->GetName()) == 0 - this part was always false
14:48<Samu>version was never checked
14:49<Samu>it's quite a labyrinth of information being passed around
14:49<Wormnest>I bet
14:49<@Alberth>hi hi andythenorth
14:50-!-kais58__ [~kais58@88.98.85.222] has joined #openttd
14:50<Samu>i->CanLoadFromVersion - this i is the script it found on the system, there's 2 wormais in my rig, wormai version 4 and wormai version 5
14:50<Samu>it finds wormai 4 first
14:51-!-FLHerne [~flh@cpc4-papw5-2-0-cust175.5-3.cable.virginm.net] has joined #openttd
14:51<Samu>compares the min_loadable_version that is on wormai 4 in the system and the version stored in the savegame
14:51<Samu>then does the same for wormai 5
14:52<Samu>and comes up with the most up to date version which can load that save, it's wormai 5
14:52<Samu>at least it's what I think it's doing
14:52-!-kais58_ [~kais58@88.98.85.222] has quit [Ping timeout: 480 seconds]
14:52<Samu>had you set wormai 5 with a min loadable version of 5
14:53<Samu>i bet it would only load wormai 4
14:53<Wormnest>That´s what I think it´s doing too, the question is was that really the intention
14:53<Wormnest>possibly
14:54<Samu>savegame stored it as 4, forgot to mention
15:00<Samu>i'm reading those docs
15:01<Samu>heh, how do you guys always find this documentation
15:02<@Alberth>usually by function name
15:02<@Alberth>ie you look for a function name that looks what you are looking for , go to it, and start reading
15:03<Samu>i just follow visual studio jumping around and try to understand why
15:03<@Alberth>I run grep with parts of names that the function likely contains
15:05<Samu>there's 5 different results i see
15:05<Samu>with the bug it was jumping towards result 3 in that doc
15:06<Samu>with the fix, it jumps to 2, it seems
15:06<Samu>load the latest version of the same script that supports loading data from the saved version (the version of saved data must be equal or greater than AIInfo::MinVersionToLoad),
15:07<Samu>it skips version 4, because version 5 is compatible
15:07<Samu>step 1 is skipped
15:15<Samu>https://wiki.openttd.org/AI:Save/Load - hmm the explanation here
15:15<Samu>convinces me there's something wrong
15:16<Samu>As soon as one step succeeds, the sequence is stopped: Try to load the exact same version of the same AI.
15:16<Samu>it isn't doing exactly like this
15:17<Samu>why?
15:18<Samu>as soon as one step succeeds, the sequence is stopped, hmm... that's not what I see it doing
15:19<Samu>Try to load the exact same version of the same AI. - it finds it, but it doesn't stop here, it is iterating over all versions
15:20<Samu>and gets the most up to date that is compatible
15:20<Samu>documentation says that it stops :(
15:21<Samu>it's not what it does in the code, or .... uhm... whatever
15:21<@Alberth>one of them is wrong, and it's not the code :p
15:22<@Alberth>although you should probably check the changes in that area, see what was changed last
15:25<Wormnest>I think loading the latest version is indeed the intention
15:25<Wormnest>Except maybe for the case where you explicitly loaded an older version of an ai
15:26-!-sla_ro|master [~sla.ro@89.136.141.100] has joined #openttd
15:26-!-rahtgaz [~rahtgaz@00021442.user.oftc.net] has quit [Quit: I'll be Bach]
15:27<Samu>i see, let me check what comes next
15:27<Samu>line 86 config->Change(_ai_saveload_name, _ai_saveload_version, false, _ai_saveload_is_random);
15:28<Samu>tries to match name and version
15:28<Samu>it does not find it, so next step is in line...
15:28<Samu>line 90 config->Change(_ai_saveload_name, -1, false, _ai_saveload_is_random);
15:29<Samu>only tries to match name now
15:30<Samu>name is still wormai.4, it takes .4 it splits it into 2 parts, wormai and 4
15:31<Samu> /* We want to load the latest version of this AI; so find it */
15:31<Samu> /* If we didn't find a match AI, maybe the user included a version */
15:32<Samu>yes, but why in the name?
15:32-!-HerzogDeXtEr [~farci@i59F6B6FD.versanet.de] has quit [Read error: Connection reset by peer]
15:33<Samu> /* See if there is a compatible AI which goes by that name, with the highest * version which allows loading the requested version */
15:33<Samu>it's doing this cycle again, but now with the name separated from the version
15:34<Samu>versionParam is 4, even though it was -1 initially
15:35<Samu>it is still going to load version 5
15:35<Samu>but this time, it is discarding the saved data
15:36<Samu>DEBUG(script, 0, "The latest version of that AI has been loaded instead, but it'll not get the savegame data as it's incompatible.");
15:36<Samu>yeah
15:36<Samu>AI::Load(index, _ai_saveload_version); _ai_saveload_version at this point is -1
15:36<Samu>-1 means, discard whatever was saved
15:37<Samu> if (this->engine == NULL || version == -1) { LoadEmpty();
15:38<Samu>yep
15:48<Samu>gonna try force_exact_match = true
15:49<Samu>line 86 of ai_sl.cpp, changing it to config->Change(_ai_saveload_name, _ai_saveload_version, true, _ai_saveload_is_random);
15:49<Samu>see what happens :p
15:50<Samu>oops, this is line 90, i'm stupid
15:50<Samu>config->Change(_ai_saveload_name, _ai_saveload_version, true, _ai_saveload_is_random);
15:51<Samu>line 86
15:55<Samu>0x000000caf1cfe340 "wormai.4.4" - keks, it's doing worse
15:58<Samu>it took wormai.4, and appended .4 at the end of it, now it's searching for wormai.4.4 on the system, there is no such version
15:58<Samu>there is wormai.4 and wormai.5
15:58<Samu>this is so messed up
16:00<Samu>and now, it is comparing wormai.4 to WormAI
16:00-!-Gja [~Martin@93-167-84-102-static.dk.customer.tdc.net] has quit [Quit: Going offline, see ya! (www.adiirc.com)]
16:01<Samu>hmm :(
16:02<Samu>gonna try yet another approach
16:06-!-Alberth [~alberth@2001:981:c6c5:1:be5f:f4ff:feac:e11] has left #openttd []
16:07<Samu>yes, it loaded wormai.4 from the savegame
16:08<Samu>but.... hmm....
16:09<Samu>i'm not totally sure if this is without issues, needs more testing
16:10<Samu>if I do this to ai_sl.cpp, i must do it to game_sl.cpp as well
16:13<Samu>https://paste.openttdcoop.org/pyjs64de7 - here's what i'm testing
16:13<Samu>however, i'm worried about passing -1 as the version
16:13<Samu>dinner time, will check this later
16:18-!-Arveen [~Arveen@ip-95-223-75-47.hsi16.unitymediagroup.de] has quit [Quit: Nettalk6 - www.ntalk.de]
16:26-!-Wormnest [~Wormnest@s5596abd2.adsl.online.nl] has quit [Quit: Leaving]
16:35-!-sla_ro|master2 [~sla.ro@89.136.141.100] has joined #openttd
16:36-!-Tirili [~Unknown@HSI-KBW-082-212-030-207.hsi.kabelbw.de] has quit [Quit: Leaving]
16:39-!-sla_ro|master [~sla.ro@89.136.141.100] has quit [Ping timeout: 480 seconds]
16:42<Samu>back
16:42<Samu>can't pass version -1 at that point :(
16:49-!-gelignite [~gelignite@x4e3192cb.dyn.telefonica.de] has joined #openttd
16:50-!-frosch123 [~frosch@00013ce7.user.oftc.net] has quit [Quit: be yourself, except: if you have the opportunity to be a unicorn, then be a unicorn]
16:52-!-andythenorth [~Andy@cpc87201-aztw31-2-0-cust156.18-1.cable.virginm.net] has quit [Quit: andythenorth]
17:07-!-rahtgaz [~rahtgaz@00021442.user.oftc.net] has joined #openttd
17:09-!-Emiel [~oftc-webi@541E0E63.cm-5-7a.dynamic.ziggo.nl] has joined #openttd
17:09<Emiel>hello?
17:09*Emiel slaps ST2 around a bit with a large fishbot
17:17-!-Emiel [~oftc-webi@541E0E63.cm-5-7a.dynamic.ziggo.nl] has quit [Ping timeout: 480 seconds]
17:29-!-tokai|noir [~tokai@00012860.user.oftc.net] has joined #openttd
17:29-!-mode/#openttd [+v tokai|noir] by ChanServ
17:32-!-sla_ro|master2 [~sla.ro@89.136.141.100] has quit []
17:35-!-tokai [~tokai@00012860.user.oftc.net] has quit [Ping timeout: 480 seconds]
17:44-!-roidal [~roland@194-152-171-20.adsl.highway.telekom.at] has quit [Quit: WeeChat 1.4]
17:44-!-Biolunar [Biolunar@x4d0839f4.dyn.telefonica.de] has quit [Ping timeout: 480 seconds]
17:53<Samu>i have another fix in testing
17:54<Samu>this one required me to write a bit of code
17:54<Samu>more like... copy pasting code
18:00-!-Hiddenfunstuff [~Geth@y32.ip1.anvianet.fi] has quit [Quit: HydraIRC -> http://www.hydrairc.com <- Po-ta-to, boil em, mash em, stick em in a stew.]
18:05-!-supermop [~supermop@static-71-249-209-97.nycmny.east.verizon.net] has quit [Ping timeout: 480 seconds]
18:05-!-Progman [~progman@p57A18A69.dip0.t-ipconnect.de] has quit [Remote host closed the connection]
18:08<Samu>https://paste.openttdcoop.org/pxkc1qh0d
18:09<Samu>this is yet my best approach
18:12<Samu>seems to be doing what the documentation wants it to do
18:16<Samu>ah man... not really
18:18<Samu>:( this is tiresome work
18:53<Wolf01>'night
18:53-!-Wolf01 [~wolf01@0001288e.user.oftc.net] has quit [Quit: Once again the world is quick to bury me.]
19:50<Samu>https://paste.openttdcoop.org/phjib9464 - another attempt
19:51<Samu>I'm really tired
19:59-!-gelignite [~gelignite@x4e3192cb.dyn.telefonica.de] has quit [Quit: http://bit.ly/1kso8Ta]
20:04-!-Quatroking [~Quatrokin@ip226-139-211-87.adsl2.static.versatel.nl] has quit [Read error: Connection reset by peer]
20:20-!-Myhorta [~Myhorta@00018fad.user.oftc.net] has quit [Ping timeout: 480 seconds]
20:22-!-Snail [~jacopocol@cpe-98-14-130-227.nyc.res.rr.com] has joined #openttd
21:04<Samu>why is the forum so slow at 2:00?
21:05<Samu>betweem 2:00 - 2:15 everyday it's slow :(
21:22<Samu>finally was able to post
21:23<Samu>https://www.tt-forums.net/viewtopic.php?f=65&p=1169417#p1169417 - cyas, goodnight
21:23-!-Samu [~oftc-webi@po-217-129-255-23.netvisao.pt] has quit [Quit: Page closed]
21:32-!-rahtgaz [~rahtgaz@00021442.user.oftc.net] has quit [Quit: I'll be Bach]
22:09-!-FLHerne [~flh@cpc4-papw5-2-0-cust175.5-3.cable.virginm.net] has quit [Ping timeout: 480 seconds]
22:49-!-glx [~glx@000128ec.user.oftc.net] has quit [Quit: Bye]
---Logclosed Fri May 27 00:00:33 2016