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

Fix regression tests on S4 #842

Closed
DavidHuber-NOAA opened this issue Sep 30, 2021 · 1 comment
Closed

Fix regression tests on S4 #842

DavidHuber-NOAA opened this issue Sep 30, 2021 · 1 comment
Labels
bug Something isn't working

Comments

@DavidHuber-NOAA
Copy link
Collaborator

Description

Five regression tests are failing on S4 as the compare_ncfile.py script is failing to load the netcdf module. This is because the netcdf module is not included in the miniconda distribution that is loaded with ecflow. The tests that fail are control_fhzer, control_wrtGauss_netcdf_parallel, regional_quilt_netcdf_parallel, control_wrtGauss_netcdf_parallel_debug, and hafs_regional_atm.

To Reproduce:

After creating a baseline, rerun the regression tests on S4.

Solution

Load the miniconda/3.8-s4 module after loading ecflow in rt.sh so compare_ncfile.nc can load netcdf.
@DavidHuber-NOAA DavidHuber-NOAA added the bug Something isn't working label Sep 30, 2021
DavidHuber-NOAA added a commit to DavidHuber-NOAA/ufs-weather-model that referenced this issue Sep 30, 2021
@DavidHuber-NOAA
Added python with netcdf to RT on S4. (ufs-community#842)
07d467d
@DavidHuber-NOAA DavidHuber-NOAA mentioned this issue Sep 30, 2021
Closed
15 tasks
junwang-noaa pushed a commit that referenced this issue Oct 18, 2021
@binli2337 @DavidHuber-NOAA
Update CDEPS component and remove DATM component (#845)
7d812f9 
* Added python with netcdf to RT on S4.  (#842)
* remove files related to datm tests and clean up scripts
CDEPS points to a recently updated version

Co-authored-by: David Huber <david.huber@noaa.gov>
@junwang-noaa
Copy link
Collaborator

Code was committed, close issue.

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

Successfully merging a pull request may close this issue.

Fix regression tests on S4 DavidHuber-NOAA/ufs-weather-model
2 participants
@junwang-noaa @DavidHuber-NOAA