Skip to content

[GeoMechanicsApplication] add a new oedometer test to the test_element_lab#14403

Merged
Sabine-van-Dijk merged 25 commits intomasterfrom
geo/add-oedometer-test
May 5, 2026
Merged

[GeoMechanicsApplication] add a new oedometer test to the test_element_lab#14403
Sabine-van-Dijk merged 25 commits intomasterfrom
geo/add-oedometer-test

Conversation

@Sabine-van-Dijk
Copy link
Copy Markdown
Contributor

📝 Description

Set up a test for the oedometer experiment. + small changes to the triaxial test.

🆕 Changelog

  • Made a mesh and set the material and project parameters.
  • Implemented an assertion function by Wijtze Pieter.
  • Assert the displacements and normal stresses for the oedometer test.
  • Apply assert function also for the triaxial test.
  • Add a readme file & schematic.

@Sabine-van-Dijk Sabine-van-Dijk requested a review from a team as a code owner April 24, 2026 14:54
@Sabine-van-Dijk Sabine-van-Dijk linked an issue Apr 24, 2026 that may be closed by this pull request
Copy link
Copy Markdown
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.

Hi Sabine, Thank you for this nice first setup. I have a lot of small nagging comments, but am happy with the completeness of test, documentation and picture. It would be nice to also have pictures of stress development in time in the documentation, as this is the way these kind of experiments usually are assessed.

Apparently you have not experienced non-uniform deformations in the test. That is something I fear from past experience and that ( if it starts happening ) should be curable with master-slave constraints that force a uniform displacement field.

Lets await Marjan's comments before starting to change things.
Regards, Wijtze Pieter

"PERMEABILITY_YY": 4.5e-30,
"PERMEABILITY_XY": 0.0,
"DYNAMIC_VISCOSITY": 8.9e-07,
"THICKNESS": 1.0,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

For plane strain and axi-symmetric models, THICKNESS should not be provided.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Removed it.
For which should it be provided?

"move_mesh_flag": false,
"reform_dofs_at_each_step": false,
"block_builder": true,
"reset_displacements": true,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In a 1 stage computation, false would be a more natural value. reset_displacements is used for setting total displacement fields back to 0.0 at the start of a later stage.

indigocoral
indigocoral previously approved these changes Apr 29, 2026
"properties_id": 1,
"Material": {
"constitutive_law": {
"name": "GeoLinearElasticPlaneStrain2DLaw"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@WPK4FEM Does it make sense to have the same material properties for all these lab tests? Sabine has already kindly made the Triaxial test with the Mohr-Coulomb material, and I'm wondering if we can reuse the same material properties for this oedometer test as well? Would Mohr-Coulomb make sense, or would you prefer linear elastic? (Then we can basically have one material file in a common folder and reuse that for all types of lab tests.)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I had a brief discussion with @WPK4FEM about this and he would like to have 2 sets of tests per each lab test: one for linear elastic material and another one for Mohr-Coulomb material. For Triaxial test you already have the Mohr-Coulomb test and for this oedometer test, you have set up a linear elastic one. Perhaps after merging this, you can also add the Mohr-Coulomb model (most likely the same material file you used for the Triaxial test) to this oedometer test and also (the same) linear elastic material for the Triaxial test.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Having shared material parameters files abolutely makes sense. It would enable running all ( triaxial, oedometer, CRS etc ) tests for a material.

"model_part_name": "PorousDomain.Top_load",
"variable_name": "LINE_LOAD",
"active": [false, true, false],
"value": [0.0, 0.0, 0.0],
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

To make it stepwise load function:

Suggested change
"value": [0.0, 0.0, 0.0],
"value": [0.0, -1000.0, 0.0],

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Very nice detailed picture! 👍🏼

End SubModelPartNodes
End SubModelPart

Begin SubModelPart Sides_L_R
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Maybe a more clear name for this would be:

Suggested change
Begin SubModelPart Sides_L_R
Begin SubModelPart Sides_rollers

@indigocoral indigocoral self-requested a review April 29, 2026 22:22
Copy link
Copy Markdown
Contributor

@indigocoral indigocoral left a comment

Choose a reason for hiding this comment

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

Thanks a lot, Sabine, for making this test and also producing the nice graph and nice results checking. Regarding non-uniform deformations that @WPK4FEM mentioned, yes we have not experienced non-uniform deformations in this test. Perhaps this is something to discuss later? :)
Thanks a lot Sabine!

"properties_id": 1,
"Material": {
"constitutive_law": {
"name": "GeoLinearElasticPlaneStrain2DLaw"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I had a brief discussion with @WPK4FEM about this and he would like to have 2 sets of tests per each lab test: one for linear elastic material and another one for Mohr-Coulomb material. For Triaxial test you already have the Mohr-Coulomb test and for this oedometer test, you have set up a linear elastic one. Perhaps after merging this, you can also add the Mohr-Coulomb model (most likely the same material file you used for the Triaxial test) to this oedometer test and also (the same) linear elastic material for the Triaxial test.

@indigocoral indigocoral self-requested a review April 29, 2026 22:39
@Sabine-van-Dijk
Copy link
Copy Markdown
Contributor Author

Sabine-van-Dijk commented May 1, 2026

To do:

  • Discuss value "BULK_MODULUS_FLUID"
    Value was not used in the tests, so I removed it from the material parameter files.
    For the drained triaxial test it is used so we need to make a new issue to look at that.
  • Discuss axisymmetric element type
    For the axisymmetric element type the displacement in "the middle" (near the symmetry axis) is way more than on the outside. This needs to be constrained. --> axisymmetric loading
  • Apply stepwise loading
  • Discuss value of the permeability.
    Value was not used in the tests, so I removed it from the material parameter files.

Copy link
Copy Markdown
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.

Hi Sabine,
Some more small remarks,
Wijtze Pieter

Comment thread applications/GeoMechanicsApplication/tests/test_element_lab.py Outdated
Comment thread applications/GeoMechanicsApplication/tests/test_element_lab.py Outdated
@Sabine-van-Dijk Sabine-van-Dijk merged commit 7585c09 into master May 5, 2026
10 checks passed
@Sabine-van-Dijk Sabine-van-Dijk deleted the geo/add-oedometer-test branch May 5, 2026 15:10
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.

[GeoMechanicsApplication] add oedometer test

3 participants