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

TODO: artifacts.consolidate_nodes() -- MultiLineString avoidance #9

Open
jGaboardi opened this issue Jul 31, 2024 · 1 comment
Open
Assignees

Comments

@jGaboardi
Copy link
Collaborator

jGaboardi commented Jul 31, 2024

Need to avoid MultiLineString geometries where possible

neatnet/neatnet/nodes.py

Lines 493 to 494 in dfb9290

# TODO: not optimal but avoids some MultiLineStrings but not all
cookies = clusters.buffer(tolerance / 2).convex_hull

neatnet/neatnet/nodes.py

Lines 503 to 505 in dfb9290

# TODO: this may result in MultiLineString - we need to avoid that
# TODO: It is temporarily fixed by that explode in return
geom.iloc[inds] = geom.iloc[inds].difference(cookie)

@jGaboardi jGaboardi transferred this issue from uscuni/simplification Oct 3, 2024
@jGaboardi

This comment was marked as outdated.

@jGaboardi jGaboardi changed the title TODO: core.algorithms.simplfy.consolidate_nodes() -- several TODO: artifacts.consolidate_nodes() -- MultiLineString avoidance Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants