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

[WIP] Aggregation Extension #36

Closed

Conversation

philvarner
Copy link
Collaborator

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:

  • This PR has no breaking changes.
  • I have added my changes to the CHANGELOG or a CHANGELOG entry is not required.
  • API only: I have run npm run generate-all to update the generated OpenAPI files.

@philvarner philvarner changed the base branch from master to dev August 18, 2020 02:56
@philvarner philvarner changed the title Aggregation Extension (WIP) [WIP] Aggregation Extension Aug 18, 2020
@m-mohr
Copy link
Collaborator

m-mohr commented Aug 19, 2020

@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.

@m-mohr m-mohr linked an issue Aug 19, 2020 that may be closed by this pull request
@pomadchin pomadchin mentioned this pull request Aug 20, 2020
4 tasks
@matthewhanson matthewhanson changed the base branch from dev to master October 6, 2020 19:20
@m-mohr m-mohr removed a link to an issue Dec 1, 2020
@m-mohr m-mohr linked an issue Dec 1, 2020 that may be closed by this pull request
| 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 |

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?

@philvarner
Copy link
Collaborator Author

Note to self: add /aggregatables endpoint

@philvarner
Copy link
Collaborator Author

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.

@philvarner philvarner closed this Mar 1, 2022
@philvarner philvarner deleted the pvarner/aggregation-extension branch January 31, 2023 00:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Aggregation extension
3 participants