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

chgres_cube - vertical velocity bug #626

Closed
GeorgeGayno-NOAA opened this issue Feb 7, 2022 · 3 comments · Fixed by #627
Closed

chgres_cube - vertical velocity bug #626

GeorgeGayno-NOAA opened this issue Feb 7, 2022 · 3 comments · Fixed by #627
Assignees
@GeorgeGayno-NOAA
Labels
bug Something isn't working

Comments

@GeorgeGayno-NOAA
Copy link
Collaborator

When using certain GRIB2 data as input, the vertical velocity must be converted from omega to dzdt. This conversion is controlled by the logical 'conv_omega'. However, that logical is not being set on all MPI tasks. As a result, vertical velocity is only being converted on part of the grid.
@GeorgeGayno-NOAA GeorgeGayno-NOAA added the bug Something isn't working label Feb 7, 2022
@GeorgeGayno-NOAA GeorgeGayno-NOAA self-assigned this Feb 7, 2022
@GeorgeGayno-NOAA
Copy link
Collaborator Author

This is a problem in the following regression tests -

  • 3km.conus.hrrr.gfssdf.grib2
  • 3km.conus.hrrr.newsfc.grib2
  • 13km.conus.rap.grib2

The GFS GRIB2 data does not need to be converted.

GeorgeGayno-NOAA added a commit to GeorgeGayno-NOAA/UFS_UTILS that referenced this issue Feb 7, 2022
@GeorgeGayno-NOAA
Add broadcast of variable 'conv_omega' to all mpi tasks
98b868d 
so the vertical velocity conversion routine (convert_omega)
is call on all tasks.

Fixes ufs-community#626
@GeorgeGayno-NOAA GeorgeGayno-NOAA mentioned this issue Feb 7, 2022
Merged
@GeorgeGayno-NOAA
Copy link
Collaborator Author

Ran the consistency tests on WCOSS-Dell. Three tests failed:

consistency.log02:<<< 3-km CONUS HRRR W/ GFS PHYSICS GRIB2 TEST FAILED. >>>
consistency.log03:<<< 3-km CONUS HRRR W/ GSD PHYSICS AND SFC FROM FILE GRIB2 TEST FAILED. >>>
consistency.log05:<<< 13-km CONUS RAP W/ GSD PHYSICS AND SFC FROM FILE GRIB2 TEST FAILED. >>>

@GeorgeGayno-NOAA
Copy link
Collaborator Author

Files from the failed tests are on Mars: /gpfs/dell1/stmp/George.Gayno/reg-tests/chgres-cube.

The difference between and test and baseline file is 'atm.diff.nc'.

GeorgeGayno-NOAA added a commit that referenced this issue Feb 11, 2022
@GeorgeGayno-NOAA
chgres_cube - Run routine 'convert_omega' on all tasks. (#627)
d1bdd10 
When using certain GRIB2 data as input, the vertical velocity must be converted from omega to dzdt. This conversion is controlled by the logical 'conv_omega'. Ensure that logical is set on all MPI tasks. 

Fixes #626
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@GeorgeGayno-NOAA GeorgeGayno-NOAA

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chgres_cube - Call routine 'convert_omega' on all MPI tasks GeorgeGayno-NOAA/UFS_UTILS
1 participant
@GeorgeGayno-NOAA