Code: Select all
if self.requestSlots == nil then
self:updateRequestSlots()
end
Code: Select all
if self.requestSlots == nil then
self:updateRequestSlots()
end
Ok, I'll see if I can reproduce it and I'll fix it then.Rens2Sea wrote:Got a little bug.
Happens sometimes when a train (electric (5dim)) with requester wagons stops at a station.
Partly fixed it by changing line 57 in requesterwagon.lua from:To:if self.proxy ~= nil and self.requestSlots ~= {} thenThis ONLY fixed the crashing, the request wagons lose the set logistics though, so they become uselessif self.proxy ~= nil and self.slots ~= nil and self.requestSlots ~= {} then
Please fix, my entire factory has been designed with requester wagons in mind.
EDIT: This was dumb, there isn't even a self.slots variable. Nevermind me, tired...
It seems to have to do with that the requester slots weren't loaded properly. Essentially, I forgot a line. I threw in a nil check for good measure but in my tests I have a requester wagon going around for a good couple of autosaves now and save-load works.Rens2Sea wrote:Ok, so instead of doing that silly if check from my previous post add this to the top of function RequesterWagon:setRequestSlots():
This immediately gets called after i load my bugged savegame (crashed happened a few second after the save), and my requester wagons seem to keep their logistics. Must test some more. Tomorrow.Code: Select all
if self.requestSlots == nil then self:updateRequestSlots() end
Interesting Does it also duplicate rail signals/poles/lamps? If it is only rails then it's my/FARLS bug.Kane wrote:I found a bug today with FARL Sadly. It seems that it does not remove the required items out of the logistic wagons. Instead it just duplicates them and never removes anything from the wagon.
Code: Select all
return true
I will go ahead and test changing line 60. It duplicates signals and power lines as well I'm 99% sure but I will check and reply if I'm wrong on that. Will also reply back after my results of testing this possible fix.Choumiko wrote:Interesting Does it also duplicate rail signals/poles/lamps? If it is only rails then it's my/FARLS bug.Kane wrote:I found a bug today with FARL Sadly. It seems that it does not remove the required items out of the logistic wagons. Instead it just duplicates them and never removes anything from the wagon.
If not then changing line 60 in wagons/proxywagon.lua tocould solve the bug, not sure though (but could lead to other problems/bugs)Code: Select all
return true
The FARL update changed it to this? It kind of sounds like a bug on my end but it's strange an update to FARL broke it. I will have a look as quickly as I can.Kane wrote:Sadly it seems that the update had killed the logistics wagons. For example you move the train and stop it would sort of place that icon down where the train is at and you know the bots would work. Well that only happens when you place it down. Now if you move and stop it does not seem to reset it correctly anymore. Meaning that you have to replace the wagon every single time you want the bots to interact with it.
But the good news is Farl is now using its contents
Thanks it could be something else causing it but I have not been able to get them to work correctly for a bit I'm pretty sure right after this update but I don't use them that often.gnz wrote:The FARL update changed it to this? It kind of sounds like a bug on my end but it's strange an update to FARL broke it. I will have a look as quickly as I can.Kane wrote:Sadly it seems that the update had killed the logistics wagons. For example you move the train and stop it would sort of place that icon down where the train is at and you know the bots would work. Well that only happens when you place it down. Now if you move and stop it does not seem to reset it correctly anymore. Meaning that you have to replace the wagon every single time you want the bots to interact with it.
But the good news is Farl is now using its contents
I think i saw this when fixing FARL (meaning pre-update). Could be factorio 0.11.17 ? That's what i was usingKane wrote:Sadly it seems that the update had killed the logistics wagons. For example you move the train and stop it would sort of place that icon down where the train is at and you know the bots would work. Well that only happens when you place it down. Now if you move and stop it does not seem to reset it correctly anymore. Meaning that you have to replace the wagon every single time you want the bots to interact with it.
Could well been the 17 update. I am using that now updated it when it came out the next day.Choumiko wrote:I think i saw this when fixing FARL (meaning pre-update). Could be factorio 0.11.17 ? That's what i was usingKane wrote:Sadly it seems that the update had killed the logistics wagons. For example you move the train and stop it would sort of place that icon down where the train is at and you know the bots would work. Well that only happens when you place it down. Now if you move and stop it does not seem to reset it correctly anymore. Meaning that you have to replace the wagon every single time you want the bots to interact with it.
I updated to .17 and tried the mod again and as far as I can see the requester wagon puts down the proxy chest correctly. Admittedly, I haven't tried this with FARL yet and I will get to that to see if there is an incompatibility.Kane wrote:Could well been the 17 update. I am using that now updated it when it came out the next day.Choumiko wrote:I think i saw this when fixing FARL (meaning pre-update). Could be factorio 0.11.17 ? That's what i was usingKane wrote:Sadly it seems that the update had killed the logistics wagons. For example you move the train and stop it would sort of place that icon down where the train is at and you know the bots would work. Well that only happens when you place it down. Now if you move and stop it does not seem to reset it correctly anymore. Meaning that you have to replace the wagon every single time you want the bots to interact with it.
I just tried it with the latest FARL and it worked for me, both with normal loco and a FARL. So i can't confirm the bug.gnz wrote:Just for sanities sake, are these the steps you do to reproduce the bug on your end?
- Put down a requester wagon, the square for the proxy chest is there
Get in train and stop, the square for the proxy chest does not show up
That might very well be the bug that Kane found. I've updated the mod to address that for now, but I want to change this so that they join the logistics systems at small speeds. The thinking process is that if you're manual then sliding in towards the base should be sufficient to trigger it but when moving too quickly it takes it out so that the robots doesn't go chasing a quick train.Choumiko wrote:I just tried it with the latest FARL and it worked for me, both with normal loco and a FARL. So i can't confirm the bug.gnz wrote:Just for sanities sake, are these the steps you do to reproduce the bug on your end?
- Put down a requester wagon, the square for the proxy chest is there
Get in train and stop, the square for the proxy chest does not show up
I noticed though that chests don't get removed when you are reversing, they only get removed once you go forward again. Should be ~= 0 on line 57 i guess