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

bugfix_fhr24snow #460

Merged
merged 2 commits into from
Mar 29, 2024
Merged

Conversation

malloryprow
Copy link
Contributor

@malloryprow malloryprow commented Mar 28, 2024

Pull Request Testing

  • Describe testing already performed for this Pull Request:

This PR has been running daily under my account producing stats for the GFS, IMD, and ECMWF. Plots were generated to show the impact of the changes done in this PR.

snowfall
old_evs global_det fbias_ge0p0254 weasd_a24_z0 last31days fhrmean_valid12z_f240 g211_buk_conus
new_evs global_det fbias_ge0p0254 snod_a24_z0 last31days fhrmean_valid12z_f240 g211_buk_conus
old_evs global_det fbias_ge0p0254 snod_a24_z0 last31days fhrmean_valid12z_f240 g211_buk_conus
new_evs global_det fbias_ge0p0254 weasd_a24_z0 last31days fhrmean_valid12z_f240 g211_buk_conus

  • Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions:

Set-up

  1. Clone my fork and checkout branch bugfix_fhr24snow
  2. ln -sf /lfs/h2/emc/vpppg/noscrub/emc.vpppg/verification/EVS_fix fix

✔️ global_det atmos stat

  1. cd dev/drivers/scripts/stats/global_det
  2. Set COMIN to /lfs/h1/ops/prod/com/evs/v1.0 in jevs_global_det_atmos_ecmwf_grid2grid_stats.sh. Run jevs_global_det_atmos_ecmwf_grid2grid_stats.sh.
  3. Set COMIN to /lfs/h1/ops/prod/com/evs/v1.0 in jevs_global_det_atmos_gfs_grid2grid_stats.sh. Run jevs_global_det_atmos_gfs_grid2grid_stats.sh.
  4. Set COMIN to /lfs/h1/ops/prod/com/evs/v1.0 in jevs_global_det_atmos_imd_grid2grid_stats.sh. Run jevs_global_det_atmos_imd_grid2grid_stats.sh.
  • Has the code been checked to ensure that no errors occur during the execution? Yes

  • Do these updates/additions include sufficient testing updates? Yes

  • Please complete this pull request review by 4/28/2024.

Pull Request Checklist

  • Review the source issue metadata (required labels, projects, and milestone).

  • Complete the PR description above.

  • Ensure the PR title matches the feature branch name.

  • Check the following:

  • Instructions provided on how to run

  • Developer's name is replaced by ${user} where necessary throughout the code

  • Check that the ecf file has all the proper definitions of variables

  • Check that the jobs file has all the proper settings of COMIN and COMOUT and other input variables

  • Check to see that the output directory structure is followed

  • Be sure that you are not using MET utilities outside the METplus wrapper structure

  • After submitting the PR, select Development issue with the original issue number.

  • After the PR is approved, merge your changes. If permissions do not allow this, request that the reviewer do the merge.

  • Close the linked issue.

@ShelleyMelchior-NOAA
Copy link
Contributor

Hi @malloryprow ! I have cloned your fork and checked out branch bugfix_fhr24snow. I will follow the rest of the instructions and begin the checkout of this PR.

@malloryprow
Copy link
Contributor Author

Sounds good!

@ShelleyMelchior-NOAA
Copy link
Contributor

The runs are underway!
Logfiles are in /lfs/h2/emc/ptmp/shelley.melchior/EVS_out
COMOUT=/lfs/h2/emc/vpppg/noscrub/shelley.melchior/evs/v1.0/stats/global_det

@malloryprow
Copy link
Contributor Author

I got some missing data emails

Warning: No IMD forecast was available for 2024032800f012
Missing file is /lfs/h1/ops/prod/com/evs/v1.0/prep/global_det/atmos.20240328/imd/imd.t00z.f012
Job ID: jevs_global_det_atmos_imd_grid2grid_stats_00.140326817.dbqs01

This would make sense since global_det atmos prep doens't run until 1815Z.

@ShelleyMelchior-NOAA
Copy link
Contributor

OH ok! Let me run again w/ an earlier date. I forgot about that.

@malloryprow
Copy link
Contributor Author

I forgot too!

@AliciaBentley-NOAA
Copy link
Contributor

I got the missing emails as well as was about to write in! Thanks, all! An earlier date sounds good!

@ShelleyMelchior-NOAA
Copy link
Contributor

ShelleyMelchior-NOAA commented Mar 29, 2024

Re-runs are underway w/ VDATE=20240327.

[dlogin01 /lfs/h2/emc/ptmp/shelley.melchior/EVS_out]$ lltr jevs_global_det_atmos_*grid2grid_stats_00.o140328*
-rw-r--r-- 1 shelley.melchior emc 9214232 Mar 29 16:47 jevs_global_det_atmos_imd_grid2grid_stats_00.o140328148
-rw-r--r-- 1 shelley.melchior emc 6268810 Mar 29 16:47 jevs_global_det_atmos_gfs_grid2grid_stats_00.o140328147
-rw-r--r-- 1 shelley.melchior emc 8212512 Mar 29 16:47 jevs_global_det_atmos_ecmwf_grid2grid_stats_00.o140328146

@AliciaBentley-NOAA
Copy link
Contributor

No missing file emails so far!

@malloryprow
Copy link
Contributor Author

Everything looks good!

Copy link
Contributor

@ShelleyMelchior-NOAA ShelleyMelchior-NOAA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code change yields the expected results.

@ShelleyMelchior-NOAA ShelleyMelchior-NOAA merged commit 2fada6b into NOAA-EMC:develop Mar 29, 2024
@malloryprow malloryprow deleted the bugfix_fhr24snow branch March 29, 2024 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants