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

Context should include gravity fields #89

Open
Tracked by #22
ChristopherRabotin opened this issue Jun 18, 2023 · 3 comments
Open
Tracked by #22

Context should include gravity fields #89

ChristopherRabotin opened this issue Jun 18, 2023 · 3 comments

Comments

@ChristopherRabotin
Copy link
Member

ChristopherRabotin commented Jun 18, 2023

These should be stored with C_nm and S_nm separate from each other, a flag specifying whether the coefficients are normalized, and (tbd) the uncertainty around these coefficients (which will allow tools to generate gravity clones, and provide lossless info from PDS)

@ChristopherRabotin ChristopherRabotin changed the title Context should include gravity fields. These should be stored with C_nm and S_nm separate from each other, a flag specifying whether the coefficients are normalized, and (tbd) the uncertainty around these coefficients (which will allow tools to generate gravity clones, and provide lossless info from PDS) Context should include gravity fields Jun 18, 2023
@ChristopherRabotin
Copy link
Member Author

Note that this cannot be done using a clever heapless structure because there are too many items (it's a triangular number). Instead, this new format should follow a packed array approach, similar to what's in the DAF format. It may be worth making up a new such format, e.g. DAF/GRV with gravity coefficients.

@ChristopherRabotin
Copy link
Member Author

I'm not convinced that a DAF is a reasonable approach here. DAF has a summary with a validity domain, unique ID, etc. But gravity fields are essentially always valid, and one chooses a degree and order. The more I think of this, the more I think it should be separate from ANISE and work only in Nyx.

@ChristopherRabotin
Copy link
Member Author

ChristopherRabotin commented Jun 23, 2024

I'm re-opening this to consider storing the coefficients as a flattened matrix and using the indexing for O(1) access to any coefficients. For platform compatibility, these doubles should be stored in ASN1 using the DataSet structure. Each data should also contain the frame in which it must be applied.
This ticket should compare the size of this proposed serialization to the gunzipped version of the same data. This ticket shall also include a tool to convert typical PDS data into the chosen format.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant