Skip to content

Added option for ice on lakes.#13

Merged
climbfuji merged 1 commit into
climbfuji:move_sfc_emis_alb_calculation_to_its_own_scheme_and_z0_composite_consistentfrom
tanyasmirnova:move_sfc_emis_alb_calculation_to_its_own_scheme_and_z0_composite_consistent_tgs
May 14, 2021
Merged

Added option for ice on lakes.#13
climbfuji merged 1 commit into
climbfuji:move_sfc_emis_alb_calculation_to_its_own_scheme_and_z0_composite_consistentfrom
tanyasmirnova:move_sfc_emis_alb_calculation_to_its_own_scheme_and_z0_composite_consistent_tgs

Conversation

@tanyasmirnova
Copy link
Copy Markdown
Collaborator

In the check if there is a fraction of ice in the grid cell, the min_seaice threshold is used for sea ice and min_lakeice is used for lake ice.

@tanyasmirnova tanyasmirnova requested a review from climbfuji as a code owner May 13, 2021 21:30
@climbfuji
Copy link
Copy Markdown
Owner

This looks good to me - @SMoorthi-emc, do you agree?

@@ -109,6 +112,14 @@ subroutine GFS_radiation_surface_run ( &
! Intialize intent(out) variables
sfcalb = 0.0

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is good. However, I can't see the full code here. Is fraco open water fraction? If so, fracl+fraco+fraci=1 right?

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for checking. You are correct, fracl+fraco+fraci=1 for the fractional mask case. Not that in the git diff view, you can expand the sections that are hidden by clicking on the arrows in the blue boxes between the line numbers on the left. I'll merge this into my branch/PR.

@climbfuji climbfuji merged commit c31b1ba into climbfuji:move_sfc_emis_alb_calculation_to_its_own_scheme_and_z0_composite_consistent May 14, 2021
climbfuji pushed a commit that referenced this pull request Dec 2, 2021
remove some false values over the glacial ice
climbfuji pushed a commit that referenced this pull request Nov 7, 2022
NCAR/main PR NCAR#967 (Bugfix and optimization of prognostic closure for the P8 physics suit)
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.

3 participants