Python bind CalcCenterOfMassTranslationalVelocity/Acceleration() #21730
Conversation
mitiguy
added
priority: medium
status: do not review
release notes: feature
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
… to velocity test.
|
Per f2f discussion, this line in the message above is the likely cause:
|
…ression testing in Python bindings.
There was a problem hiding this comment.
feature review +@sherm1
Reviewable status: LGTM missing from assignee sherm1(platform), needs at least two assigned reviewers, commits need curation (https://drake.mit.edu/reviewable.html#curated-commits) (waiting on @mitiguy)
mitiguy
changed the title
There was a problem hiding this comment.
Feature
+@SeanCurtis-TRI for platform review per rotation, please
Reviewed 1 of 2 files at r2, 1 of 1 files at r3, all commit messages.
Reviewable status: LGTM missing from assignee SeanCurtis-TRI(platform), commits need curation (https://drake.mit.edu/reviewable.html#curated-commits) (waiting on @mitiguy)
There was a problem hiding this comment.
Reviewed 1 of 2 files at r2, 1 of 1 files at r3, all commit messages.
Reviewable status: commits need curation (https://drake.mit.edu/reviewable.html#curated-commits) (waiting on @mitiguy)
SeanCurtis-TRI
added
the
status: squashing now
There was a problem hiding this comment.
+(status: squashing now)
Reviewable status:
complete! all discussions resolved, LGTM from assignees SeanCurtis-TRI(platform),sherm1(platform)
PR #21710 created C++ code for the new function Multibody::CalcCenterOfMassTranslationalAccelerationInWorld(), which is work towards issue #14269.
This PR creates Python bindings for this new function along with the older C++ function Multibody::CalcCenterOfMassTranslationalVelocityInWorld() (which is missing Python bindings).
This change is