Skip to content

Allow raster bands in Collection's item assets#16

Merged
emmanuelmathot merged 1 commit intostac-extensions:mainfrom
kylebarron:kyle/collection-item-assets
Jun 30, 2021
Merged

Allow raster bands in Collection's item assets#16
emmanuelmathot merged 1 commit intostac-extensions:mainfrom
kylebarron:kyle/collection-item-assets

Conversation

@kylebarron
Copy link
Contributor

This PR modifies the JSON Schema to allow raster band information in a Collection's item_assets key. This is identical to what the EO extension does: https://github.com/stac-extensions/eo/blob/89da8dc305f43cf34980b3d52f3bfbbc4d8c88d4/json-schema/schema.json#L85-L96

@emmanuelmathot emmanuelmathot self-requested a review June 2, 2021 07:50
Copy link
Member

@emmanuelmathot emmanuelmathot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Do you have an example to put in the repo?

@kylebarron
Copy link
Contributor Author

I don't have an example off hand. I could take one of the existing planet or sentinel item examples and convert it to a collection, moving assets to item_assets.

Copy link
Contributor

@m-mohr m-mohr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

I don't think we necessarily need an example, most other extensions also don't have one at hand. It's the normal behavior in STAC anyway and there's nothing specific in raster.

@emmanuelmathot emmanuelmathot merged commit 77c160d into stac-extensions:main Jun 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants