Skip to content

Energy mortar gap enforcement at quadrature points - #234

Merged
ebchin merged 17 commits into
developfrom
feature/ebchin/quad-point-gap
Aug 20, 2026
Merged

Energy mortar gap enforcement at quadrature points#234
ebchin merged 17 commits into
developfrom
feature/ebchin/quad-point-gap

Conversation

@ebchin

@ebchin ebchin commented Aug 6, 2026

Copy link
Copy Markdown
Member

No description provided.

@ebchin
ebchin requested a review from srwopschall August 6, 2026 07:42
@ebchin ebchin self-assigned this Aug 6, 2026

@srwopschall srwopschall left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

First round of comments. Some minor things and some clarification on the design etc. for a discussion.

Comment thread src/tests/shared_par_sparse_mat.cpp
Comment thread src/tests/tribol_energy_mortar_patch.cpp Outdated
Comment thread src/tests/tribol_energy_mortar_patch.cpp Outdated
Comment thread src/tests/tribol_mfem_jacobian.cpp Outdated
Comment thread src/tribol/common/Parameters.hpp Outdated
Comment thread src/tribol/physics/EnergyMortarAdapter.cpp Outdated
Comment thread src/tribol/physics/EnergyMortarAdapter.hpp Outdated
Comment thread src/tribol/physics/EnergyMortarAdapter.hpp Outdated
Comment thread src/tribol/physics/EnergyMortarAdapter.hpp Outdated
Comment thread src/tribol/physics/EnergyMortarAdapter.hpp

@srwopschall srwopschall left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@ebchin - Great review. Make changes for remaining comments.

Comment thread src/tribol/physics/EnergyMortarAdapter.hpp
@ebchin
ebchin merged commit baa93fa into develop Aug 20, 2026
14 checks passed
@ebchin
ebchin deleted the feature/ebchin/quad-point-gap branch August 20, 2026 05:35
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