Skip to content
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

Changed VOIGT_SIZE_2D_AXISYMMETRIC to VOIGT_SIZE_2D_PLANE_STRAIN #12418

Merged
merged 1 commit into from
Jun 4, 2024

Conversation

rfaasse
Copy link
Contributor

@rfaasse rfaasse commented Jun 3, 2024

These two variables point to the same number (see definition), but it is clearer to have the plane strain version in the plane strain stress state (as is done in the other functions in this class).

@rfaasse rfaasse requested review from avdg81 and WPK4FEM June 3, 2024 12:54
Copy link
Contributor

@WPK4FEM WPK4FEM left a comment

Choose a reason for hiding this comment

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

The expected thing.

@rfaasse rfaasse merged commit f0175e3 into master Jun 4, 2024
10 of 11 checks passed
@rfaasse rfaasse deleted the geo/fix-voigt-size-of-plane-strain-stress-state branch June 4, 2024 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants