-
Notifications
You must be signed in to change notification settings - Fork 17
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
Default time step is too long for high RHi cases #27
Comments
Just to follow up on this. I have conducted my own tests on a 125% RHi ISSL case (as opposed to the 140% RHi ISSL performed by @sdeastham above). Here are the results: Variation of the time-integrated intOD with timestep: Observations The convergence is likely numerical diffusion, as @sdeastham mentioned. I might choose to manually verify this by checking APCEMM's transport equation at a later date. This competes with the pulse-like behaviour induced divergence, which becomes increasingly problematic at higher timesteps. Conclusion I recommend the use of a 5 minute timestep for both ice growth and transport |
The default transport and ice growth time steps are currently 10 minutes, which is too long for high RHi cases. See below - when ice crystals are able to grow large, they settle through many grid cells in a single step. As a result, water in the intervening cells is not taken up, resulting in "pulse-like" behaviour:
Reducing the time step to one minute improves (but does not fully fix) this issue:
Not only is this an imperfect fix, it increases run times significantly (~4x in the given test case, going from 30 seconds to 2 minutes). It would be better to find other fixes (e.g. include simplified ice growth in transport sub-cycling).
The text was updated successfully, but these errors were encountered: