Open Zhengyu-Huang opened 1 year ago
For the cosine bells case (12 day / 1200 steps, ne = 16): element 23 has smallest averaged q, -5.4674658761003794e-5, all nodal values are negative [-5.064671434859331e-5 -5.501461995104535e-5 -5.501461995104534e-5 -5.4975440065034456e-5; -4.964183135120574e-5 -5.492509505874185e-5 -5.501461995104534e-5 -5.501461995104533e-5; -5.5006230363461435e-5 -5.501461995104534e-5 -5.501461995104534e-5 -5.501461995104534e-5; -5.2596713587859534e-5 -5.501461995104534e-5 -5.2834909059669035e-5 -5.2927768138292654e-5]
For the cosine bells case (120 day / 12000 steps, ne = 16): element 1118 has smallest averaged q,-0.004942804044896136, all nodal values are negative [-0.005017511375060559 -0.005011217573981225 -0.004867585800051765 -0.004691781410777255; -0.004890483207688044 -0.004898234251674527 -0.004922189197276045 -0.0048917399323111605; -0.004915287575676262 -0.00499609889886179 -0.005024308420427279 -0.005024308420427279; -0.004629447065001463 -0.0048151350251337665 -0.004889046868750097 -0.004880251388114655]
Suggestions from Mark Taylor:
I spoke with @Zhengyu-Huang today about these issues. It seems to me that we have a few critical questions to answer:
It seems to me that the two most critical parts of these issues revolve around two points:
The first point can be tested in a script rather quickly by testing limiters with a variety of fields values. For the second point, I was hoping that we added a meta collection utility into ClimaTimeSteppers.jl to improve transparency: https://github.com/CliMA/ClimaTimeSteppers.jl/issues/106.
Are we calling limiters everywhere we need to in CTS? It's a bit difficult to extract this information in a non-invasive way, maybe there's introspection utilities?
@charleskawczynski , @szy21 : Is this issue still active?
@charleskawczynski , @szy21 : Is this issue still active?
We should be able to close this, and all other dycore related issues after the recent modification/refactoring of the dycore.
cosine bells with $q \in [0, 1]$ , the setup in the main is $q \in [0.1, 1]$, so no negative value is observed
extrema with 12 day / 1200 steps: ne = 6 ne = 16 (-4.33710126096385e-22, 0.7604223043784831) (-5.501461995104536e-5, 0.9508024233242366)
extrema with 120 day / 12000 steps: ne = 6 ne = 16 (0.001258494405958857, 0.06971253964379465) (-0.005024378170539984, 0.10875800730425647))
The modified branch is dzh/limiter-issue,