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 Initial Conditions and Fix Drag Related Bug #329

Merged
merged 5 commits into from
Mar 16, 2021

Conversation

kylekrol
Copy link
Member

Closes #326.
Relates to #325.

Summary of changes

There ended up being a lot of changes made in this PR (mainly for code cleanliness) but I will try and outline the big takeaways here.

  • Orbital initial conditions were moved from the ECI to ECEF coordinates frame.This was an issue that wasn't fixed when the orbit propagator was moved to the ECEF frame and caused the simulated orbit to be much more elliptical than it should have been.
  • Refactored some of the orbit utility functions to return by value and added more debug information to the orbit and attitude orbit models like altitude and acceleration readouts.
  • Other general structural and syntactical improvements.

Testing

The following plots were generated with:

python -m psim -s 8000000 -p truth/orbit_long_duration -ps 10000 -c sensors/base,truth/base,truth/deployment -v SingleOrbitGnc

and notably show monotonically decrasing orbital energy, decaying altitude, and increasing atmospheric density.

altitude
density
energy

@kylekrol kylekrol marked this pull request as ready for review March 13, 2021 20:53
@kylekrol kylekrol merged commit fd518e2 into master Mar 16, 2021
@kylekrol kylekrol deleted the bugfix/drag-model branch March 16, 2021 03:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Drag Model Seems to be Bricking Long Duration Orbit Simulations
3 participants