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.

Sign up for GitHub

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

Jump to bottom

Eo clarifications #974

Closed
wants to merge 17 commits into from
Closed

Eo clarifications #974

wants to merge 17 commits into from

Conversation

cholmes
Copy link
Contributor

@cholmes cholmes commented Feb 6, 2021

Related Issue(s): #960 #822

Proposed Changes:

  1. Updated eo bands descriptions for more clarity
  2. Put in h4's for better linking

PR Checklist:

  • This PR is made against the dev branch (all proposed changes except releases should be against dev, not master).
  • This PR has no breaking changes.
  • a CHANGELOG entry is not required.

cholmes and others added 17 commits January 19, 2021 21:49
@cholmes
first pass at setting a media type
62931ef
@cholmes
added changelog
72ef0ce
@cholmes @m-mohr
Update catalog-spec/catalog-spec.md
f7c73cf 
Co-authored-by: Matthias Mohr <m.mohr@uni-muenster.de>
@cholmes
updates on language for the profile parameter
bcaca69
@cholmes
clean up of the media type list, added stac types
61dea84
@cholmes @m-mohr
Update item-spec/item-spec.md
84dd460 
Co-authored-by: Matthias Mohr <m.mohr@uni-muenster.de>
@cholmes
reworked common media types to mostly go in best practices, added mor…
c38a073 
…e info there
@cholmes @m-mohr
Update item-spec/item-spec.md
ac380e5 
Co-authored-by: Matthias Mohr <m.mohr@uni-muenster.de>
@cholmes
moved main media type section to catalog
60cd563
@cholmes
moved COG/geotiff to best practices, various cleanup
10eae9a
@cholmes
resolved conflicts
f601924
@cholmes
fixed type, redux
e6d95e3
@cholmes
Merge branch 'dev' into add-media-type
ee553b0
@cholmes
removed the stac specific media type
a7e9728
@cholmes
updates from PR comments
68d5510
@cholmes
clarifications on use of eo:bands
476ff86
@cholmes
markdown fix
541f966
@cholmes cholmes mentioned this pull request Feb 6, 2021
Merged
12 tasks
@cholmes
Copy link
Contributor Author

cholmes commented Feb 7, 2021

Ugh, made this branch wrong, it's not clean off dev. Will redo it.

@cholmes cholmes closed this Feb 7, 2021
@cholmes cholmes deleted the eo-clarifications branch February 7, 2021 14:52
@cholmes cholmes mentioned this pull request Feb 9, 2021
Open
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matthewhanson matthewhanson Awaiting requested review from matthewhanson

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@cholmes