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

Ft higher order derivatives #16

Merged
merged 6 commits into from
Dec 7, 2023
Merged

Ft higher order derivatives #16

merged 6 commits into from
Dec 7, 2023

Conversation

jzwar
Copy link
Collaborator

@jzwar jzwar commented Nov 17, 2023

Higher order derivatives and bugfix for Fit functions

Check list

Test documentation (delete if not applicable)

  • Build Debug
  • Build Release Mode
  • Executed Tests
  • Build Tests
  • Executed Examples
  • Added new unit test for new feature

Documentation of changes and theory

  • I have updated the relevant documentations, as far as necessary.

Style guide compliance

  • Edited C++ files formatted with clang-format (10.x.x+)

@jzwar jzwar requested a review from j042 November 17, 2023 17:23
j042
j042 previously approved these changes Nov 29, 2023
Copy link
Member

@j042 j042 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice and natural extension of DerivativeWRTParametricDimension, thanks!

src/rational_bezier_spline.inc Outdated Show resolved Hide resolved
src/rational_bezier_spline.inc Outdated Show resolved Hide resolved
src/bezier_spline.inc Show resolved Hide resolved
Copy link
Member

@j042 j042 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thank you!

@jzwar jzwar merged commit d9fdf10 into main Dec 7, 2023
8 checks passed
@jzwar jzwar deleted the ft-higher-order-derivatives branch December 7, 2023 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants