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

Mass, reference area and initial state of Delfi-C3 corrected in all examples #36

Merged

Conversation

alopezrivera
Copy link
Contributor

@alopezrivera alopezrivera commented Nov 2, 2023

Updated the properties of Delfi-C3 in all examples to make them more realistic:

  • Mass: 2.2kg
  • Reference area for aerodynamic and radiation pressure accelerations: calculated based on the average shadow of a convex solid (proof in the Math StackExchange) to be 1/4th of the surface area of Delfi-C3 (0.035m2)
  • Initial state: obtained from the first available Delfi-C3 TLE (from April the 28th, 2008) in space-track.org

@DominicDirkx
Copy link
Member

Thanks for implemeting these changes! Two small comments:

  • I see that you still have lageos mentioned in some of the tle/ephemeris variables. Since lageos is the name of a different satellite, please change this to delfic3
  • For the very first example (Keplerian orbit), I'd like to keep the code as simple as possible, and not have the TLE in there explicitly. Could you (outside of one of the examples) convert the initial state to Keplerian elements, and then hardcode these in the example as before?

@alopezrivera
Copy link
Contributor Author

I've fixed the first point. If I'm not mistaken, the state is manually coded in for the keplerian_satellite_orbit example already. Do you mean another example?

@alopezrivera
Copy link
Contributor Author

I've now manually programmed in the actual initial state of Delfi from the first available TLE, and I've fixed the comment above to say we're manually setting it

@DominicDirkx DominicDirkx merged commit 88a602a into tudat-team:master Nov 3, 2023
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.

None yet

2 participants