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

Soil-veg-atm coupling #885

Closed
wants to merge 24 commits into from
Closed

Conversation

wzzheng90
Copy link
Collaborator

Revised sensible heat flux for under-canopy sublayer in Noah-MP land surface model.
-- Modify the eddy diffusivity coefficients for momentum and heat at the top of the canopy.
-- Tune the parameters cd (the drag coefficient of leaves) and lm (mean mixing length).

@grantfirl
Copy link
Collaborator

@wzzheng90 It looks like the changes in this pull request are identical to #873 except for one extra commit: 4284846

Is the intent for this PR to be committed after #873, or would you like to combine them? @HelinWei-NOAA

@grantfirl
Copy link
Collaborator

@wzzheng90 Also, to confirm, this is needed for P8c?

@HelinWei-NOAA
Copy link
Collaborator

HelinWei-NOAA commented Mar 18, 2022 via email

@grantfirl
Copy link
Collaborator

Weizheng was supposed to create PR on my fork but accidentally on the NCAR repository. Can you please delete this PR? Thanks.

On Fri, Mar 18, 2022 at 11:49 AM Grant Firl @.> wrote: @wzzheng90 https://github.com/wzzheng90 It looks like the changes in this pull request are identical to #873 <#873> except for one extra commit: 4284846 <4284846> Is the intent for this PR to be committed after #873 <#873>, or would you like to combine them? @HelinWei-NOAA https://github.com/HelinWei-NOAA — Reply to this email directly, view it on GitHub <#885 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALPHKYDJ27O3SLVYATGYYATVASQXXANCNFSM5RCE4IXA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you were mentioned.Message ID: @.>

Sure, no problem.

@grantfirl grantfirl closed this Mar 18, 2022
@wzzheng90
Copy link
Collaborator Author

wzzheng90 commented Mar 18, 2022 via email

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.

5 participants