-
Notifications
You must be signed in to change notification settings - Fork 47
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
[WIP] Aggregation Extension #36
[WIP] Aggregation Extension #36
Conversation
@matthewhanson and I spoke a bit about this on the sprint and thought /aggregation should just return an extended version of a STAC Collection with summaries. |
| ids | \[string] | Array of Item ids to return. All other filter parameters that further restrict the number of search results (except `next` and `limit`) are ignored | | ||
| collections | \[string] | Array of Collection IDs to include in the search for items. Only Items in one of the provided Collections will be searched | | ||
| aggregations | \[string] | A list of aggregations to compute and return | | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This table appears to be a duplicate of the table here? https://github.com/radiantearth/stac-api-spec/tree/master/item-search#query-parameters-and-fields
For maintenance purposes, I wonder if maybe we want to just reference that table here?
Note to self: add |
Closing this PR in favor of putting this in separate repo. This PR https://github.com/philvarner/aggregation/pull/1/files how has these changes. |
This is a early draft of a spec for an aggregation extension to solicit feedback. This is similar to the Elasticsearch aggregation api and to the similar /aggregate endpoint (https://eod-catalog-svc-prod.astraea.earth/aggregate) that Astraea Earth OnDemand implements.
Related Issue(s): #
Proposed Changes:
PR Checklist:
npm run generate-all
to update the generated OpenAPI files.