Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Mortar deuce fixups #13392

Merged
merged 4 commits into from May 9, 2019
Merged

Mortar deuce fixups #13392

merged 4 commits into from May 9, 2019

Conversation

lindsayad
Copy link
Member

Refs #13056

@YaqiWang
Copy link
Contributor

YaqiWang commented May 9, 2019

A temp solution or needs to run on 2 processors?

@lindsayad
Copy link
Member Author

This test runs on a higher number of processes but it exodiffs. This is a displaced mesh mortar example. Another one runs fine.

@lindsayad lindsayad changed the title Restrict test to 2 processes. Mortar deuce fixups May 9, 2019
@YaqiWang
Copy link
Contributor

YaqiWang commented May 9, 2019

Something is not working as expected with displaced mesh? I might add more tests and codes in Rattlesnake with the new mortar implementation.

@lindsayad
Copy link
Member Author

Just have some patience. This is one of multiple displaced mesh tests that I had in the mortar PR. I have to focus on fixing some other targets and then I can look in depth at this

@lindsayad
Copy link
Member Author

I definitely want you to be using the new stuff, so I'll make sure it's working well.

@permcody permcody added the PR: Auto Merge Add this label to have CIVET merge on success label May 9, 2019
permcody
permcody previously approved these changes May 9, 2019
View reviewed changes
@moosebuild
Copy link
Contributor

moosebuild commented May 9, 2019 •
edited

Job Documentation on 08a52b6 wanted to post the following:

View the site here

This comment will be updated on new commits.

permcody
permcody previously approved these changes May 9, 2019
View reviewed changes
@permcody permcody merged commit 86789e5 into idaholab:next May 9, 2019
@lindsayad lindsayad deleted the dmgc-fix branch May 9, 2019 23:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@permcody permcody permcody approved these changes

Assignees
No one assigned
Labels
PR: Auto Merge Add this label to have CIVET merge on success PR: Failed but allowed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@lindsayad @YaqiWang @moosebuild @permcody