-
Notifications
You must be signed in to change notification settings - Fork 5
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
Widths don't seem to be perpendicular to centerline #4
Comments
At a glance it looks like the centerline is jagged and the width lines are made in reference to the centerline so that might be impacting the direction they generate You can try smoothing the centerline but if you include your data I can give it a look in more depth |
Sure, I've retried it with Here's the data: |
So, the width lines are not generated by making a line that is perpendicular to the centerline, but perpendicular to the average slopes of the nearby "Evenly Spaced" centerline points. This is what But looking over your question, I think I could make this more clear and expand the functionality to match your question better. Right now, the centerline that you see is not the value that the width lines are generated from. Instead, on the backend it is using either "Evenly Spaced" or "Smoothed" coordinates to find the width lines. This was done to make the width points generate at an even space from each other along the centerline The Voronoi centerline that you see when using But, if you want to be able to get the width lines generated from the Voronoi instead of the default "Evenly Spaced" (or optional Smoothed) it will likely run into the issue where the Voronoi points are not evenly spaced, so the slopes between them are going to be more dramatic and unpredictable. But for the sake of completeness I will add this functionality. Can you post the code that you are using to generate the |
Thanks for the very detailed explanation. I think I'm generating the
And then for the widths:
|
Yes, it is currently in development with a few other pieces, but if all the tests pass it should be able to go out this weekend |
New release is out with the new flag, but some of the current testing structure for width lines seems to having some trouble, so I'll fix that this week |
Ok, hot fix is up for the bug in v1.4.0. The newest release v1.4.1 is up and seems to working as expected. I'll be running it through more tests this week, but should be able to add the new |
I've attached an image showing part of a centerline-widths plot. I'd like to understand why the widths don't seem to be perpendicular to the centerline. (Please let me know if you'd prefer feedback in another way, I'm not sure this is really an "issue" or a problem with the options I'm passing to
centerline-width
.The text was updated successfully, but these errors were encountered: