-
Notifications
You must be signed in to change notification settings - Fork 69
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Trajectory above surface on Phobos ends in Collision #2237
Comments
I think that you linked to the logs twice but didn't link to the save. It would be useful to get it because from the logs everything is as expected, ie the vessel seems to be killed (which is what we are trying to do after the collision). Now it's a bit embarrassing that we incorrectly detect the collision in this case, but it's not easy to fix: on the one hand we cannot trust KSP to do the right thing, on the other hand we cannot afford to take the body shape into account in the middle of the force computation. |
Oops, my bad. Here's the save: phobos-plunge.zip
Fair enough. This is not really much of an issue for me, I just wanted to visit Phobos and Deimos once. I'll keep away from space potatoes for now. As a workaround , are there any rules where I can and can't safely fly? |
We detect a collision at 90% of the mean radius (this is working fine for just about everything, except the Mars satellites). The mean radius of Deimos is 6.2 km and that of Phobos is 11.08 km. So around 10 km you should be fine. |
We should obtain the min radius from the game and use that instead of our mean radius (a margin like 90%, 95% or 99% would probably be good for safety). |
#2298 obtains the min radius from the game and uses that (with some tolerance) to detect collisions. With that change we were able to land on Phobos. The trajectories will still be missing in some areas close to Phobos (as seen on the picture for the bottom part of the prediction) but that's much harder to fix. |
Closing as trajectories are hidden by mountains after #3035. |
Some trajectories around Phobos end up by Principia detecting collisions.
In this case, when the vessel goes below 2685m "sea level" and 738 m above ground while time warping, it freezes in place. If the timewarp is stopped, the vessel continues moving away but the camera stays in place. The Warning log file reads:
Here is a screenshot of the trajectory:
![screenshot218](https://user-images.githubusercontent.com/173609/60768432-2bc6e500-a0c4-11e9-9589-2a09aa894739.png)
Here is a quicksave file to reacreate the situation: phobos-plunge-logs.zip
Logs: phobos-plunge-logs.zip
The text was updated successfully, but these errors were encountered: