Skip to content

[Diffusion] Explicit MechanicalObject link#6073

Open
GaetanDesrues wants to merge 2 commits intosofa-framework:v25.12from
GaetanDesrues:v25.12
Open

[Diffusion] Explicit MechanicalObject link#6073
GaetanDesrues wants to merge 2 commits intosofa-framework:v25.12from
GaetanDesrues:v25.12

Conversation

@GaetanDesrues
Copy link
Copy Markdown

Adding an explicit link to the MechanicalObject instead of relying on tags


By submitting this pull request, I acknowledge that
I have read, understand, and agree SOFA Developer Certificate of Origin (DCO).


Reviewers will merge this pull-request only if

  • it builds with SUCCESS for all platforms on the CI.
  • it does not generate new warnings.
  • it does not generate new unit test failures.
  • it does not generate new scene test failures.
  • it does not break API compatibility.
  • it is more than 1 week old (or has fast-merge label).

@alxbilger alxbilger changed the title Explicit MechanicalObject link [Diffusion] Explicit MechanicalObject link Apr 8, 2026
@fredroy fredroy added pr: status to review To notify reviewers to review this pull-request pr: clean Cleaning the code labels Apr 8, 2026
Copy link
Copy Markdown
Contributor

@fredroy fredroy left a comment

Choose a reason for hiding this comment

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

I see GaetanDesrues:v25.12 -> sofa-framework:v25.12
I think this PR should be merged into master, and might be backported to v25.12 later (or not)

@bakpaul
Copy link
Copy Markdown
Contributor

bakpaul commented Apr 16, 2026

Hello, all ForceFields are SingleStateAccessor, which are member called mstate and already are a link to the mechanical state. Can you check if this fits your need ?

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

Labels

pr: clean Cleaning the code pr: status to review To notify reviewers to review this pull-request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants