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

WIP: implement ocean albedo calculation for UFS #391

Closed
DeniseWorthen opened this issue Jun 22, 2023 · 1 comment · Fixed by #395
Closed

WIP: implement ocean albedo calculation for UFS #391

DeniseWorthen opened this issue Jun 22, 2023 · 1 comment · Fixed by #395

Comments

@DeniseWorthen
Copy link
Collaborator

Currently UFS does not use the ocean albedo calculation. To allow this feature for UFS, I have been working on a feature branch which uses the current med_phases_prep_ocn_custom_cesm (renamed as med_phases_prep_ocn_custom) which can be called for either CESM or UFS.

Within the med_phases_ocnalb_mod, I've needed to add configuration parameters for applicability to UFS. These include defining several variables which will default to CESM values if not provided via a configuration.

! used, reused in module
  logical  :: flux_albav      ! use average dif and dir albedos
  logical  :: use_nextswcday  ! use the scalar field for next time (otherwise, will be set using clock)
  logical  :: use_min_albedo  ! apply minimum value of albedo for direct vis, nir
  real(R8) :: min_albedo      ! minimum value of albedo for direct vis, nir
  real(R8) :: albdif          ! 60 deg reference albedo, diffuse
  real(R8) :: albdir          ! 60 deg reference albedo, direct 

The implementation of the ocean albedos is currently being science tested for UFS before the changes will need to come back to ESCOMP. The current changes can be view here. I'd appreciate any feedback.

@DeniseWorthen
Copy link
Collaborator Author

I actually thought that the abldif and albdir constants should be set as local copies of shr constants. I know how to do that for UFS, but I wasn't sure if these might already exist as shr constants in CESM and where they would be.

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 a pull request may close this issue.

1 participant