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.

Sign up for GitHub

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

Jump to bottom

Remove cartesian coordinates in SkyRegion model #757

Open
mauch opened this issue Oct 3, 2024 · 0 comments
Open

Remove cartesian coordinates in SkyRegion model #757

mauch opened this issue Oct 3, 2024 · 0 comments
Labels
low priority Issue is not of immediate concern.

Comments

@mauch
Copy link
Contributor

mauch commented Oct 3, 2024

The cartesian (x,y,z) positions of the sky regions are currently calculated and stored in the SkyRegion model. These are never used as far as I can tell so can safely be removed.

Also check they aren't being calculated anywhere else for individual measurements since they are slow to compute.
@mauch mauch added the low priority Issue is not of immediate concern. label Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
low priority Issue is not of immediate concern.
Projects
Pipeline Backlog
Status: To do
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mauch