Skip to content

[WIP] Correct spatial momentum calculations for fused links. - #24830

Open
mitiguy wants to merge 1 commit into
RobotLocomotion:masterfrom
mitiguy:calcFusedBodyLinkSpatialMomentum
Open

[WIP] Correct spatial momentum calculations for fused links.#24830
mitiguy wants to merge 1 commit into
RobotLocomotion:masterfrom
mitiguy:calcFusedBodyLinkSpatialMomentum

Conversation

@mitiguy

@mitiguy mitiguy commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

This change is Reviewable

@mitiguy mitiguy added priority: medium status: do not review component: multibody plant MultibodyPlant and supporting code release notes: none This pull request should not be mentioned in the release notes and removed component: multibody plant MultibodyPlant and supporting code labels Aug 5, 2026

@mitiguy mitiguy left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updates spatial momentum calculations so we get correct results whether links are fused together or not.

Changes here:

  • Changes in MultibodyTree::CalcBodiesSpatialMomentumInWorldAboutWo() and parallel changes to comments and code made in related method MultibodyTree:CalcSpatialInertia() -- whose signature includes std::vector.
  • Uses new position and velocity kinematics calculations from PR# 24746.
  • New unit tests in fused_welds_test.cc
  • Some terminology cleanup to support wider use of "link" internally.

This PR is a separately-reviewable piece of the larger fused-body project issue #24350. There are no user-visible changes here unless fusing is enabled (which is "internal only" for now).

Note: To parallel CalcSpatialInertia(), anticipate a future PR to make a renamed CalcBodiesSpatialMomentumInWorldAboutWo() public into MultibodyPlant.

@mitiguy made 1 comment.
Reviewable status: needs platform reviewer assigned, needs at least two assigned reviewers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: medium release notes: none This pull request should not be mentioned in the release notes status: do not review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant