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

Investigate exactextract for gridded aggregations #275

Open
samlamont opened this issue Oct 14, 2024 · 2 comments
Open

Investigate exactextract for gridded aggregations #275

samlamont opened this issue Oct 14, 2024 · 2 comments
Assignees

Comments

@samlamont
Copy link
Collaborator

samlamont commented Oct 14, 2024

Python bindings are available for the exactextract package which summarizes gridded data to polygons based on fractional pixel coverage.

It seems to fairly well supported/maintained and used by others CIROH, so it might make sense to implement in TEEHR for the sake of consistency.

First we should understand the implications on performance, etc. compared to the current approach.

@samlamont samlamont self-assigned this Oct 14, 2024
@samlamont
Copy link
Collaborator Author

Maybe the fetching methods should be pulled out into their own package?

@mgdenno
Copy link
Contributor

mgdenno commented Oct 18, 2024

Yeah, it would seem to make sense that some of the functionality we have should not be in Teehr. I think we originally added it because you can't evaluate data you don't have,so we needed a way to fetch it. Perhaps we should split it off and make it its own package now that we have a better sense of what the core functionality is. Maybe as part of the upcoming doc updates we should try to document what is and isn't in the core vision.

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

No branches or pull requests

2 participants