Skip to content
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

Add minimum exchange coeff + exponentially decaying diffusion to Frierson functions #3244

Merged
merged 1 commit into from
Aug 22, 2024

Conversation

akshaysridhar
Copy link
Member

@akshaysridhar akshaysridhar commented Aug 16, 2024

Closes #3228
summary_2D.pdf
summary.pdf

edt instantaneous at upper levels shows ~ 0 diffusivity.
Screenshot 2024-08-16 at 12 37 55 PM
Screenshot 2024-08-16 at 12 42 03 PM

@akshaysridhar akshaysridhar force-pushed the as/sandbox branch 2 times, most recently from 4dfe3f0 to aa68b37 Compare August 16, 2024 19:49
@akshaysridhar akshaysridhar requested a review from szy21 August 16, 2024 19:49
src/cache/precomputed_quantities.jl Show resolved Hide resolved
src/cache/precomputed_quantities.jl Outdated Show resolved Hide resolved
src/cache/precomputed_quantities.jl Outdated Show resolved Hide resolved
src/cache/precomputed_quantities.jl Outdated Show resolved Hide resolved
@szy21
Copy link
Member

szy21 commented Aug 17, 2024

I left some minor comments, but feel free to merge it if you don't agree with them. We can talk later. It would be good to try this with the longrun tonight.

@akshaysridhar
Copy link
Member Author

akshaysridhar commented Aug 20, 2024

@szy21 @trontrytel With the latest commit 1998e87

We get the following results, 30d avg at day ~360 (see pdf summary plots below) for the aquaplanet 1M microphysics test ( no topography )

summary_2D.pdf
summary.pdf

@szy21 Ri_c is now the default value; however mean surface windspeed at the 15m level appear to be ~7m/s so empirically this O(10) gust norm is perhaps reasonable (we can revisit this). With h_boundary_layer set to level 1 prior to the column-wise check, our runs blow up during the spinup period within hours of simulation startup presumably due to specific columns where we don't meet the Ri <= Ri_c criterion.

@szy21
Copy link
Member

szy21 commented Aug 20, 2024

Sounds good. I still think h_boundary = 0.1 * domain_top is too high though, and may hide some problems. So the minimum diffusivity when z > h is insufficient to stabilize it. Could you try setting h_boundary to something like 500m and see if it helps?

@akshaysridhar akshaysridhar force-pushed the as/sandbox branch 3 times, most recently from f7062dd to bf6aa85 Compare August 22, 2024 00:02
@akshaysridhar akshaysridhar added this pull request to the merge queue Aug 22, 2024
Merged via the queue into main with commit 6f58037 Aug 22, 2024
10 of 14 checks passed
@akshaysridhar akshaysridhar deleted the as/sandbox branch August 22, 2024 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a lower limit for diffusivity in Frierson diffusion
2 participants