Skip to content

Fem: Fix restoring old Tie constraint and add support for cyclic symmetry - fixes #12155#12289

Merged
chennes merged 1 commit intoFreeCAD:mainfrom
marioalexis84:fem-cyclic_symmetry
Feb 8, 2024
Merged

Fem: Fix restoring old Tie constraint and add support for cyclic symmetry - fixes #12155#12289
chennes merged 1 commit intoFreeCAD:mainfrom
marioalexis84:fem-cyclic_symmetry

Conversation

@marioalexis84
Copy link
Member

@marioalexis84 marioalexis84 commented Feb 8, 2024

Add support for cyclic symmetry.
This also fixes the loading of the old Tie constraint object prior to #12133 since in that commit the existence check for the Adjust property was not added when restoring the object (currently old files are loaded without adding this property).
Normally I would have generated two pull requests, one for the correction and another to add the cyclic symmetry, but since some helper functions are added for use in creating properties and restoring the DocumentObject, I decided to put it together in a single commit.
The idea of the helper functions is to avoid code duplication and string comparison that usually occurs in onDocumentRestored when modifications are made to the properties.

The location of the axis of symmetry is obtained by applying the placement transformation to the Z versor (similar to what is done with the lines in Part module).

Still is needed add some view providers, options to task panel and automatically calculate the axis of symmetry from the planes defined by the surfaces that define the sector angle.

@FEA-eng

@github-actions github-actions bot added the Mod: FEM Related to the FEM Workbench label Feb 8, 2024
@FEA-eng
Copy link
Contributor

FEA-eng commented Feb 8, 2024

@marioalexis84 Thank you very much for implementing this. I tested it on Ubuntu and it's working properly.

Btw. I hope that you don't mind if I ask but could you take a look at this issue too: #10864 ? As explained in the issue description, it seems to be easy to implement (for someone familiar with the source code so not for me, unfortunately). It's just a matter of adding one option to SolverCcxTools and internally replacing element names and SHELL with SOLID in the section definition (thickness is already there). Then ensuring zero Z coordinates for nodes. Just some input file swaps basically. Quite a few people asked for those analyses (especially axisymmetric) recently so it would be great to have them implemented.

@chennes Can you merge this PR ?

@chennes chennes self-assigned this Feb 8, 2024
@chennes chennes merged commit dfbc90d into FreeCAD:main Feb 8, 2024
@marioalexis84
Copy link
Member Author

@FEA-eng the plan is to resolve all bugs/feature requests, #10864 included.

@FEA-eng
Copy link
Contributor

FEA-eng commented Feb 10, 2024

@FEA-eng the plan is to resolve all bugs/feature requests, #10864 included.

That would be more than awesome. I’m so glad you’re doing so much for this workbench. I gave those issues some priority here: FEM Roadmap but of course it’s subjective and totally up to you. I chose #10864 as particularly interesting because it seems to be rather easy to implement while adding a lot of functionality to the FEM workbench.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants