ASFHyP3 / hyp3-autorift

A HyP3 plugin for feature tracking processing with AutoRIFT-ISCE
BSD 3-Clause "New" or "Revised" License
7 stars 2 forks source link

Metadata fix: comma- to space-seperated list #196

Closed jhkennedy closed 1 year ago

jhkennedy commented 1 year ago

According to CF-Conventions, the flag_meanings attribute should be space separated even though the values should be comma separated for "reasons": https://cfconventions.org/Data/cf-conventions/cf-conventions-1.8/cf-conventions.html#flags

While the conventions also says

If the string contains any commas, it is assumed to be a comma-separated list.

https://cfconventions.org/Data/cf-conventions/cf-conventions-1.8/cf-conventions.html#identification-of-conventions

we should still fix this as tools expect the data to come as a space-separated list.

Pointed out upstream here: https://github.com/nasa-jpl/autoRIFT/pull/74#pullrequestreview-1201516515