-
-
Notifications
You must be signed in to change notification settings - Fork 210
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
Bezier curve interpolated with tension #228
Comments
Hi @maurogzz, thank you for reporting this issue. From your description I have the feeling that there is a misunderstanding of what the purpose of You seem to refer to the concept of tension which is known from Catmull-Rom splines (e.g., https://qroph.github.io/2018/07/30/smooth-paths-using-catmull-rom-splines.html and https://link.springer.com/article/10.1007/s42979-021-00770-x). This is currently not implemented in TinySpline. You can only interpolate Catmull-Rom splines with given |
Thanks, yes you got it right in the first and second case. If you could put 'tension' to the Catmull-Rom spline it would be interesting. |
I added the parameter Please let me know whether this is what you expect from the tension parameter. If it suits your needs, I'll merge the branch into master. |
In case you cannot build the demo application (e.g., due to missing FLTK packages), here is a pre-built executable. It should also work with wine on Linux. |
I am happy with what you have done. |
I can probably fix that. I will try to work on this bug the next days. |
If you build an interpolated curve composed of several bezier splines and apply tension, it modifies the curve which is no longer interpolated but becomes a bezier that connects the first and last points of the original curve losing coincidence with all points intermediate. It would be desirable that applying tension to each individual bezier component so that the interpolated points are all on the resulting curve.
The text was updated successfully, but these errors were encountered: