Hi,
Is anyone opposed to me removing key value pairs from CdmParser that aren’t mentioned in any published CCSDS CDM standard? I’d like to make it match the blue book a little closer.
While I was working on the CCSDS parsers, I noticed that the CDM parser can parse many additional parameters that are not in any published CCSDS CDM standard that I can find. The CDM parser supports all of the OCM physical parameters, plus many more that don’t seem to be mentioned in any standard.
In the code at [1] I saw a reference to pink book CCSDS 508.0-P-1.0.2. Perhaps that means that CCSDS was considering adding these parameters, but ultimately decided not to? I couldn’t find a copy of that pink book.
The latest CCSDS blue book lists only 8 additional parameters on page 3-9 of [2].
The original MR at [1] is to address issue #940. It involved Giuseppe Cavallaro, @MaximeJ, and @Serrof. Maybe you know why it was originally added or if it’s OK to remove?
Regards,
Evan
[1] Fix Issue #940: Erroneous check when parsing CDMs (!280) · Merge requests · Orekit / Orekit · GitLab