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

Reduce Bend enhancement suggestion - provide a way to filter more features #17

Open
AlisterH opened this issue Jun 1, 2021 · 2 comments

Comments

@AlisterH
Copy link
Contributor

AlisterH commented Jun 1, 2021

I generated 100mm contours from 1m resolution lidar data for quite a flat site.
They included a lot of small rings that I wanted to get rid of as well as generalising the contours.
Processing the layer with reduce bend I could filter them out by using a large "diameter tolerance". But I wanted to use a smaller tolerance for generalising the rest of the contours that aren't being filtered out.
I worked around it another way - I stored the length of each contour into an attribute and filtering out those shorter than a certain length.
But a great enhancement for reduce bend would be an option to specify one diameter tolerance for generalisation, and a different one for filtering.
Alternatively, perhaps it would be good to have a separate tool that just filters and doesn't generalise.

@Dan-Eli
Copy link
Collaborator

Dan-Eli commented Jun 4, 2021

What do you exactly mean when you say filter in your text. Do you mean delete the feature (including polygon holes) or do you mean not simplifying a feature and output an ungeneralized feature.

Knowing that there are so many possible condition for filtering features depending on the context that it would be difficult to identify all of them (length, area, compactness, ...) Is the idea of a specific or individual diameter tolerance stored in an attribute of the feature would meet such a need (each feature could have a different tolerance).

@AlisterH
Copy link
Contributor Author

Sorry, it seems I wasn't notified of your comment.
I meant to delete the small features, but I guess an option to skip simplifying small features would also be useful.

Yes, storing a tolerance in feature attributes sounds like a very flexible solution.

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