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

[develop] Add IMS snow cover obs #181

Draft
wants to merge 11 commits into
base: develop
Choose a base branch
from

Conversation

EdwardSnyder-NOAA
Copy link
Collaborator

@EdwardSnyder-NOAA EdwardSnyder-NOAA commented Feb 5, 2025

Description

This PR adds the IMS snow cover obs as an optional obs input. To use the IMS snow cover obs, a new subcomponent was added. This subcomponent built successfully on Hera (/scratch1/NCEPDEV/stmp4/Edward.Snyder/IMS_snowcover_obs/git-repo/land-DA_workflow), and from what I can tell, runs successfully.

There are a number of changes that still need to be made in the prep_obs script, which are denoted by the TODO tags. The main problem is that I cannot find observations to test these code changes. See below for a breakdown of this issue:

Running the standard LND cold start case

  • IMS data doesn't exist in the netcdf format the script expects
  • possible solution is to convert the asc files to netcdf - haven't figured it out yet
  • a solution that sort of worked was to convert the IMS obs to the date of this case (see first bullet) and place restart files for this case that matches what the prep_obs file is looking for (see second bullet). The experiment (see third bullet) ran but returns an empty forecast file, and I don't think the analysis task ran properly.
    • /scratch1/NCEPDEV/stmp4/Edward.Snyder/IMS_snowcover_obs/git-repo/ims_case_data/ims2023121_4km_v1.3.nc
    • /scratch1/NCEPDEV/stmp4/Edward.Snyder/IMS_snowcover_obs/land-DA_update/jedi/restarts/20000104.000000.sfc_data.tile1.nc
    • /scratch1/NCEPDEV/stmp4/Edward.Snyder/IMS_snowcover_obs/git-repo/exp_case/00_96_lnd_coldstart

Running the standard LND cold start case with a newer date

  • if we update the date to a newer date that has IMS snow obs in netcdf format we run into the following problems:
    • need the climate forcing files to run the fcst expt with the newer date (i.e. clmforc.GSWP3*)
    • need the restart files to run the fcst expt with the newer date (i.e. ufs_land_restart*) - don't think this is needed
    • sfc files as mentioned in the previous section's second bullet are still needed

Once the obs issue is resolved and the Land DA system incorporates the IMS obs properly, then the plot_stats script will need to be updated to handle IMS snow cover obs.

Anticipated changes to regression tests:

  • Is baseline change expected ?

Subcomponents involved:

  • apply_incr.fd (NOAA-PSL/land-apply_jedi_incr)
  • ufs_model.fd (ufs-community/ufs-weather-model)
  • UFS_UTILS.fd (ufs-community/UFS_UTILS)
  • none

Linked PR's and Issues:

Testing (for CM's):

  • RDHPCS
    • Hera
    • Orion
    • Hercules
  • CI
    • Completed
  • WE2E
    • Completed
  • PW-Clouds
    • AWS
    • AZURE
    • GCP

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.

1 participant