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

Update CelestialRotation to be used for other planets #164

Closed
200km opened this issue Jul 19, 2022 · 0 comments · Fixed by #511
Closed

Update CelestialRotation to be used for other planets #164

200km opened this issue Jul 19, 2022 · 0 comments · Fixed by #511
Assignees
Labels
environment environment calculation major update incompatible API changes priority::medium priority medium

Comments

@200km
Copy link
Member

200km commented Jul 19, 2022

Overview

Update CelestialRotation to be used for other planets

Details

Currently, the CelestialRotion supports the earth rotation only. We need to support other planets and the moon at leaset the Simple rotation case.
We can get the rotation speed and axis from the CSPICE.

Related Issue : #90

Conditions for close

NA

Supplement

NA

Note

NA

@200km 200km added the priority::medium priority medium label Jul 19, 2022
@200km 200km mentioned this issue Jul 19, 2022
3 tasks
@200km 200km added environment environment calculation major update incompatible API changes labels Nov 22, 2022
@200km 200km added this to the Major Update for v7.0.0 milestone Apr 12, 2023
@200km 200km added this to S2E Apr 19, 2023
@200km 200km moved this to Todo in S2E Apr 19, 2023
@200km 200km self-assigned this Oct 2, 2023
@200km 200km moved this from 🔲 Todo to 🏗 In Progress in S2E Oct 2, 2023
This was referenced Oct 9, 2023
@200km 200km linked a pull request Oct 9, 2023 that will close this issue
@github-project-automation github-project-automation bot moved this from 🏗 In Progress to 💮 Done in S2E Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
environment environment calculation major update incompatible API changes priority::medium priority medium
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant