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

stability issue with equal order in fractional step method #26

Open
yuxianglin opened this issue Aug 4, 2019 · 16 comments
Open

stability issue with equal order in fractional step method #26

yuxianglin opened this issue Aug 4, 2019 · 16 comments

Comments

@yuxianglin
Copy link

Hi,

Just curious in the equi-order of velocity and pressure setting in problem/Channel, taylorgreen, will the LBB condition be satisfied? Does the Oasis use any specific stabilization trick?

Shawn

@mikaem
Copy link
Owner

mikaem commented Aug 6, 2019

Hi,

The LBB is only relevant for coupled solvers, there's no need for stabilization tricks for segregated solvers.

@yuxianglin
Copy link
Author

Well, this is not very true, some papers claim equi-order scheme in projection method is not very stable for small time step, see AN OVERVIEW OF PROJECTION METHODS FOR IN-COMPRESSIBLE
FLOWS.

@mikaem
Copy link
Owner

mikaem commented Aug 7, 2019

I am well aware of that paper. And for steady state problems there may be issues I have still not encountered. But for transient problems, like the channel and the Taylor Green, there are no instabilities observed in the solution. See for example the P1P1 results for TG in the Oasis paper. Note that we take care of the pressure nullspace in Oasis by modifying the Krylov solver directly, and with this modification the spurious pressure modes are removed.

@yuxianglin
Copy link
Author

remove pressure nullspace you mean to avoid p and p+c to be the solution when solve a pure neumman problem in pressure poission equation?

@yuxianglin
Copy link
Author

yuxianglin commented Sep 19, 2019

I tried cavity driven flow with velocity degree=1 in Oasis, but I got spurious pressure and velocity, other cases like flow around cylinders, the pressure got wiggles when using equi-order too

Screenshot from 2019-09-19 13-34-43

@mikaem
Copy link
Owner

mikaem commented Sep 19, 2019

Yes, that's a good example of a steady state problem that should not use P1P1. I remember now that I have seen those oscillations as well. For steady state problems I always recommend using a coupled solver.

@yuxianglin
Copy link
Author

For transient problem too, like the flow around cylinder case, the pressure do not show very good result, but velocity does perform well

@mikaem
Copy link
Owner

mikaem commented Sep 19, 2019

I have not seen any instability for the cylinder. But the pressure is only first order so I guess it's only to be expected that velocity should perform better, right?

@yuxianglin
Copy link
Author

yuxianglin commented Sep 19, 2019

Well, the pressure is stable though, but we can see a lot wiggles for P1P1 element compared to the smooth result for TH element, see the above warped pressure screenshot for P1P1 element. This is not something we want if we are interested in boundary traction, for exampled FSI problems.

@mikaem
Copy link
Owner

mikaem commented Sep 19, 2019

I think you can get rid of these wiggles using a different velocity update method, like lumping or weighted gradient. It has to do with how the velocity divergence is projected. I haven't looked at this for quite some time, though, so not sure the non-default methods will work out of the box. Weighted gradient is using fenistools and I'm not sure it has been updated to latest version of fenics.

@mikaem
Copy link
Owner

mikaem commented Sep 20, 2019

I also think you should try a longer timestep. That will probably make the wiggles disappear.

@yuxianglin
Copy link
Author

Well, I tried lump mass or weigted gradient but the wiggle still exist for P1P1 element though. Since I wanted to have more time resolve of the fluid, large dt isn't an option.

@mikaem
Copy link
Owner

mikaem commented Sep 20, 2019

But large dt gets rid of the wiggles, right?

@yuxianglin
Copy link
Author

no, increase dt cann't completely remove the wiggle. Only slightly better

@mikaem
Copy link
Owner

mikaem commented Sep 20, 2019

Which solver is this?

@yuxianglin
Copy link
Author

the cylinder problem with Von Karman vortex street, the solver is NSfracstep with P1P1

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

No branches or pull requests

2 participants