-
Notifications
You must be signed in to change notification settings - Fork 258
Modify the plastic dilation terms to reduce noises in effective viscosity #6546
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
Conversation
|
First, let us see the comparison between the results of the strip footing test and Duretz's (2018) test before and after the modifications in this PR (the computations have also applied the modifications in #6471): It can be observed that before the modifications, some areas near the shear bands have a slightly lower viscosity than the unyielding area. These "noises" in the viscosity field affects the velocity field and eventually lead to incorrect strain rate patterns (see the results of Duretz's test). I have suspected that the "noises" were caused by the modifications in elastic rheology, but after two days' debugging it turns out that the mistake is mine (apologies to Anne). In #6373, the plastic dilation is calculated by the following steps:
This implementation will cause a problem: assume that there are two compositional fields with the same mechanical parameters at one quadrature point. Further, assume that the volume fractions of the two compositional fields are 1/2 and 1/2, and one is yielding, while the other is not, then the left-hand side dilation term will be calculated by |
|
The broken test is the one I added in #6373 for plastic dilation. On a refined grid, the results before and after the modification are: The result is actually improved after modification. So I applied the changes directly. |
gassmoeller
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Yimin this is pretty cool, thank you!
I have one comment on the code (just a request for documentation), and one question: I completely agree the new results look less noisy and more localized. However, it looks like the Duretz picture you posted has even more pronounced and spatially limited shear bands than the picture in Duretz et al 2018 (I am looking at figure 5e for example). So my question is: Do we even expect such a localized shear band? I.e. is our result now "better" than the one in Duretz or just "different" and we do not know which one is the correct result? I am ok merging this PR if you are convinced this result is better, but I wanted to ask what you think about this.
| if (dilation_rhs_term - dilation_lhs_term * in.pressure[i] > 0) | ||
| plastic_dilation->dilation_rhs_term[i] = dilation_rhs_term; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you leave a comment here explaining why you need this if-condition and what the individual if-else branches compute?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have added comments here. Please let me know if I need to expand them in more details.
I am not sure about this question. I remember that I have obtained sharper shear bands with elASPECT too, but I do not have that figure at hand. The following is a similar test obtained by pseudo-transient method (https://arxiv.org/abs/2305.01701): The sharp shear bands do have a problem: the convergence rates of both the linear solver and the linear solver are low. When applying cellwise harmonic averaging to viscosity, the strain rate field looks like |
|
I forgot to say: the reason that the strain field in each "block" cut by the shear bands is nearly constant in my result is perhaps because the material is incompressible (the tests in Duretz et al. (2018) are compressible). |
|
I give up implementing @gassmoeller The result of test1 (which is very similar to test3) produced by elASPECT looks like: |
Thanks, I agree that it is normal to see differences, I was mostly wondering if we have any way to say if the differences are reasonable.
Ah, that is good to know, this explains it well. I will restart the testers and if they are ok we can go forward with this PR. |
|
/rebuild |
gassmoeller
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like the testers are fine. lets merge.






This PR reduces noises in the effective viscosity by modifying the plastic dilation terms. It greatly improves the results of Duretz's (2018) test:
I am preparing to board the plane. I will provide a detailed explaination when I arrive at Davis.