-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Investigate potential issues with duplicate polyline vertices #898
Comments
Also see this thread for the wall geometry. These should be calling PolylinePipeline.removeDuplicates. |
Is this fixed now? I seem to remember some recent pull request in regards to removing duplicates. @bagnell? |
Yes @bagnell's change should have fixed this, but I'm having a hard time finding the pull request. |
OK to close this? |
I was ready to close this but a quick search of the code seems to indicate that we aren't removing duplicates anywhere in Polyline.js or PolylineColleciton.js. I'm also not sure what the performance implications of doing so would be. We may just want to close this anyway until it becomes an issue again @bagnell? |
See this thread, which applies to Linux.
On Windows there seem to be some minor artifacts as well, but the lines are clearly drawn.
The text was updated successfully, but these errors were encountered: