Enable viscous flux contribution at supersonic outlet boundary condition#2829
Enable viscous flux contribution at supersonic outlet boundary condition#2829yiluntam wants to merge 5 commits into
Conversation
Update master
The viscous flux contribution in BC_Supersonic_Outlet was previously commented out due to reported convergence issues. However, for viscous NS simulations this omission is physically incorrect and causes severe convergence problems at outlet boundary cells, which also prevents the use of second-order MUSCL reconstruction. For supersonic flows, downstream disturbances cannot propagate upstream, so the outlet state can be safely extrapolated from the domain interior. Using SetPrimitive(V_domain, V_domain) instead of (V_domain, V_outlet) resolves the convergence problems that motivated the original commenting.
|
@pcarruscag Could you take a look at this when you have time? Thanks! |
f6a58b3 to
092f23a
Compare
798b746 to
cd4bb21
Compare
|
It looks like I left out a line of code, since I had been using version 8.0.1 all along. The code has now compiled successfully. Sorry for uploading the incorrect code earlier. @bigfooted @pcarruscag |
|
Hi, can you add a figure of the residual plot of before and after here, and can you add your testcase to the PR? It looks like none of the existing testcases are using this. |
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
04f04f6 to
82d8e59
Compare
|
Hi @yiluntam, The meshes go into our separate Testcases repository, https://github.com/su2code/TestCases Please add the .cfg file to the existing subdirectory navierstokes/flatplate/ in this PR. Then finally, you can add this case as a new regression case to TestCases/parallel_regression.py |


Proposed Changes
The viscous flux contribution in BC_Supersonic_Outlet was previously commented out due to reported convergence issues. However, for viscous NS simulations this omission is physically incorrect and causes severe convergence problems at outlet boundary cells, which also prevents the use of second-order MUSCL reconstruction.
For supersonic flows, downstream disturbances cannot propagate upstream, so the outlet state can be safely extrapolated from the domain interior. Using SetPrimitive(V_domain, V_domain) instead of (V_domain, V_outlet) resolves the convergence problems that motivated the original commenting.
PR Checklist
pre-commit run --allto format old commits.