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

add /statistics, /feature, and /bbox endpoints #30

Merged
merged 9 commits into from
Oct 22, 2024

Conversation

hrodmn
Copy link
Contributor

@hrodmn hrodmn commented Oct 14, 2024

The rasterio backend worked out-of-the-box but the xarray backend required a few small tweaks to which parameters get passed to the reader. The logic for handling options conditionally for each backend is kind of a mess, I would like to clean that up somehow but I tested the /statistics endpoint on HLS data and MUR SST data and it works!

Right now you can pass max_size/height/width to the rasterio backend but those args will be ignored for xarray backend.

Since they are so similar, I added the /bbox and /feature endpoints, too.

I was having some trouble reproducing the exact test behavior from my local environment in GH actions so I switched everything to use uv for installation/running things in CI and it worked the first time.

Resolves #29

@hrodmn hrodmn self-assigned this Oct 14, 2024
@hrodmn hrodmn force-pushed the feature/add-statistics branch from 6fed76e to 426268d Compare October 14, 2024 19:47
@hrodmn hrodmn force-pushed the feature/add-statistics branch from 8e61733 to 8abcf4f Compare October 16, 2024 19:21
@hrodmn hrodmn force-pushed the feature/add-statistics branch from 2f48b64 to cf7c954 Compare October 17, 2024 01:21
@hrodmn hrodmn marked this pull request as ready for review October 17, 2024 01:51
@hrodmn hrodmn requested a review from vincentsarago October 17, 2024 01:54
@hrodmn hrodmn changed the title add /statistics endpoint add /statistics, /feature, and /bbox endpoints Oct 17, 2024
@hrodmn
Copy link
Contributor Author

hrodmn commented Oct 21, 2024

@vincentsarago I know you already approved but do you want to take another quick look since I added the /feature and /bbox endpoints? Thanks!

@hrodmn hrodmn merged commit afe8b6d into develop Oct 22, 2024
4 checks passed
@hrodmn hrodmn deleted the feature/add-statistics branch October 22, 2024 18:46
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.

Add /statistics POST endpoint
2 participants