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

Rephrase summaries requirement #1106

Closed
m-mohr opened this issue Apr 27, 2021 · 2 comments · Fixed by #1107
Closed

Rephrase summaries requirement #1106

m-mohr opened this issue Apr 27, 2021 · 2 comments · Fixed by #1107
Assignees
Milestone

Comments

@m-mohr
Copy link
Collaborator

m-mohr commented Apr 27, 2021

From Gitter:

@fredliporace wrote:

Checking the use of summaries. The spec states "Summaries should summarize all values in every Item underneath the collection, including in any nested sub-Catalogs.". If 'all values' is to be interpreted as a requirement this increases significantly the implementation work. For CBERS case I consider that the only field that would be interesting (and easy) to include in summaries is the GSD for the CB4A/WPM camera, which has assets with 2m and 8m.

My response:

It seems we need a PR to rephrase this.
This was meant to say that for the fields you decide to add to summaries, you should list all potential values from all items underneath for that specific field.

@m-mohr m-mohr added this to the 1.0.0-rc.3 milestone Apr 27, 2021
@fredliporace
Copy link
Contributor

@m-mohr I can work on this PR.

@m-mohr
Copy link
Collaborator Author

m-mohr commented Apr 27, 2021

That would be great, @fredliporace ! Thanks.

fredliporace added a commit to fredliporace/stac-spec that referenced this issue Apr 27, 2021
@fredliporace fredliporace mentioned this issue Apr 27, 2021
4 tasks
@m-mohr m-mohr linked a pull request Apr 29, 2021 that will close this issue
4 tasks
@m-mohr m-mohr closed this as completed Apr 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants