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

Compute free-floating Jacobian derivative using batched math operations #330

Merged
merged 7 commits into from
Jan 10, 2025

Conversation

flferretti
Copy link
Collaborator

@flferretti flferretti commented Jan 7, 2025

This PR implements a batched approach for computing the Jacobian derivative, enhancing performance and efficiency in the calculations. Adjustments were made to the input and output representations to streamline the process


📚 Documentation preview 📚: https://jaxsim--330.org.readthedocs.build//330/

@flferretti flferretti requested a review from xela-95 as a code owner January 7, 2025 16:21
@flferretti flferretti self-assigned this Jan 8, 2025
@flferretti flferretti force-pushed the batched_jacobian_derivative branch from 1839965 to 008244d Compare January 8, 2025 08:46
Copy link
Member

@xela-95 xela-95 left a comment

Choose a reason for hiding this comment

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

Thanks @flferretti ! It LGTM!

@flferretti flferretti force-pushed the batched_jacobian_derivative branch 2 times, most recently from 52ab842 to 7d9724e Compare January 8, 2025 12:29
@flferretti
Copy link
Collaborator Author

@CarlottaSartore when you have time, could you please take a look at this? Thanks!

@flferretti flferretti force-pushed the batched_jacobian_derivative branch from 7d9724e to 596f47e Compare January 10, 2025 14:43
@flferretti flferretti enabled auto-merge January 10, 2025 14:44
@flferretti flferretti merged commit 70b23aa into main Jan 10, 2025
24 checks passed
@flferretti flferretti deleted the batched_jacobian_derivative branch January 10, 2025 15:01
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