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

net_sfc_heating does not include all ice-ocean heat fluxes for ACCESS #319

Closed
rmholmes opened this issue Jun 1, 2020 · 2 comments
Closed

Comments

@rmholmes
Copy link
Contributor

rmholmes commented Jun 1, 2020

As discussed in an issue on the ACCESS-OM2 repository (COSIMA/access-om2#139), the net_sfc_heating diagnostic is missing the heat fluxes associated with the mass fluxes wfimelt, wfiform and licefw (noting that this last one has been added only recently).

These heat fluxes are included in sfc_hflux_pme through the pme variable:

pme(ii,jj) = (Ice_ocean_boundary%lprec(i,j) + Ice_ocean_boundary%fprec(i,j) &
#if defined(ACCESS_CM) || defined(ACCESS_OM)
! PME is meant to include "melt", in a MOM+SIS configuration it
! is added by the coupler. We add it here.
+ Ice_ocean_boundary%wfimelt(i,j) &
+ Ice_ocean_boundary%wfiform(i,j) &
+ Ice_ocean_boundary%licefw(i,j) &
#endif
- Ice_ocean_boundary%q_flux(i,j))*Grd%tmask(ii,jj,1)
liquid_precip(ii,jj) = Ice_ocean_boundary%lprec(i,j)*Grd%tmask(ii,jj,1)
frozen_precip(ii,jj) = Ice_ocean_boundary%fprec(i,j)*Grd%tmask(ii,jj,1)
evaporation(ii,jj) = -Ice_ocean_boundary%q_flux(i,j)*Grd%tmask(ii,jj,1)

But they are not included in net_sfc_heating:

if(id_net_sfc_heating > 0) then
wrk1_2d(:,:) = 0.0
do j=jsc,jec
do i=isc,iec
wrk1_2d(i,j) = T_prog(index_temp)%conversion*( &
T_prog(index_temp)%stf(i,j) &
+ T_prog(index_temp)%runoff_tracer_flux(i,j) &
+ T_prog(index_temp)%calving_tracer_flux(i,j) &
+ (frozen_precip(i,j)+liquid_precip(i,j)+evaporation(i,j))*T_prog(index_temp)%tpme(i,j) )
enddo
enddo
call diagnose_2d(Time, Grd, id_net_sfc_heating, wrk1_2d(:,:))
endif

A second related issue is that there is no separate mh_flux diagnostic for the heat flux due to melting ice (although this term is correctly included in sfc_hflux_coupler).

I am about to submit a PR that addresses these issues.

@aekiss
Copy link
Collaborator

aekiss commented Jun 10, 2020

thanks @rmholmes, this has been merged - can the issue now be closed?

@rmholmes
Copy link
Contributor Author

Yes I think so!

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

No branches or pull requests

2 participants