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

Instability in Hypoelastic Simulations #771

Closed
ChrisZYJ opened this issue Dec 24, 2024 · 3 comments · Fixed by #773
Closed

Instability in Hypoelastic Simulations #771

ChrisZYJ opened this issue Dec 24, 2024 · 3 comments · Fixed by #773

Comments

@ChrisZYJ
Copy link
Contributor

Description

Commit 10b2209 (#727) introduces instability in some hypoelastic simulations compared to previous commit 108805c.

Test Case

  • Simulation: Simple acoustic wave propagation with sound waves from left focused at a centered stone

  • Case file: case_hypo.txt

Step 300 of the simulation (with commit 10b2209):

image

Comparison with the previous commit 108805c (top:108805c7 ; bottom:10b2209):

output.mp4

Environment & Configuration

  • Ubuntu 22.04.4 LTS
  • GNU 11.4.0
  • Using CPU with both debug enabled and disabled
@sbryngelson
Copy link
Member

Unsure on this, perhaps @mrodrig6 can comment or @mcarcana

@mrodrig6
Copy link
Member

@ChrisZYJ Are you running this on two ranks? It might be worth running this on one rank.

If it is ran on 1 rank, we will need to investigate which terms in the hypoelastic formulation are breaking the symmetry.

@ChrisZYJ
Copy link
Contributor Author

ChrisZYJ commented Dec 24, 2024

@mrodrig6 Thanks for getting back to me! The same instability occurs with 1 rank or more ranks.

I'm creating a new PR to revert the changes for just the hypoelasticity module for now, if that's alright. It should not affect your PR in progress.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

3 participants