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

Multiline does not support sortPoints #199

Open
stanch opened this issue Oct 1, 2024 · 2 comments
Open

Multiline does not support sortPoints #199

stanch opened this issue Oct 1, 2024 · 2 comments

Comments

@stanch
Copy link
Contributor

stanch commented Oct 1, 2024

Now that it supports intersect, this would be very useful.

@stanch
Copy link
Contributor Author

stanch commented Oct 1, 2024

Note that Ray does not support this either, which makes sense. In the context of a Multiline, if there is just one ray and no other shape, I suppose this method should throw. But if there are two rays (on both ends), or a single ray and arcs/segments, then it is valid to order points along them.

@stanch
Copy link
Contributor Author

stanch commented Oct 12, 2024

FWIW the algorithm I am working on no longer requires sorting points. So feel free to close this if you don’t consider it important. I also realized that a multiline can cross itself, which makes this tricky: should the points be sorted according to their first appearance, or should some points appear more than once in the returned array?

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

1 participant