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

Updated the calculation of the 2-meter temperature, potential temperature, and specfic humidity over seaice points #1147

Merged
merged 3 commits into from
Mar 20, 2024

Conversation

ldfowler58
Copy link
Contributor

In MPAS-v8.0.1, the 2-meter temperature (t2m), potential temperature (th2m), and specific humidity (q2) are computed in subroutine sfcdiags which is called immediately after the call to seaice_noah in subroutine driver_lsm (see mpas_atmphys_driver_lsm.F). This implies that the input variables in subroutine sfcdiags have not yet been weighted by the seaice fraction which in turn impacts the calculation of the 2-meter diagnostics.

In order to have t2m, th2m, and q2 to depend on the seaice fraction, we moved the correction of local variables as functions of the seaice fraction and call to subroutine sfcdiags at the top of subroutine lsm_to_MPAS.

  -> renamed module mpas_atmphys_driver_lsm_shared.F to mpas_atmphys_lsm_shared.F.

  -> in mpas_atmphys_driver_lsm.F, now use mpas_atmphys_lsm_shared to call subroutine
     correct_tsk_over_seaice. removed subroutine correct_tsk_over_seaice at the bottom
     of the module.

  -> corrected Makefile accordingly.
@mgduda mgduda self-requested a review March 13, 2024 16:52
@mgduda
Copy link
Contributor

mgduda commented Mar 13, 2024

Is this a fix that should be included in v8.0.2?

…gh the local variables

  needed in the calculation of t2m, th2m, and q2 prior to updating all LSM variables (see
  subroutine lsm_to_MPAS).
…he call to sfcdiags

  from subroutine driver_lsm to subroutine lsm_to_MPAS. This allows t2m, th2m, and q2 to
  be correctly computed over seaice points.
@ldfowler58 ldfowler58 force-pushed the NSF-MPAS/lsm_sfcdiags branch from 264f970 to 348f404 Compare March 13, 2024 18:18
@ldfowler58
Copy link
Contributor Author

ldfowler58 commented Mar 13, 2024 via email

@mgduda mgduda merged commit 9571d54 into MPAS-Dev:develop Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants