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
Mortar augmented lagrange #24349
Mortar augmented lagrange #24349
Conversation
7d637de
to
62e0907
Compare
Job Documentation on 4c3b716 wanted to post the following: View the site here This comment will be updated on new commits. |
62e0907
to
32ae4ee
Compare
Is the only test that needs to be regolded. However
Now fails to converge :-o |
ha, looks like something is uninitialized... Update: This is now a consistent exodiff. |
30a6a5c
to
d999271
Compare
modules/contact/src/problems/AugmentedLagrangianContactProblem.C
Outdated
Show resolved
Hide resolved
modules/contact/test/tests/sliding_block/in_and_out/frictional_02_penalty_out.e
Outdated
Show resolved
Hide resolved
d999271
to
8ec7269
Compare
Job Coverage on 4c3b716 wanted to post the following: Framework coverage
Modules coverageContact
Full coverage reportsReports
Warnings
This comment will be updated on new commits. |
8a9049b
to
2fab312
Compare
Ugh, regolding the cylinder test I noticed that the result for a serial run are different (of course we're not testing that) |
2fab312
to
4314bae
Compare
4612e3c
to
1e066b0
Compare
b33cfbc
to
7c4292f
Compare
Job Python 3.8 on 7c4292f : invalidated by @dschwen fails repeatedly |
aeadd24
to
5acf99a
Compare
5acf99a
to
11e7370
Compare
Job Conda MOOSE (ARM Mac) on 11e7370 : invalidated by @recuero |
1 similar comment
Job Conda MOOSE (ARM Mac) on 11e7370 : invalidated by @recuero |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still a few minor things that I think should still be addressed. As we discussed, by far the most important one is getting rid of the default tolerances.
Job Conda MOOSE (ARM Mac) on c9be92e : invalidated by @recuero |
c9be92e
to
4c3b716
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks for addressing my issues.
Follow-on items that we shouldn't forget:
- Rename those "force" variables to "pressure"
- Change the default value of "penalty_multiplier" to 100 in the Contact action
Rewrite penalty mortar formulation using Simo's paper, implement augmented Lagrange for normal contatct.
Refs #24168