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

"Interpolate NURBS Curve" node - FreeCAD implementation #4889

Merged
merged 14 commits into from
Mar 14, 2023

Conversation

rendetto
Copy link
Contributor

@rendetto rendetto commented Feb 6, 2023

This PR implements the interpolation method from FreeCAD/OCCT. It supports standard parametrization metrics, custom knots, per point tangents and can create open and smoothly closed curves. Also it is very fast and stable and can deal with thousands of points.

Here are some examples of the options available:

Interpolate to cyclic curve using Euclidean metric
Curve_Interpolation_Cyclic_Euclidean

Interpolate to open curve with constrained start and end tangents
Curve_Interpolation_EndpointTangents_Euclidean

Interpolate to cyclic curve using full tangent constraints and custom knots
Curve_Interpolation_FullTangents_ExplicitKnots

@rendetto rendetto changed the title "Interpolate NURBS Surface" node - FreeCAD implementation "Interpolate NURBS Curve" node - FreeCAD implementation Feb 6, 2023
@rendetto
Copy link
Contributor Author

@portnov can you please take a look? I finished testing most of the cases it will encounter. So it should be ready for merge.

Thanks

@rendetto
Copy link
Contributor Author

rendetto commented Mar 3, 2023

Oops! I started getting errors after merging master here. What can I do to solve those?

EDIT: Never mind, seems I fixed it :)

@rendetto
Copy link
Contributor Author

rendetto commented Mar 3, 2023

The node is fully operational. If there are no concerns for this PR it can be merged.

@portnov portnov merged commit 2a00eb3 into nortikin:master Mar 14, 2023
@portnov portnov added the NURBS label Mar 14, 2023
@rendetto
Copy link
Contributor Author

@portnov Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants