su2code/SU2

Barth-Jesperson Implementation

mitchwall85 opened this issue · 0 comments

Is your feature request related to a problem? Please describe.
Per our previous discussion with @pcarruscag, in this pull request, the Barth-Jesperson flux limiter is actually implemented as the Venkat limiter. This ticket is noting the issue so it can be resolved at a later date.

Describe the solution you'd like
An implementation of the BARTH_JESPERSON limiter.

Describe alternatives you've considered
Complete removal of the BARTH_JESPERSON limiter option to avoid confusion in the future.

Additional context
To demonstrate the issue, we have modified the flow over a cylinder test case. The SLOPE_LIMITER_FLOW option was set to "VENKATAKRISHNAN" (VENKAT_LIMITER_COEFF = 0.0), "BARTH_JESPERSON", and "NONE." The results of these are shown below. Both VENKATAKRISHNAN and BARTH_JESPERSON yielded exactly the same results, and NONE was different. These results demonstrate that the limiters are implemented as described above.

Flow over a cylinder test case. Results were extracted along a line at y=0 across the center of the domain:
This issue ticket written in collaboration with @tomtom252525
Limiter_Comp
Momentum_Comp