-
Notifications
You must be signed in to change notification settings - Fork 55
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
Consistently evaluate Neumann BC in generalized-alpha method #88
Comments
Yes, the paper states that the applied load should be at I don't think this makes a noticeable difference in the solution or convergence as the applied load is not a function of the state variable and this term doesn't contribute to LHS. In the case of the follower pressure load, the contribution to LHS is evaluated at |
Thanks for that paper! I see that pressure as a state variable for the fluid system is evaluated at I agree that it probably won't make much of a difference. I'm thinking that if this really is an error, then the effect is that we are applying a slightly higher or lower pressure to the solid than we think we are. |
Adding a relevant recent paper A note on the accuracy of the generalized- scheme for the incompressible Navier-Stokes equations. This paper discusses whether to use |
@aabrown100-git Earlier our pressure integration was based on the Bazilevs 2008 paper which is different from Scheme 1 but I changed it to Scheme 2 later. Reg. the pressure boundary condition, let us change it to evaluate it at |
Adding my two cents here, Ingrid and Ju’s paper demonstrates a nice increase in accuracy in their paper (the one that Aaron linked to) and I think it would be great to update svFSI to use this method – this could also be done after the C++ conversion, and could be provided as an option to the user.
From: Vijay Vedula ***@***.***>
Date: Tuesday, October 4, 2022 at 10:03 AM
To: SimVascular/svFSI ***@***.***>
Cc: Subscribed ***@***.***>
Subject: Re: [SimVascular/svFSI] Error in Neumann BC residual contribution in generalized-alpha method? (Issue #88)
@aabrown100-git<https://github.com/aabrown100-git> Earlier our pressure integration was based on the Bazilevs 2008 paper which is different from Scheme 1 but I changed it to Scheme 2 later. Reg. the pressure boundary condition, let us change it to evaluate it at $n+\alpha_f$.
—
Reply to this email directly, view it on GitHub<#88 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ADJGJZ5N55HDEKIQ7NNXVPDWBPQELANCNFSM6AAAAAAQUHAPEU>.
You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>
|
In terms of implementation in svFSI, I see two options. Denoting the Neumann BC value
I'm not sure if there is any meaningful difference between the two, but one may be easier to implement than the other. Option 2 may be more appropriate when the Neumann BC value is obtained from an external LPN code. |
As I understand the generalized-alpha method (see Chung and Hulbert, 1993) if we have, for example, a time-dependent pressure boundary condition, then this BC contributes to the residual vector using the value of pressure at$t_{n+\alpha_f}$ (or $t_{n+1-\alpha_f}$ depending on the definition of $\alpha_f$ ). Looking through the code (starting from $t_{n+1}$ is being used instead. Is this is an error in the code, or else please let me know where I am mistaken.
SETBCNEU()
), it appears the value of pressure atThe text was updated successfully, but these errors were encountered: