Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add rededge1,2,3 to common names #19

Open
m-mohr opened this issue Dec 8, 2022 · 1 comment · May be fixed by #22
Open

Add rededge1,2,3 to common names #19

m-mohr opened this issue Dec 8, 2022 · 1 comment · May be fixed by #22
Assignees

Comments

@m-mohr
Copy link
Contributor

m-mohr commented Dec 8, 2022

It's a bit weird that the common name rededge has 3 bands included for S2, which is not ideal if you try to identify bands based on the common names.
The great repository Awesome Spectral Indices splits the rededge into 3 parts: Red Edge 1, 2 and 3.
I'd propose to add these three in addition.

@m-mohr m-mohr changed the title Add redgedge1,2,3 to common names Add rededge1,2,3 to common names Jan 5, 2023
m-mohr added a commit that referenced this issue Jan 9, 2023
@m-mohr m-mohr linked a pull request Jan 9, 2023 that will close this issue
m-mohr added a commit that referenced this issue Jan 9, 2023
@m-mohr m-mohr linked a pull request Jan 9, 2023 that will close this issue
@matthewhanson
Copy link
Member

Discussed in STAC sprint

@emmanuelmathot to create draft PR for v2.0 of this extension which will remove eo:bands for compliance with upcoming STAC 1.1 - see radiantearth/stac-spec#1254

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 a pull request may close this issue.

3 participants