skai2 / EDAutopilot

Elite Dangerous computer vision based auto-navigation including long range routing
MIT License
148 stars 66 forks source link

Refuel not working correctly #20

Open TekuSP opened 5 years ago

TekuSP commented 5 years ago

It did not even try to refuel, it just stopped, turned around ship and then located target.

skai2 commented 5 years ago

Hey there, for me to be able to help you I'm going to need some more details about what happened. Also, please attach your autopilot.log.

TekuSP commented 5 years ago

https://pastebin.com/BrYk7U0d

hbzxc commented 5 years ago

I thought I was having a similar issue with it skipping the refuel step. However it was working I just wasn't comfortable with how low it got before it triggered.

You can change the refuel threshold from 33% to 100% to check and see if its triggering. That is located in the dev_autopilot.py and is part of the Refuel function.

If that still does not fix the issue check the star class. I think the refuel step only triggers when scoopable_stars = ['F', 'O', 'G', 'K', 'B', 'A', 'M']

ScreaminSteve commented 5 years ago

An easy thing to do for exploration would be to filter your route by the KGB-FOAM (scoopable star classes) at least that's what I did and it worked just fine, since then any star on the route would be scoopable.

TekuSP commented 5 years ago

No no, It was trying to refuel, but it did loop too soon. Not around star but before it. Not sure why.

TekuSP commented 5 years ago

Or is loop before star normal procedure before aligning to target?

skai2 commented 5 years ago

Hey there, this project has been on a hiatus for a while so please forgive my late reply. The autopilot assumes fuel has been checked before it is started, so if you had low fuel and turned on the autopilot expecting it would try to refuel immediately, that is not the case. The procedure it follows is to only attempt refueling after a jump, and as @hbzxc stated, it will only refuel on scoopable stars, so the ideal is to set your route filter to said stars as recommended by @ScreaminSteve. I hope this answers your question.