Skip to content

Ryans 2 d mortar smooth#1429

Merged
ebchin merged 96 commits into
developfrom
ryans_2D_mortar_smooth
Jul 1, 2026
Merged

Ryans 2 d mortar smooth#1429
ebchin merged 96 commits into
developfrom
ryans_2D_mortar_smooth

Conversation

@rjl33

@rjl33 rjl33 commented Jul 30, 2025

Copy link
Copy Markdown

2D smoothed integrated Mortar Method

@white238 white238 changed the title Ryans 2 d mortar smooth WIP: Ryans 2 d mortar smooth Jul 30, 2025
@ebchin ebchin changed the title WIP: Ryans 2 d mortar smooth Ryans 2 d mortar smooth Jun 30, 2026
@ebchin ebchin self-assigned this Jun 30, 2026

@tupek2 tupek2 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.

Why is there a patch test in contact_finite_diff.cpp. And, there is a contact_patch_2D.cpp, and an energy_mortar_patch.cpp. But I thought the only 2D contact right now is energy_mortar? Or is that not correct?

@ebchin

ebchin commented Jun 30, 2026

Copy link
Copy Markdown
Member

Why is there a patch test in contact_finite_diff.cpp. And, there is a contact_patch_2D.cpp, and an energy_mortar_patch.cpp. But I thought the only 2D contact right now is energy_mortar? Or is that not correct?

Added a comment to the contact_finite_diff.cpp test. It was originally set up to finite diff a whole Smith solve with contact to find issues with solid_mechanics_contact.hpp integration with functional.

Good catch on the multiple patch tests. I removed contact_patch_2D.cpp.

@chapman39 chapman39 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.

thanks eric

@ebchin ebchin merged commit 4e5a8c0 into develop Jul 1, 2026
17 checks passed
@ebchin ebchin deleted the ryans_2D_mortar_smooth branch July 1, 2026 17:18
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.

4 participants