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/feature_global_det_rev4 #437

Conversation

malloryprow
Copy link
Contributor

@malloryprow malloryprow commented Feb 6, 2024

Pull Request Testing

  1. Fixes EVS v1.0 Issues 87
  • Describe testing already performed for this Pull Request:

Tested FNOMC atmos grid2grid stats job

  • 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/global_det_rev4
  2. ln -sf /lfs/h2/emc/vpppg/noscrub/emc.vpppg/verification/EVS_fix fix
  3. cd sorc

global_det atmos stats

  1. Run dev/drivers/scripts/stats/global_det/jevs_global_det_atmos_fnmoc_grid2grid_stats.sh. Point COMIN to /lfs/h2/emc/vpppg/noscrub/emc.vpppg/evs/v1.0 or /lfs/h1/ops/com/para/evs/v1.0.
  • 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 02/09/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

Getting started on this PR for the next 20 mins ...

@ShelleyMelchior-NOAA
Copy link
Contributor

I have cloned your fork and checked out branch bugfix/feature_global_det_rev4.

@ShelleyMelchior-NOAA
Copy link
Contributor

The job has been submitted.

132054130.dbqs01               shelley.melchi* dev             jevs_global_de*      --     1    20   25gb 00:15 R   -- 

logfile: /lfs/h2/emc/ptmp/shelley.melchior/EVS_out/jevs_global_det_atmos_fnmoc_grid2grid_stats_00.o132054130

@PerryShafran-NOAA can merge this PR once @malloryprow approves the results are satisfactory in case I am unavailable.

@AliciaBentley-NOAA
Copy link
Contributor

Adding this email from Wei for additional context:

"Thanks, Mallory, for fixing this so quickly!

I've tested your changes with today's failure. It ran to completion as expected.
/lfs/h1/ops/para/output/20240206/evs_global_det_fnmoc_atmos_grid2grid_stats.o115477045

Thanks,
Wei"

@malloryprow @ShelleyMelchior-NOAA @PerryShafran-NOAA

@malloryprow
Copy link
Contributor Author

The job has been submitted.

132054130.dbqs01               shelley.melchi* dev             jevs_global_de*      --     1    20   25gb 00:15 R   -- 

logfile: /lfs/h2/emc/ptmp/shelley.melchior/EVS_out/jevs_global_det_atmos_fnmoc_grid2grid_stats_00.o132054130

@PerryShafran-NOAA can merge this PR once @malloryprow approves the results are satisfactory in case I am unavailable.

All looks good and with Wei's additional testing and approval, we can merge this!

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.

Tests yield anticipated results. This PR is approved for merge.

@ShelleyMelchior-NOAA ShelleyMelchior-NOAA merged commit 0c1e87f into NOAA-EMC:develop Feb 7, 2024
@malloryprow malloryprow deleted the bugfix/feature_global_det_rev4 branch February 8, 2024 12:50
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