st1206: implement Local set for SAR motion imagery #247
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation and Context
This is a ST0601 nested local set that we don't currently support.
Resolves #166
Relates #198
Description
Full implementation of ST 1206.1
Also implement ST 1403.2 SARMI Minimum Metadata validation. That is useful because the test tools don't check this part completely.
Also added SARMI to generator example. There are some minor cleanups mixed into that, because I got a new CMITT which showed new problems during test.
Includes initial work on ST 1303 for floating point values. That in turn required extensions to the PrimitiveConverter code, which also got some more unit tests for existing code.
How Has This Been Tested?
There is no public / releasable test data that I know of - asked MISB, but not expecting a positive answer.
I did extend the generator example to output SAR Motion Imagery metadata, and pushed that through CMITT. It passes, but I don't think CMITT is checking all the values, so that is of limited confidence. Better than nothing though.
There is good unit test coverage.
Screenshots (if appropriate):
Types of changes
Checklist: