Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@gadomski
Copy link
Contributor

@gadomski gadomski commented Mar 22, 2022

Related Issue(s):

Description: Remove eo bands except for cloud cover.

PR checklist:

  • Code is formatted (run scripts/format).
  • Code lints properly (run scripts/lint).
  • Tests pass (run scripts/test).
  • Documentation has been updated to reflect changes, if applicable.
  • Example STAC Catalog has been updated to reflect changes, if applicable.
  • Changes are added to the CHANGELOG.

@gadomski gadomski requested a review from pjhartzell March 22, 2022 19:45
Copy link
Contributor

@pjhartzell pjhartzell left a comment

Choose a reason for hiding this comment

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

I see cloud cover in items.

@pjhartzell
Copy link
Contributor

Forgot to ask. EO Bands were removed. Is that info being replaced with anything?

@gadomski
Copy link
Contributor Author

EO Bands were removed. Is that info being replaced with anything?

Yeah, it remains in bands.json and is used as COG asset properties, e.g. https://github.com/stactools-packages/modis/blob/main/src/stactools/modis/fragments/21A2/061/bands.json.

@gadomski gadomski merged commit e2ed636 into main Mar 22, 2022
@gadomski gadomski deleted the cloud-cover branch March 22, 2022 22:44
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.

Add cloud cover and other metadata, when available

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.