-
Notifications
You must be signed in to change notification settings - Fork 235
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix Saint-Venant Kirchhoff implementation #677
base: master
Are you sure you want to change the base?
Conversation
The CI output and the issue ought to be discussed in the weekly meeting,
|
@DongWuTUM Could you also have check? Thanks. |
@FabienPean-Virtonomy Hi Fabien, I totally agree with you. The expression Have you checked how much the results differ when using these two constitutive relations? I think the difference is small. |
It seems that all the failed tests are using the shell model. Are the failures all caused by the negative B matrix computed from the corrected Almansi strain, which is based on the linear elastic material assumption? |
The equality must obviously be preserved, a material model has to behave exactly as per its definition no matter the stress measure one asks for. Personally, I would shudder at a world where To the best of my knowledge, the equation implemented for the Cauchy stress in The equivalence @WeiyiVirtonomy Not all, some tests seem to finish without a NaN |
|
The small strain formula is commonly used, as used in SPHinXsys/src/shared/materials/elastic_solid.cpp Lines 81 to 90 in 00815cb
|
StressCauchy
is inherited fromLinearElasticSolid
which is an incorrect formula.This PR fixes that and restores$\mathbf{F}\mathbf{S}\mathbf{F}^T/J = \boldsymbol\sigma$ invariant
Implementation follows https://help.febio.org/Manuals/FEBioTheory/FEBio_tm_3-0-Subsection-5.2.1.html