nsidc / granule-metgen

Metadata generator for direct-to-Cumulus era
Other
0 stars 0 forks source link

Create template UMM-G output to be filled with info #43

Closed lisakaser closed 2 days ago

lisakaser commented 1 month ago

1 story point

acceptance criteria: Visual inspection of UMM-G JSON format by Amy

juliacollins commented 4 days ago

@afitzgerrell I've created a pull request for template file review! https://github.com/nsidc/granule-metgen/pull/59 Let me know if you have file formatting suggestions or spy a typo amongst all the { and [.

afitzgerrell commented 2 days ago

The Checksum object group can be removed from the templates. This is another case where I missed editing the examples I added to the UMM-G guidelines to remove it. In our guidelines, it's not included as a recommended attribute since it's optional and potentially adds another place where something can go awry. If, however, this is a subelement that would make generating CNM messages easier, I could add this in the UMM-G guidelines as a DAAC recommendation!!

E.g., from ummg_template.json ~"Checksum": { "Value": "$checksum", "Algorithm": "SHA256"~

To my eyes, everything else checks out in the base template as well as all the spatial and temporal ummg templates.