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

Add Equality Constraints ROS Interface #2

Closed
wants to merge 20 commits into from

Conversation

DavidPL1
Copy link
Owner

@DavidPL1 DavidPL1 commented Oct 9, 2023

Test for GHA on PR

Checklist

  • Required by CI: Code is auto formatted using clang-format

DavidPL1 and others added 14 commits September 1, 2023 13:50
* Apply formatting.
* Replace weld constraint relpose quaternion with valid quaternion.
* Add verification of mjNEQDATA, mjNIMP and mjNREF constants.
* Fix EQC id fetching.
* Add failcase strings for more information to gtest EXPECT macros.
* Fix expected content of eq_data for weld constraint.
* Replaced prev quaternion with valid normalized quaternion.
* Fixed wrong access of solimp and solref arrays (missing mult).
* Add verification that parameters change through service call.
* Add string information on failed tests.
* Fix EQ id retrieval.
* Fix switch case values and replaced with enum values for consistency.
* Fix eq_solimp and eq_solref array access (missing mult).
* Fix setting content of eq_data for weld constraints (swapped anchor
and relpose[0:3])
Fixes analogous to Set callback
@codecov-commenter
Copy link

codecov-commenter commented Oct 9, 2023

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@DavidPL1 DavidPL1 closed this Nov 13, 2024
@DavidPL1 DavidPL1 deleted the eq-constraints-interface branch November 13, 2024 17:18
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