Closed CyprienBosserelle closed 2 years ago
I tested the kurganov damping briefly (f6ed4c9) and got unsatisfying results... Need to look at this again in the future also need to chaeck why Basilisk never implemented the damping corresction
Alternatively the correction could be applied only to output for a particular function. However that will not resolve the issue that the spurious velocity lead to tiny time step. Maybe it need to be applied only when calculating dt...
Also need to try to apply the correction only when calculating dt. the suprious velocity could also be filtered out in a post processing. may be the best of both world here
This paper proposes a solution that is accurate in 1D and may work OK in 2D. https://www.sciencedirect.com/science/article/pii/S0309170818305335
The Buttinger solver fixes some of these issues
In steep slopes with rain on grid lead to high velocities breaking the C-properties of the model. This can be tamed by applying a limiter or a damping factor to the flux term. Vacondio et al. apply a flux damping as uhc=uh*sqrt(2)/sqrt(1+max((1,(hlim/h)^4)); This seem pretty crude to me... but was proposed by kurganov and Petrova (2007).
Cyp