-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Positive preserving scheme (SSP & limiter ) #1327
Comments
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: CliMA/ClimaTimeSteppers.jl#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? |
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,
The text was updated successfully, but these errors were encountered: