-
Notifications
You must be signed in to change notification settings - Fork 6
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
max_size parameter for polygonize step #72
Comments
Yeah, that's a great idea. Should be pretty easy to implement. If you have any desire to try out adding it you'd be more than welcome. I coded most of it with Cursor, so AI written contributions are welcome. Any thoughts on what a good default max_size parameter should be? |
I've created a PR for it: #73 |
As you know, this might depend on the particular agricultural regime in question. In my case, I'm filtering out anything beyond 10 ha., as this is the limit of smallholder farm sizes in north-western India. |
@ando-shah sure, please test the PR |
It would be useful to have a
--max_size
parameter (in sq. m) for theftw inference polygonize
command.Motivation: would make it easier to use out-of-the-box, as I see some truly gigantic polygons sometimes that are too big to be agricultural fields in smallholder settings.
The text was updated successfully, but these errors were encountered: