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

Re-enable regression test fv3_gsd_diag_debug / fix bug of using wrong temperature over water in RUC LSM #883

Closed
climbfuji opened this issue Oct 21, 2021 · 2 comments · Fixed by #850 or NCAR/ccpp-physics#749
Labels
bug Something isn't working

Comments

@climbfuji
Copy link
Collaborator

Description

As discussed in length in #821, a bug fix is required to initialize cold start runs with RUC LSM correctly over water points:

diff --git a/physics/sfc_drv_ruc.meta b/physics/sfc_drv_ruc.meta
index 52b76825..692f1712 100644
--- a/physics/sfc_drv_ruc.meta
+++ b/physics/sfc_drv_ruc.meta
@@ -199,7 +199,7 @@
   intent = in
   optional = F
 [tsfc_wat]
-  standard_name = surface_skin_temperature_over_water
+  standard_name = sea_surface_temperature
   long_name = surface skin temperature over water
   units = K
   dimensions = (horizontal_dimension)

This will change the results for all regression tests using RUC LSM.

Once this change is made, re-enable regression test fv3_gsd_diag_debug in rt.conf.

@climbfuji climbfuji added the bug Something isn't working label Oct 21, 2021
@DeniseWorthen
Copy link
Collaborator

DeniseWorthen commented Oct 21, 2021

Do we think this explains the failures for the fv3_gsd test seen in PR #863?

@SMoorthi-emc
Copy link
Contributor

SMoorthi-emc commented Oct 21, 2021 via email

junwang-noaa pushed a commit that referenced this issue Oct 25, 2021
… for CCPP variables and metadata, port model to XSEDE Expanse, switch WW3 from develop to dev/ufs-weather-model (#850)

This PR updates the submodule pointers for fv3atm (GFDL_atmos_cubed_sphere, ccpp-framework, ccpp-physics) and stochastic_physics for the changes described in detail in the associated PRs below:

It also ports the model to the XSede Expanse system (contributed by @MinsukJi-NOAA).

Switch WW3 from develop to dev/ufs-weather-model (i.e. pull #850 into this PR). This changes the results (metadata) for two regression tests, see #850 for more information. No change to input data.

Also: bug fix in run_compile.sh, replace UNIT_TEST with OPNREQ_TEST.

The bug fixes for issue #883 change the results of all regression tests using RUC LSM.

Co-authored-by: MinsukJi-NOAA <minsuk.ji@noaa.gov>
Co-authored-by: aliabdolali <ali.abdolali@noaa.gov>
Co-authored-by: Dom.Heinzeller@noaa.gov <Dom.Heinzeller@v71a1.ncep.noaa.gov>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment