Skip to content

Add annulus kernel to focal statistics #125

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

Closed
chase-dwelle opened this issue Aug 31, 2020 · 2 comments
Closed

Add annulus kernel to focal statistics #125

chase-dwelle opened this issue Aug 31, 2020 · 2 comments

Comments

@chase-dwelle
Copy link
Contributor

In addition to the circular kernel, I would like to have the ability to use an annulus as a mask for calculating spatial statistics.

I am dealing with high resolution datasets and radii in the [km] range. Don't think there would be issues with converting from circle->ellipse at larger radii for an annulus, but correct me if I'm wrong.

Happy to contribute this if the authors feel it's useful.

@brendancol
Copy link
Contributor

@chase-dwelle hey thanks for the suggestion

Would love an annulus kernel. We also just generally need more eyes on the focal module and would love more contributors.

@chase-dwelle
Copy link
Contributor Author

Thanks, opened up a PR to add the kernel. Probably worth considering abstracting out kernel generation into its own class or set of functions, though I am not sure what other kernels are popular with geospatial analysis.

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