-
Notifications
You must be signed in to change notification settings - Fork 147
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
UFS-dev PR#223 #1095
UFS-dev PR#223 #1095
Conversation
… into feature/gfs_debug_update
… into feature/gfs_debug_update
… into feature/gfs_debug_update
… into feature/gfs_debug_update
…/CLM/clm_lake.f90 and remove OPTIONAL keyword from Fortran code to fix intel 19 runtime issues
… into feature/gfs_debug_update
… into feature/refactor_h2ophys
Land surface upgrades for HR4
Convert GFS DDTs from blocked data structures to contiguous arrays (aka update GFS_debug.F90)
Understood. I can re-request reviews as previous ones get merged to make this easier to review. I understand that it is difficult to review them as they are "stacked" on top of each other. It's just easier for me to do it this way. |
@climbfuji This PR has been updated and should only contain changes from ufs-community#223 |
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.
Very happy to see these photochemistry refactor changes coming to NCAR main.
The MYNN bug fix looks good, too!
if (allocated(h2o_lat) ) deallocate(h2o_lat) | ||
if (allocated(h2o_pres)) deallocate(h2o_pres) | ||
if (allocated(h2o_time)) deallocate(h2o_time) | ||
if (allocated(h2oplin) ) deallocate(h2oplin) |
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.
yay! Stateless CCPP :-)
Identical to ufs-community#223
Note: One small typo fixed in GFS_phys_time_vary.scm.F90