Skip to content

Fem: Update constraint symbols when moving reference shape - fixes #6093#12527

Merged
chennes merged 2 commits intoFreeCAD:mainfrom
marioalexis84:fem-update_constraint_symbols
Feb 26, 2024
Merged

Fem: Update constraint symbols when moving reference shape - fixes #6093#12527
chennes merged 2 commits intoFreeCAD:mainfrom
marioalexis84:fem-update_constraint_symbols

Conversation

@marioalexis84
Copy link
Member

@chennes
Now the symbols are updated even having the shape within a sequence of nested containers.

some points:

  • FEM is currently a bit inflexible and does not take into account the possibility of working with shapes inside App::Part or PartDesign::Body. This pull request unmasks other bugs that appear during the workflow.

  • This PR is marked as a draft to perform some tests.

  • The boost connection will be replaced by a DocumentObserver.

  • The first commit resolves a bug in Part. SetRotation and SetTranslation convert the transformation into pure rotation and pure translation, respectively, so they are replaced by SetRotationPart and SetTranslationPart.

@FEA-eng Could you do some tests?

@marioalexis84 marioalexis84 marked this pull request as draft February 20, 2024 22:36
@github-actions github-actions bot added Mod: FEM Related to the FEM Workbench Mod: Part Related to the Part Workbench labels Feb 20, 2024
@FEA-eng
Copy link
Contributor

FEA-eng commented Feb 21, 2024

It seems that this fixes the constraint symbol display but the mesh is shown in an old place (where the part was before changing the placement). Same with the results and results mesh. Initially, I also had an error suggesting that the constraint is misplaced but I can’t recreate it now. It can be important because the issue description mentions that not only the symbols were misplaced but also the constraints were not applied properly.

@chennes chennes self-assigned this Feb 21, 2024
@marioalexis84
Copy link
Member Author

the mesh is shown in an old place (where the part was before changing the placement). Same with the results and results mesh

Good news regarding this issue. It's mostly resolved. Works with arbitrary nested containers and arbitrary placement.
I will prepare the complete PR.

image

@marioalexis84 marioalexis84 force-pushed the fem-update_constraint_symbols branch from 1b55e66 to 5938489 Compare February 26, 2024 07:02
@marioalexis84 marioalexis84 marked this pull request as ready for review February 26, 2024 07:04
@marioalexis84
Copy link
Member Author

@FEA-eng could you test?

@FEA-eng
Copy link
Contributor

FEA-eng commented Feb 26, 2024

@FEA-eng could you test?

I tested this and can confirm that it works now.

@chennes chennes merged commit e1ca8c8 into FreeCAD:main Feb 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Mod: FEM Related to the FEM Workbench Mod: Part Related to the Part Workbench

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants