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

Refine summaries language #1067

Closed
cholmes opened this issue Mar 29, 2021 · 0 comments · Fixed by #1069
Closed

Refine summaries language #1067

cholmes opened this issue Mar 29, 2021 · 0 comments · Fixed by #1069
Assignees
@m-mohr
Milestone
1.0.0-rc.2

Comments

@cholmes
Copy link
Contributor

cholmes commented Mar 29, 2021

The summary definition is a bit ambiguous. We should clarify what exactly can go in it. From @m-mohr:

Okay, I've planned this to be improved for RC2 and add something like "summaries must be compliant with the original types as defined by STAC".

Originally posted by @m-mohr in #1045 (comment)
@cholmes cholmes added this to the 1.0.0-rc.2 milestone Mar 29, 2021
@m-mohr m-mohr mentioned this issue Mar 30, 2021
Merged
4 tasks
m-mohr added a commit that referenced this issue Mar 30, 2021
@m-mohr
Clarify data types for summary values #1067 #1045
97b3423
@m-mohr m-mohr mentioned this issue Mar 30, 2021
Merged
4 tasks
matthewhanson added a commit that referenced this issue Mar 30, 2021
@matthewhanson
Merge pull request #1069 from radiantearth/clarify-summaries-datatypes
c1a40ae 
Clarify data types for summary values #1067 #1045
@m-mohr m-mohr linked a pull request Mar 30, 2021 that will close this issue
Clarify data types for summary values #1067 #1045 #1069
Merged
4 tasks
@m-mohr m-mohr closed this as completed Mar 30, 2021
@cholmes cholmes mentioned this issue Apr 1, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@m-mohr m-mohr

Labels
None yet
Projects
None yet
Milestone
1.0.0-rc.2
Development

Successfully merging a pull request may close this issue.

Clarify data types for summary values #1067 #1045 radiantearth/stac-spec
2 participants
@cholmes @m-mohr