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

Would it be possible to add the capability to allow RUC LSM physics suites to work with internal ice temperature (tiice) #854

Open
MichaelLueken opened this issue Jun 28, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@MichaelLueken
Copy link

Description

When chgres_cube was updated to use fractional grids, internal ice temperature (tiice) was added. While updating the SRW App to the hash that introduced fractional grids, the workflow end to end (WE2E) tests that use RAP and HRRR SDFs began failing while attempting to read tiice from the sfc_data file. Since these SDFs use RUC LSM, the number of vertical layers are set as 9, but tiice only has 2 vertical layers.

Solution

Is it possible to add the ability for SDFs that use RUC LSM to read in the tiice field that is generated in the current chgres_cube?

Alternatives

I've reached out to the UFS_UTILS team to see if they could add the ability to generate v1 coldstart surface files. They noted that continued support of v1 coldstart surface files is deprecated and will be going away. Either removing or replacing the RAP and HRRR SDF WE2E tests is also a possibility for the SRW App moving forward.

If RAP and HRRR SDFs are being deprecated, then I can move forward with removing or replacing the RAP and HRRR SDF WE2E tests.

Testing:

No changes have been made to test. Only the SRW App's WE2E tests have been run and all RAP and HRRR SDF tests have failed in the weather-model with the following output:

NetCDF: Start+count exceeds dimension bound: netcdf_read_data_3d: file:INPUT/sfc_data.nc- variable:tiice

Dependent PRs:

This capability would assist the SRW App with ufs-community/ufs-srweather-app/pull/1089, where we need the chgres_cube to accept RRFS ICs and LBCs.

@MichaelLueken MichaelLueken added the enhancement New feature or request label Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant