GreyHak / dsp-drone-clearing

Drone Clearing mod for Dyson Sphere Program
BSD 3-Clause "New" or "Revised" License
5 stars 6 forks source link

Drone never come back #1

Closed Crazyht closed 3 years ago

Crazyht commented 3 years ago

I start a new game, after harvesting a few resources manually, my Icarus panel 0/3 drones, I wait few minutes but the counter still on 0/3 :(

If I try disable your plugin my save game launch but generate an error : image

GreyHak commented 3 years ago

Thank you for the report @Crazyht .

I agree the game shouldn't produce an error when the mod gets disabled. I'll work on fixing that today. It will mean that clearing tasks won't be saved, but I think that's a reasonable compromise to avoid this error.

Even in the unmodded game it's possible to fly away from your drones so the count stays at zero. So I chose not to override that mechanic. With clearing always on this problem is definitely emphasized. I can look into adding a method to recover lost drones.

GreyHak commented 3 years ago

@Crazyht I never saw it during testing before, but I did when I loaded my game today. A drone got stuck on its task. I'll resolve this today. Thank you again for the feedback.

GreyHak commented 3 years ago

The drone loss is occurs when Icarus clears an object that a drone has been tasked to clear.

Crazyht commented 3 years ago

@GreyHak yes I think that happens when the drone and Icarus try to get the same resource, maybe one hotkey to disable temporarily your mod should be a good idea, it's easy to go out of range of the drone when you move on the planet and sometime I just don't want my drone to get resource when I only move from one building to another :)

thezak48 commented 3 years ago

I second the idea of a hotkey to disable/enable the mod in game

GreyHak commented 3 years ago

Drone lock-up fixed in GitHub. Will release to thunderstore today. I would like a method to enable/disable clearing in game, but I need to learn how to do this. So it's on the list of feature enhancements.

thezak48 commented 3 years ago

As long as it is on the table to be done then I am happy about it.

Could always be a UI button to handle it

GreyHak commented 3 years ago

Error message after removing the mod should now be fixed in GitHub.

I have confirmed that drones will by default catch up to Icarus in a not-unreasonable amount of time, even after a warp to another star. So now that the drone-lock-up issue is resolved, drone returns shouldn't be a problem. (I believe when you switch star systems the drone appears in the new star system in the same location it was in in the old star system. Drones do not move when you're not on or around a planet.)

I am going to push a release for these two bug fixes so I can close this issue. Then I'll look into a mechanism to enable/disable clearing in-game.

Crazyht commented 3 years ago

Nice thanks

GreyHak commented 3 years ago

Version 1.1.0 released.