radiantearth / stac-spec

SpatioTemporal Asset Catalog specification - making geospatial assets openly searchable and crawlable
https://stacspec.org
Apache License 2.0
794 stars 178 forks source link

Clarified stac_extensions #1078

Closed m-mohr closed 3 years ago

m-mohr commented 3 years ago

Related Issue(s): #1077

Proposed Changes:

  1. For context see #1077, but it basically makes it more consistent that stac_extensions also allow to list extensions that are used in collection summaries.
  2. Re-phrased the stac_extensions description in all specs.
  3. Removed the wording around "multiple affected parts" as that should be clear from each individual stac_extensions description, which requires to list stac_extensions if they are implemented. That was mostly for the commons extension back in the days, but nowadays had no real good example anymore. The data cube example in collections for example was not really correct.

PR Checklist:

emmanuelmathot commented 3 years ago

Shouldn't we add in the collection spec a clarification that explains that stac_extensions in collection should not declare extensions fields that are only summarized as in example collection https://github.com/radiantearth/stac-spec/blob/master/examples/collection-only/collection.json

m-mohr commented 3 years ago

Good point @emmanuelmathot, although I'm not sure whether we should add extensions to stac_extensions that are summarized in colelctions or not. I think best would be to add them, but then we need to change examples.

m-mohr commented 3 years ago

DON'T MERGE YET!

We should still discuss, whether summaries should have an influence on stac_extensions or not.

m-mohr commented 3 years ago

Agreed on it for future-proofness etc.