opera-adt / COMPASS

COregistered Multi-temPorAl Sar Slc
Apache License 2.0
39 stars 18 forks source link

Correct inconsistencies inside CSLC-S1 metadata #107

Closed vbrancat closed 1 year ago

vbrancat commented 1 year ago

This PR tries to correct some inconsistencies in the allocation of the product metadata. Also, it tries to improve the description of some allocated metadata fields.

vbrancat commented 1 year ago

@scottstanie I have received a comment from some product reviewers that a string is more intuitive than a flag. Personally, I find the flag more intuitive. Since you are closer to the user community than I am, what would you recommend?

scottstanie commented 1 year ago

@scottstanie I have received a comment from some product reviewers that a string is more intuitive than a flag. Personally, I find the flag more intuitive. Since you are closer to the user community than I am, what would you recommend?

If others were confused by it being boolean (maybe it got read as an integer for them or something?) then I don't have any real strong objections. Neither field looks particularly important for usual processing, they're more informational