-
Notifications
You must be signed in to change notification settings - Fork 4
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
Patchwork of changes to ADC implementation #11
Patchwork of changes to ADC implementation #11
Conversation
Results for change 1. Plots of various terms after 4 days of simulation for "Cooling2" case (-100 W heat flux). Red line = results using code prior to this PR, green line = with Gryanik & Hartmann 2001 changes (commit a98b72f), black line = LES results. Same plots but after only 1 hour of simulation (crashed) for "Wind1" case (tau = 0.1). |
Results for change 2. For both cases, "Cooling2" and "Wind1", there were no discernable differences made by this change. |
Thanks for putting these figures together Kat, it's great to see that the change in w4 seems to improve the w3 profiles significantly! The plots you added are really interesting, and I have a few questions/points that we could discuss later:
|
Results from change 3. For the "Cooling2" case after 4 days. Red line = with the u2 and v2 BC changes (Cmom=Ctherm=4), green line = same previous G&H 2001 lines (code at commit a51d872), black line = LES. Plots with varying Cmom and Ctherm to come... |
Hi Brody,
|
Cool, I agree that these test cases are the most useful ones for identifying issues. I don't see the need for mixed ones yet. You're right, I'm referring to the wstar term in the u2 and v2 boundary conditions. It would be interesting to see whether u2 and v2 improve in the convective case when this term is included, perhaps with 0.3 (or 0.5) coefficients out front. Some of the results after you added the uwsfc boundary condition in your above cases makes sense to me (u2 is non-zero at the surface in the Wind simulation), but other changes don't. For example, why do u2 and v2 profiles change in the Cooling simulation where uwsfc is zero so shouldn't change anything. Did you adjust other parameters? I'm not sure what Cmom and Ctherm are, and whether they were also changed. |
I've got runs going right now that test the inclusion of the wstar term (with 0.5 coefficient out front) for u2 and v2. Should have those results soon. Yes, oops! I forgot to mention that by turning on C_mom and C_therm in the namelist file (they were =0 before), it turns on the 3rd order moments here: MPAS-Model/src/core_ocean/shared/mpas_ocn_adcReconstruct.F Lines 462 to 477 in a96c470
I believe C_mom is equivalent to the S_q in Eq (21) of your ADC w/CL manuscript... If I turn C_mom=C_therm=0, but leave the u2 and v2 terms non-zero, then the cooling case results go back to the G&H2001 results, no change, and the wind case it again crashes at 1hour and the only differences from the G&H2001 results are shown here (removed LES results as they make it hard to see): Hopefully that clarifies things, let me know if it does not. |
Note, @BrodiePearson and @vanroekel, this has been updated: Results with 0.5*wstar^2 added to u2 and v2 boundary condition Wind case appears that there is no effects of adding the wstar term. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approving based on visual inspection and testing by @katsmith133
This PR makes several changes:
Changes the formulation of the transport terms in w3tend to that in Gryanik & Hartmann 2001. This was suggested by Brodie in issue Issues with transport terms in $\overline{w'w'w'}$ budget #7 .
Removes extra 0.5 factor from certain KE terms, as was mentioned as point 1 in issue Some small bugs/questions about second-moment budgets #9 .
Turns on u2 and v2 boundary conditions and changes their formulations.
Will add figures to demonstrate effect of each change shortly...