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

Include all FOV rings in ring summary table #99

Open
jnspitale opened this issue Oct 15, 2024 · 2 comments
Open

Include all FOV rings in ring summary table #99

jnspitale opened this issue Oct 15, 2024 · 2 comments

Comments

@jnspitale
Copy link
Collaborator

Ring columns should now appear in the ring summary table for any rings intersecting the FOV, not just those of the current system.

@rfrenchseti
Copy link
Collaborator

I think we agreed that this wasn't necessary for planets that were very far away even if they're in the FOV.

@jnspitale
Copy link
Collaborator Author

I think that's right. I downgraded it to minor, as I'm not convinced it won't eventually be useful.

@jnspitale jnspitale changed the title All FOV rings should appear in ring summary table Include all FOV rings in ring summary table Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants