Skip to content

[GeoMechanicsApplication] Made two Python tests more tolerant when comparing results#13579

Merged
avdg81 merged 1 commit intomasterfrom
geo/13577-more-tolerant-python-tests
Jul 1, 2025
Merged

[GeoMechanicsApplication] Made two Python tests more tolerant when comparing results#13579
avdg81 merged 1 commit intomasterfrom
geo/13577-more-tolerant-python-tests

Conversation

@avdg81
Copy link
Contributor

@avdg81 avdg81 commented Jul 1, 2025

📝 Description
To prepare for the update of the GiD post functionality, two Python tests needed a more tolerant comparison. This PR does that by using the delta parameter rather than the places parameter of the assertAlmostEqual method.

@avdg81 avdg81 requested a review from WPK4FEM July 1, 2025 14:31
@avdg81 avdg81 self-assigned this Jul 1, 2025
@avdg81 avdg81 requested a review from a team as a code owner July 1, 2025 14:31
@avdg81 avdg81 added Testing FastPR This Pr is simple and / or has been already tested and the revision should be fast labels Jul 1, 2025
@avdg81 avdg81 added the GeoMechanics Issues related to the GeoMechanicsApplication label Jul 1, 2025
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.

Looked at the failure, this change to an absolute tolerance is understandable for me.

@github-project-automation github-project-automation bot moved this from In progress to Reviewer approved in GeoMechanicsApplication (Deltares) Jul 1, 2025
@avdg81 avdg81 enabled auto-merge (squash) July 1, 2025 15:26
@avdg81 avdg81 merged commit 70adee4 into master Jul 1, 2025
11 checks passed
@avdg81 avdg81 deleted the geo/13577-more-tolerant-python-tests branch July 1, 2025 16:29
@github-project-automation github-project-automation bot moved this from Reviewer approved to Done in GeoMechanicsApplication (Deltares) Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

FastPR This Pr is simple and / or has been already tested and the revision should be fast GeoMechanics Issues related to the GeoMechanicsApplication Testing

Development

Successfully merging this pull request may close these issues.

[GeoMechanicsApplication] Change places to delta when comparing the results of some Python tests

2 participants