Conversation
Signed-off-by: ahcorde <ahcorde@gmail.com>
|
@azeey this might be the right time to try to clean up our friction parameters, per #31 and specifically this comment: #31 (comment) |
I think @ahcorde is pressed for time and needs this for the USD-SDF work. But yeah, we might be able to make time for it in the next couple months. |
oh I didn't look at the labels and thought this was for mujoco. Since this PR is creating an API, I'd like to think a little extra about its design |
my main concern is that the fact that all friction parameters are in the so, do we want to encode the hacky SDFormat structure in our C++ API in libsdformat12 and then deprecate it in libsdformat13? The advantage of that would be that we could get it merged faster, and that it maintains a transparent mapping between the DOM API and the SDFormat data structure. Or do we want to try for a better API now? |
after discussing offline, it sounds reasonable to proceed with this approach for |
Codecov Report
@@ Coverage Diff @@
## sdf12 gazebosim/sdformat#955 +/- ##
=======================================
Coverage 65.38% 65.38%
=======================================
Files 2 2
Lines 26 26
=======================================
Hits 17 17
Misses 9 9 Continue to review full report at Codecov.
|
| errors.insert(errors.end(), err.begin(), err.end()); | ||
| } | ||
|
|
||
| // \todo(nkoenig) Parse the remaining collide properties. |
There was a problem hiding this comment.
Is this TODO needed? If so, can you update it?
There was a problem hiding this comment.
Can you update the username?
Signed-off-by: Alejandro Hernández <ahcorde@gmail.com>
| errors.insert(errors.end(), err.begin(), err.end()); | ||
| } | ||
|
|
||
| // \todo(nkoenig) Parse the remaining collide properties. |
There was a problem hiding this comment.
Can you update the username?
Signed-off-by: Alejandro Hernández <ahcorde@gmail.com>
Signed-off-by: ahcorde ahcorde@gmail.com
🎉 New feature
related with this issue https://github.com/ignitionrobotics/sdformat/issues/954
Summary
Added Friction and ODE classes
Test it
Checklist
codecheckpassed (See contributing)Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining
Signed-off-bymessages.