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

hr2 accumulate #70

Merged
merged 17 commits into from
May 1, 2023
Merged

hr2 accumulate #70

merged 17 commits into from
May 1, 2023

Conversation

barlage
Copy link
Collaborator

@barlage barlage commented May 1, 2023

Branch to accumulate all the changes going to HR2

@barlage
Copy link
Collaborator Author

barlage commented May 1, 2023

@HelinWei-NOAA @RongqianYang-NOAA @wzzheng90

Sorry for all the branches here, but I'm going to use this one to accumulate all the changes.

Note there was a previous error in the z0m_hvt_ratio for two veg types fixed with 5c438fd.

@barlage
Copy link
Collaborator Author

barlage commented May 1, 2023

Leaving iopt_z0m = 2 after 5c438fd. Note everything after this will change answers.

@barlage
Copy link
Collaborator Author

barlage commented May 1, 2023

@HelinWei-NOAA @wzzheng90 @RongqianYang-NOAA

I think I'm done with modifications. Note that I found some bug fixes in the NCAR repo that we should probably add.

Also added canopy heat to parameter table for future tuning and removed zvfun/gdx effect from sfcdif3 (in a hacky way for now).

@HelinWei-NOAA HelinWei-NOAA merged commit 4fcd86a into HelinWei-NOAA:hr2_land1 May 1, 2023
HelinWei-NOAA pushed a commit that referenced this pull request Jun 27, 2023
@barlage barlage deleted the hr2_accumulate branch February 2, 2024 15:54
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.

2 participants