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

Declutter toggle #3394

Closed
eggrobin opened this issue Jul 15, 2022 · 0 comments · Fixed by #3398
Closed

Declutter toggle #3394

eggrobin opened this issue Jul 15, 2022 · 0 comments · Fixed by #3398
Assignees
Labels
Milestone

Comments

@eggrobin
Copy link
Member

eggrobin commented Jul 15, 2022

In #3035, there was a suggestion of an option to hide trajectories; the motivation there was performance, and we have greatly improved that in हरीश चंद्र.
We could improve performance further (we probably shouldn’t update all the meshes all the time), but there is a separate question which is whether we should have an option to hide trajectories; from the conversations on discord at the time,

Perhaps a declutter button may be of some use (even without performance concerns, sometimes there is a bit too much noodliness for readability).

For instance in ECEF (which should become increasingly useful even with long histories or predictions once Σκοπός makes ground tracks matter), the sky covered with the ground tracks of the entire solar system, which are useless.

We have other clutter issues; in particular apsides and node markers get out of hand:

In the reference frame selector, it is possible to pin some celestials of interest.
Map markers can be pinned by right-clicking on them.

We could have a declutter mode wherein all trajectories are hidden except those of celestials pinned in the plotting frame selector, and all map markers are hidden except the pinned ones.

The second part is a bit of a mess though, because the markers don’t have an identity; see #3157 (which effectively suggests mitigating that with a mode with more clutter!).
We could probably do a better job of trying to keep track of markers; right now if the total number of apoapsides on a trajectory changes, we change the currently-displayed periapsis, because the index is among all nodes in the pool.

We could probably improve things quite a bit by having a pool per provenance (which would fit nicely with our cap on the number of nodes being per provenance).
This would probably deal with #2196 once and for all.

@eggrobin eggrobin self-assigned this Jul 15, 2022
@eggrobin eggrobin added ui and removed enhancement labels Jul 16, 2022
This was referenced Jul 16, 2022
@pleroy pleroy added this to the Hilbert milestone Jul 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants