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

Turn off special FAA grib2 output for ensemble forecasts #333

Closed
BenjaminBlake-NOAA opened this issue Apr 19, 2024 · 4 comments
Closed

Turn off special FAA grib2 output for ensemble forecasts #333

BenjaminBlake-NOAA opened this issue Apr 19, 2024 · 4 comments
Labels
EE2-specific Focused on improving EE2 compliance

Comments

@BenjaminBlake-NOAA
Copy link
Contributor

The following FAA files need the ensemble member number added to the filenames:

rrfs.t${cyc}z.aviati.f${fhr}.conus13km.grib2 -> rrfs.t${cyc}z.m0?.aviati.f${fhr}.conus13km.grib2
rrfs.t${cyc}z.ififip.f${fhr}.ak3km.grib2 -> rrfs.t${cyc}z.m0?.ififip.f${fhr}.ak3km.grib2
rrfs.t${cyc}z.ififip.f${fhr}.conus13km.grib2 -> rrfs.t${cyc}z.m0?.ififip.f${fhr}.conus13km.grib2
rrfs.t${cyc}z.prslevfaa.f${fhr}.conus13km.grib2 -> rrfs.t${cyc}z.m0?.prslevfaa.f${fhr}.conus13km.grib2
rrfs.t${cyc}z.prslevfaa.f${fhr}.na13km.grib2 -> rrfs.t${cyc}z.m0?.prslevfaa.f${fhr}.na13km.grib2
rrfs.t${cyc}z.prslevfaa.f${fhr}.pr32km.grib2 -> rrfs.t${cyc}z.m0?.prslevfaa.f${fhr}.pr32km.grib2

@BenjaminBlake-NOAA BenjaminBlake-NOAA added the EE2-specific Focused on improving EE2 compliance label Apr 19, 2024
@MatthewPyle-NOAA
Copy link
Contributor

@BenjaminBlake-NOAA Hold off on this one, I'm not sure if these files should be generated for the ensemble forecasts.

@MatthewPyle-NOAA
Copy link
Contributor

Have confirmed that we don't need them for ensemble forecasts. So maybe I'll change the title

@MatthewPyle-NOAA MatthewPyle-NOAA changed the title Add ensemble member number to FAA grib2 output Turn off special FAA grib2 output for ensemble forecasts Apr 29, 2024
@BenjaminBlake-NOAA
Copy link
Contributor Author

@MatthewPyle-NOAA Sounds good. It should be a relatively simple change to turn off the FAA processing for ensemble forecasts, so I can create a PR for this one shortly.

@BenjaminBlake-NOAA
Copy link
Contributor Author

This issue was resolved with PR #353 . I will incorporate this simple change into the production/RRFS.v1 branch with my next upcoming PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EE2-specific Focused on improving EE2 compliance
Projects
None yet
Development

No branches or pull requests

2 participants