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

Expose histogram aggregation via API #1076

Closed
maxalbert opened this issue Jul 17, 2019 · 1 comment · Fixed by #2059
Closed

Expose histogram aggregation via API #1076

maxalbert opened this issue Jul 17, 2019 · 1 comment · Fixed by #2059
Labels
enhancement New feature or request FlowAPI Issues related to the FlowKit API

Comments

@maxalbert
Copy link
Contributor

This depends on #1075. It will also require extending the claims structure and claims checks to allow for this new aggregation type.

@maxalbert maxalbert added enhancement New feature or request FlowAPI Issues related to the FlowKit API labels Jul 17, 2019
@greenape
Copy link
Member

greenape commented Sep 2, 2019

So I think the first thing here is that we do not (at this stage) provide a mechanism for getting unsanitised histograms from the API.

As to the claims, I'm wondering if the way to structure this is to effectively bump the current spatial_aggregation claim down a level and have a structure thus:

'daily_location': {'permissions':..., 'aggregations':{'spatial':..., 'histogram':true}}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request FlowAPI Issues related to the FlowKit API
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants