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

Enable mosaic based stats requests #70

Closed
leothomas opened this issue May 31, 2022 · 2 comments
Closed

Enable mosaic based stats requests #70

leothomas opened this issue May 31, 2022 · 2 comments
Assignees

Comments

@leothomas
Copy link
Contributor

The MosaicTilerFactory enables to request statistics for a searchId (instead of a single COG url or STAC Item Id). This makes it possible to generate stats for an AOI in a single step (as opposed to retrieving a list of COGs within the AOI and then requesting the stats for each).

The danger is that API will need load every file within the AOI into memory in order to generate the stats, which can become very slow/memory intensive with a large number of files. The nature of the VEDA dashboard is such that each stats query is always bounded to a single day, so I assume that there will never be more than a few dozen files being opened.

In order to enable the endpoint, you just have to add:

add_statistics=True

to the MultiBaseTilerFactory instantiation

Again, a big thank you to @vincentsarago for being one step ahead of everyone else and having already implemented the functionality before we even realized we might need it!

@j08lue
Copy link

j08lue commented Nov 23, 2022

Blocked by monitoring implementation.

@ividito
Copy link
Collaborator

ividito commented Dec 13, 2022

Used monitoring tools to assess this endpoint and found some issues with the approach - documented in #125. Closing this issue for now.

@ividito ividito closed this as completed Dec 13, 2022
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 a pull request may close this issue.

3 participants