-
-
Notifications
You must be signed in to change notification settings - Fork 18
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
Celestial body/object trait and module #4
Comments
Maybe should the origin changing computations be done using Isometries, cf. http://nalgebra.org/points_and_transformations/. |
Here's an idea. Maybe put all the origins with their NAIF ID or some kind of unique ID in an Enum. Then, add the possibility of creating a new origin by specifying its ID, e.g.
Hot take: what I dislike here is that the state is NOT associated with a frame and coordinate system. That's gross and not representative, and therefore error prone. However, I'm not sure how to attach a frame to a state knowing that the propator takes a Vector6. Anyhow, my main priority right now is to be able to define these origins and convert between the, I can handle the ossie issues in #9. |
The time span of de436 is actually from 21 Dec 1549 to 25 Jan 2650. I'm definitely going to trim that down. |
I am having issues understanding how to parse the coefficients of DE436s.bsp. I also tried using the transfer format but that isn't easier. I reached out the JPL, and will continue my work on this once I hear back or get more info on how to parse this. |
I think this should be its separate submodule so that each object can be in its own file:
nyx::bodies::Earth;
. This will be quite verbose for sure, but it'll allow for clarity.Just like in
propagators
, the rootmod.rs
should have the trait for the Body. That should include the spherical harmonics of each body, with a possibility of setting those to zero somehow for objects were we don't have that information. It should also be possible to load these harmonics from a file, as is the case in GMAT.Must include:
The text was updated successfully, but these errors were encountered: