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 EO extension parsing #72

Closed
notthatbreezy opened this issue May 5, 2020 · 3 comments · Fixed by #92
Closed

Add EO extension parsing #72

notthatbreezy opened this issue May 5, 2020 · 3 comments · Fixed by #92
Assignees
Labels

Comments

@notthatbreezy
Copy link
Contributor

Improvement

The EO extension is helpful when describing multiband imagery and will make tile links more discoverable in the long-run.

Notes + Context

This should probably follow from #51 and once that is done this can follow

Any other thoughts @jisantuc ?

@pomadchin
Copy link
Collaborator

@notthatbreezy 💯 could be a nice demo to show how stac4s works with the Landsat8 STAC

@jisantuc
Copy link
Contributor

jisantuc commented May 5, 2020

I don't have other thoughts on this in particular, but I think it would be good to try out how the validation strategy discussed in azavea/franklin#147 works before we go too far down this path

@pomadchin
Copy link
Collaborator

I was getting familiar with stac4s and decided to create some tests with the API examples, and for tests I used some modified version of LC catalog; hopefully it would be useful as a demonstration of fields that are missing: https://github.com/azavea/stac4s/pull/73/files

Probably we're also interested in a view extension as well (sun angles are not in the eo extension anymore).

@jcahail jcahail added this to the RF Sprint 5/14 - 5/27/2020 milestone May 15, 2020
@jisantuc jisantuc self-assigned this May 20, 2020
@jisantuc jisantuc mentioned this issue May 20, 2020
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants