Skip to content
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

The tolerance and steps settings are not preserved across scene changes #2239

Closed
scimas opened this issue Jul 7, 2019 · 5 comments · Fixed by #2244 or #2254
Closed

The tolerance and steps settings are not preserved across scene changes #2239

scimas opened this issue Jul 7, 2019 · 5 comments · Fixed by #2244 or #2254

Comments

@scimas
Copy link
Contributor

scimas commented Jul 7, 2019

The tolerance and steps settings on the main Principia window reset to the default values (1.0e-2 m and 1.02e3) on every scene change.

@pleroy
Copy link
Member

pleroy commented Jul 11, 2019

Actually this is not a bug but it is confusing.

The tolerance and steps are saved by vessel: it makes sense because different parameters may be appropriate for different vessels. So when a vessel is selected, the parameters for that vessel are displayed, and to the best of my understanding they are properly saved/restored on scene changes. However, when no vessel is selected, the values displayed are the default.

It seems to me that the prediction settings should only be displayed when there is a vessel selected, pretty much like the flight plan. This is going to require some rework of that part of the UI, though.

@scimas
Copy link
Contributor Author

scimas commented Jul 18, 2019

The tolerance and steps are saved by vessel: it makes sense because different parameters may be appropriate for different vessels. So when a vessel is selected, the parameters for that vessel are displayed, and to the best of my understanding they are properly saved/restored on scene changes.

Yes, that is what I was expecting. But as of Fermat, the settings are not being saved per vessel. If you launch a rocket, make changes to said settings, return to KSC and go back to the rocket, the values are reverted to the default ones. I am able to reproduce this reliably with KSP 1.7.2 fresh install, both expansions and only Principia Fermat. The same thing happens with 1.6.1 and Fatou. I didn't check with any more older versions, but I am sure that this is recent. If I remember correctly, Principia would just remember the previous state of the settings, irrespective of the vessel.

In addition to this, the Principia main menu gets stretched to almost the full screen width due to the new version release message. I will open another issue for it. Apologise for the late reply, I haven't been able to play often in the recent days.

@pleroy pleroy reopened this Jul 18, 2019
@pleroy pleroy changed the title The tolerance and steps settings are not presereved across scene changes The tolerance and steps settings are not preserved across scene changes Jul 28, 2019
@scimas
Copy link
Contributor Author

scimas commented Jul 29, 2019

I've discovered another bug (maybe more than 1) that is related to this. Should I post the details here or raise another issue?

@eggrobin
Copy link
Member

eggrobin commented Jul 29, 2019

@scimas: opening a new issue sounds good.

By the way, so as to clarify the exact behaviour that was fixed, the specific issue that you describe in your previous comment,

If you launch a rocket, make changes to said settings, return to KSC and go back to the rocket, the values are reverted to the default ones

and that was fixed by #2254 specifically happened if you changed the settings without then going to map view, i.e., without seeing the effect of the settings (since the prediction is only shown in map view); these changes would only take effect in map view.

Following #2254, the setting change will be persisted even if you do not look at the prediction before the next scene change.

@scimas
Copy link
Contributor Author

scimas commented Sep 8, 2019

The same issue happens on EVA.

  1. Change tolerance and steps settings
  2. (optional) Go to, and come back from, map view
  3. EVA a kerbal, which is I guess considered a new vessel, so the settings are restored to default ones
  4. (optional) Change the tolerance and steps settings
  5. (optional) Go to, and come back from, map view
  6. Board the rocket

The settings have been reverted to default ones for the rocket too. I guess there are just too many edge cases for how this is handled.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants