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.

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

Fix nitsche restart #1106

Merged
merged 5 commits into from
May 1, 2024
Merged

Fix nitsche restart #1106

merged 5 commits into from
May 1, 2024

Conversation

blaisb
Copy link
Contributor

@blaisb blaisb commented Apr 29, 2024

Description of the problem

Description of the solution

  • Fixes the issue by restarting the fully distributed triangulation for the solid correctly.

How Has This Been Tested?

  • Still pending a unit test, WIP

Comments

  • While running this test case I managed to find another bug, which is that the Nitsche model is unable of mesh adaptation once it has been restarted. This bug was there all the time. I will try to figure out something, in the meantime I will try to m erge this PR anyway.

Copy link
Collaborator

@lpsaavedra lpsaavedra left a comment

Choose a reason for hiding this comment

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

So the mixer example that we are testing is still not working, am I correct? Even with mesh adaptation set to none?

source/core/solid_base.cc Outdated Show resolved Hide resolved
source/core/solid_base.cc Outdated Show resolved Hide resolved
@blaisb
Copy link
Contributor Author

blaisb commented Apr 30, 2024

So the mixer example that we are testing is still not working, am I correct? Even with mesh adaptation set to none?
With the new deal.II version coming up it will work without restart
With restart it will take more work, but this will be a follow up PR

@blaisb blaisb merged commit f96ecdd into master May 1, 2024
8 checks passed
@blaisb blaisb deleted the fix_nitsche_restart branch May 1, 2024 11:52
yashuuzi pushed a commit that referenced this pull request May 6, 2024
Description of the problem
Nitsche does not restart when the mesh was generated with simplices. See Nitsche model restart does not work with simplices #275
Description of the solution
Fixes the issue by restarting the fully distributed triangulation for the solid correctly.
How Has This Been Tested?
Still pending a unit test, WIP
Comments
While running this test case I managed to find another bug, which is that the Nitsche model is unable of mesh adaptation once it has been restarted. This bug was there all the time. I will try to figure out something, in the meantime I will try to m erge this PR anyway.

Co-authored-by: Laura Prieto Saavedra <[email protected]>
M-Badri pushed a commit to M-Badri/lethe that referenced this pull request Sep 29, 2024
Description of the problem
Nitsche does not restart when the mesh was generated with simplices. See Nitsche model restart does not work with simplices chaos-polymtl#275
Description of the solution
Fixes the issue by restarting the fully distributed triangulation for the solid correctly.
How Has This Been Tested?
Still pending a unit test, WIP
Comments
While running this test case I managed to find another bug, which is that the Nitsche model is unable of mesh adaptation once it has been restarted. This bug was there all the time. I will try to figure out something, in the meantime I will try to m erge this PR anyway.

Co-authored-by: Laura Prieto Saavedra <[email protected]>
Former-commit-id: f96ecdd
blaisb added a commit that referenced this pull request Sep 30, 2024
Description of the problem
Nitsche does not restart when the mesh was generated with simplices. See Nitsche model restart does not work with simplices #275
Description of the solution
Fixes the issue by restarting the fully distributed triangulation for the solid correctly.
How Has This Been Tested?
Still pending a unit test, WIP
Comments
While running this test case I managed to find another bug, which is that the Nitsche model is unable of mesh adaptation once it has been restarted. This bug was there all the time. I will try to figure out something, in the meantime I will try to m erge this PR anyway.

Co-authored-by: Laura Prieto Saavedra <[email protected]>
Former-commit-id: f96ecdd
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