Fem: Update constraint symbols when moving reference shape - fixes #6093#12527
Merged
chennes merged 2 commits intoFreeCAD:mainfrom Feb 26, 2024
Merged
Conversation
Contributor
|
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. |
Member
Author
1b55e66 to
5938489
Compare
Member
Author
|
@FEA-eng could you test? |
Contributor
I tested this and can confirm that it works now. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

@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?