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.

Sign up for GitHub

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

Jump to bottom

The default value for FV3GFS_FILE_FMT_ICS and FV3GFS_FILE_FMT_LBCS should no longer be "nemsio" #979

Closed
mkavulich opened this issue Nov 29, 2023 · 0 comments · Fixed by #1169
Closed

The default value for FV3GFS_FILE_FMT_ICS and FV3GFS_FILE_FMT_LBCS should no longer be "nemsio" #979

mkavulich opened this issue Nov 29, 2023 · 0 comments · Fixed by #1169
Assignees
@mkavulich
Labels
enhancement New feature or request

Comments

@mkavulich
Copy link
Collaborator

mkavulich commented Nov 29, 2023 •
edited
Loading

Description

The default value for FV3GFS_FILE_FMT_ICS and FV3GFS_FILE_FMT_LBCS is currently "nemsio". This is an out-dated format that is no longer available operationally (discontinued in March 2021).

Solution

The default should be replaced with one of the other options ("grib2" or "netcdf"). I would suggest "netcdf" as the default since it is the replacement format for native-grid GFS output, but grib2 is also acceptable.

Requirements**

Update the default value for FV3GFS_FILE_FMT_ICS and FV3GFS_FILE_FMT_LBCS to "netcdf" or "grib2".

Acceptance Criteria (Definition of Done)

Update the default value for FV3GFS_FILE_FMT_ICS and FV3GFS_FILE_FMT_LBCS to "netcdf" or "grib2".

Alternative Solutions

Leave the current default, but this is less than ideal, and will only become more obsolete as time moves on
@mkavulich mkavulich added the enhancement New feature or request label Nov 29, 2023
@mkavulich mkavulich self-assigned this Nov 29, 2023
@mkavulich mkavulich changed the title The default value for FV3GFS_FILE_FMT_ICS should no longer be "nemsio" The default value for FV3GFS_FILE_FMT_ICS and FV3GFS_FILE_FMT_LBCS should no longer be "nemsio" Nov 30, 2023
@MichaelLueken MichaelLueken mentioned this issue Dec 17, 2024
Merged
8 tasks
MichaelLueken added a commit that referenced this issue Jan 7, 2025
@MichaelLueken
[develop] Correct documentation issue and close out two issues (#1169)
f04f085 
The documentation has been failing for the last week due to the https://www.ncep.noaa.gov website being unresponsive. This site URL has been replaced with https://www.weather.gov/ncep/.

The following two issues have also been addressed in this PR:

* ufs-case-studies WE2E tests fail on Derecho in get_extrn_ics/lbcs #1144 - The ufs-case-studies tests are now successfully running through get_extrn_ics/lbcs without issue.
* The default value for FV3GFS_FILE_FMT_ICS and FV3GFS_FILE_FMT_LBCS should no longer be "nemsio" #979 - The defaults for FV3GFS_FILE_FMT_ICS and FV3GFS_FILE_FMT_LBCS have been changed from nemsio to netcdf. Additionally, the WE2E test configurations that didn't include FV3GFS_FILE_FMT_ICS and FV3GFS_FILE_FMT_LBCS have had them added.
@MichaelLueken MichaelLueken mentioned this issue Jan 7, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mkavulich mkavulich

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[develop] Correct documentation issue and close out two issues MichaelLueken/ufs-srweather-app
1 participant
@mkavulich