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

Ungrib driver is hardcoded to GFS #686

Open
jderrico-noaa opened this issue Jan 23, 2025 · 2 comments
Open

Ungrib driver is hardcoded to GFS #686

jderrico-noaa opened this issue Jan 23, 2025 · 2 comments
Labels
bug Something isn't working

Comments

@jderrico-noaa
Copy link

Expected behavior
The ungrib driver can ungrib files from different sources, e.g. RAP, RRFS, HRRR, etc

Current behavior
The driver is hardcoded to GFS, and returns errors when called with a different data source

Machines affected

To Reproduce
experiment directory using RAP ICs for mpas on Jet:
/lfs5/BMC/wrfruc/jderrico/mpas/dev/conus_rt/ush/conus_test

@jderrico-noaa jderrico-noaa added the bug Something isn't working label Jan 23, 2025
@maddenp-noaa
Copy link
Collaborator

maddenp-noaa commented Jan 24, 2025

It looks like the config YAML key ungrib.gfs_files was named short-sightedly and should be generalized (and similarly some variable names in the driver code, though users wouldn't normally see that), but looking at /lfs5/BMC/wrfruc/jderrico/mpas/dev/conus_rt/ush/conus_test/log/ungrib_2024123000.log, I see

[2025-01-23T22:36:25]  WARNING File /lfs5/BMC/wrfruc/jderrico/mpas/dev/conus_rt/ush/conus_test/2024123000/RAP/rap.t00z.pgrb2.0p25.f000: State: Not Ready (external asset)

while the files in that directory are actually named

$ ls -l /lfs5/BMC/wrfruc/jderrico/mpas/dev/conus_rt/ush/conus_test/2024123000/RAP/
total 905552
-rw-r--r-- 1 Janet.M.Derrico wrfruc 302067750 Dec 30 01:23 rap.t00z.wrfnatf00.grib2
-rw-r--r-- 1 Janet.M.Derrico wrfruc 311164120 Dec 30 01:24 rap.t00z.wrfnatf06.grib2
-rw-r--r-- 1 Janet.M.Derrico wrfruc 314035224 Dec 30 01:26 rap.t00z.wrfnatf12.grib2

The ungrib: section in /lfs5/BMC/wrfruc/jderrico/mpas/dev/conus_rt/ush/conus_test/experiment.yaml has

    gfs_files:
      ...
      path: '{{ user.experiment_dir }}/{{ cycle.strftime("%Y%m%d%H") }}/{{ get_lbcs_data.external_model }}/rap.t{cycle_hour:02d}z.pgrb2.0p25.f{forecast_hour:03d}'

Would changing that path: value to

{{ user.experiment_dir }}/{{ cycle.strftime("%Y%m%d%H") }}/{{ get_lbcs_data.external_model }}/rap.t{cycle_hour:02d}z.wrfnatf{forecast_hour:02d}.grib2'

be a quick fix? We should still generalize the YAML key name and driver code, but would the driver make progress with a path: value matching the actual filenames?

maddenp-noaa added a commit to maddenp-noaa/uwtools that referenced this issue Jan 28, 2025
@maddenp-noaa
Copy link
Collaborator

I'm about to merge #688 to generalize what might have been misleading apparent specificity about ungrib and GFS but still don't see a problem not under user control to fix, as described above. But please let me know if I missed something and I'll address it with a separate PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants