crim-ca / mlm-extension

Machine Learning Model STAC Extension (archive, see https://github.com/stac-extensions/mlm)
https://crim-ca.github.io/mlm-extension/
Apache License 2.0
8 stars 0 forks source link

remove comments for online validators #38

Closed rbavery closed 1 month ago

rbavery commented 1 month ago

Description

This removes comments so that the schema passes stricter jsonschema requirements that are common in online jsonschema validators. I think this is a non breaking change since it doesn't affect python validation. I would prefer to keep comments in but would rather be able to see errors neatly presented in online validators.

I did the quick thing in just removing all comments but probably some of these comments can be kept if they don't break jsonschema validation.

Related Issue

37

Type of Change

Checklist

rbavery commented 1 month ago

Thanks for finding and reopening that, I'll close this in favor of #39