-
Notifications
You must be signed in to change notification settings - Fork 20
fix: workflow 2b nurbs fixes #2137
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
Conversation
add length property
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2137 +/- ##
==========================================
- Coverage 94.87% 94.77% -0.11%
==========================================
Files 141 141
Lines 10294 10307 +13
==========================================
+ Hits 9766 9768 +2
- Misses 528 539 +11 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
@jacobrkerstetter @jonahrb please add tests for the newly inserted code... We should not merge PRs where the code is not being tested. |
Description
changes approximate_curve to interpolate_curve
added copy constructor ability to NURBSCurve class
added length property
Issue linked
Checklist
feat: extrude circle to cylinder
)