Fix/ldg penalty
Adding penalty term to LDGNS and switching on the LDG penalty term.
The coefficient for LDG penalty term is simply set to 1, it should be reviewed later on.
The coefficient for LDGNS is proportional to viscosity/thermal conductivity and inversely proportional to element size.
The LDG has been lightly restructured to allow for a general flux. This allows for a diffusion coefficient different than 1 for UnsteadyDiffusion
, UnsteadyAdvectionDiffusion
and ViscousBurger
, for the AV flux computation to be moved in the AV class, and eventually to merge LDG and LDGNS together.
-
cleanup LDGNS "upwinding" -
cleanup LDG "upwinding" -
move AV flux computation -
convergence study LDG -
convergence study LDGNS -
check shock capturing -
update tests -
update documentation
Merge request reports
Activity
changed milestone to %v5.0.0
added cfs-solver label
added 1 commit
- 4a4270bb - moved naife h scaling to LDGNS to avoid unnecessary comm
added 1 commit
- 735e7c27 - more cleanup, removed namespace indentation in LDGNS
added 1 commit
- 89fb90f6 - light LDG restructuring, AV still need to be fixed
@dmoxey , I need to redo a convergence study to make sure I didn't mess anything, but apart from that and some cosmetic work I think this is done. When you have time perhaps you can have a look?
Edited by Giacomo Castiglioni- Resolved by Dave Moxey