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

EVS AQMv7 verification round 2 #457

Merged

Conversation

Ho-ChunHuang-NOAA
Copy link
Contributor

Pull Request Testing

Only Prep and Stats processes has been updated.

  • Describe testing already performed for this Pull Request:

    Pre, stats, and plots on vdate=20240218
    cronjob of all 3 steps on vdate=20240219

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

    Please see instruction

  • Has the code been checked to ensure that no errors occur during the execution? [Yes or No]
    YES

  • Do these updates/additions include sufficient testing updates? [Yes or No]
    YES

  • Please complete this pull request review by [Fill in date].

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

Thanks @Ho-ChunHuang-NOAA , this looks great and I am certain NCO will accept this.

@PerryShafran-NOAA
Copy link
Contributor

Hi, @Ho-ChunHuang-NOAA - I have begun working on this PR and am working on the prep step.

@PerryShafran-NOAA
Copy link
Contributor

@Ho-ChunHuang-NOAA The prep jobs are complete.

See all the .o files here: /lfs/h2/emc/vpppg/noscrub/perry.shafran/pr457test/EVS/dev/drivers/scripts/prep/aqm
And the output prep directories are here: /lfs/h2/emc/vpppg/noscrub/perry.shafran/evs/v1.0/prep/aqm

Please take a look.

@PerryShafran-NOAA
Copy link
Contributor

@Ho-ChunHuang-NOAA The stats jobs are complete. I ran the stats job at vhr=00, 04, 11.

See all the .o files here: /lfs/h2/emc/vpppg/noscrub/perry.shafran/pr457test/EVS/dev/drivers/scripts/stats/aqm
See the stats output here: /lfs/h2/emc/vpppg/noscrub/perry.shafran/evs/v1.0/stats/aqm/____.20240219

@Ho-ChunHuang-NOAA
Copy link
Contributor Author

Ho-ChunHuang-NOAA commented Feb 22, 2024

The log files look okay and the file sizes of prep are the same as my prep output.

@Ho-ChunHuang-NOAA
Copy link
Contributor Author

@PerryShafran-NOAA all logs file look fine with me. The ozmax8 and pmave stats on 20240219 are identical to mine. Can not verify hourly statistic since it is not ran for a full 00-23.

@PerryShafran-NOAA
Copy link
Contributor

@Ho-ChunHuang-NOAA If you like, I can run all hours. You can also do a quick check on the hourly stats in atmos.20240219.

Also - there were no changes in any plot scripts, so I am assuming that I do not need to run the plots?

@AliciaBentley-NOAA
Copy link
Contributor

@PerryShafran-NOAA Could be produce the AQM plots to ensure that the new AQMv7 stats work with them? @Ho-ChunHuang-NOAA

@PerryShafran-NOAA
Copy link
Contributor

@AliciaBentley-NOAA I could. But we only have one day's worth of stats files for v7, unless Ho-Chun has a month-long archive of stats files to use.

The format of the stats files aren't any different between v6 and v7, so I would suspect that the plots would be just fine. But I can run it if you'd like.

@AliciaBentley-NOAA
Copy link
Contributor

@PerryShafran-NOAA Perhaps we can merge it (when you are ready) and put it in the emc.vpppg parallel. We could keep an eye on it as it populates the images over the next week? @Ho-ChunHuang-NOAA

@Ho-ChunHuang-NOAA
Copy link
Contributor Author

Ho-ChunHuang-NOAA commented Feb 22, 2024

@PerryShafran-NOAA @AliciaBentley-NOAA Please see instruction for EMC para testing and follow it for PR plots testing.

@Ho-ChunHuang-NOAA
Copy link
Contributor Author

Yes, you can run a full set of hourly statistic and I will compare them to mine.

@PerryShafran-NOAA
Copy link
Contributor

@Ho-ChunHuang-NOAA I ran all 24 hours of stats verification. Please compare.

@Ho-ChunHuang-NOAA
Copy link
Contributor Author

@PerryShafran-NOAA Thanks. I only have script to compare statanalysis products but not stats for each lead time.

@Ho-ChunHuang-NOAA
Copy link
Contributor Author

@PerryShafran-NOAA the 20230219 hourly statics have been checked and they are identical to mine.

@PerryShafran-NOAA
Copy link
Contributor

@Ho-ChunHuang-NOAA @AliciaBentley-NOAA Thanks! I can now then follow Ho-Chun's instructions for plots testing and run a plot test.

@PerryShafran-NOAA
Copy link
Contributor

@Ho-ChunHuang-NOAA Please check out the aqm plot log file:

/lfs/h2/emc/vpppg/noscrub/perry.shafran/pr457test/EVS/dev/drivers/scripts/plots/aqm/jevs_aqm_grid2obs_plots.o134087794

And the plot output:

/lfs/h2/emc/ptmp/perry.shafran/evs/v1.0/plots/aqm/____.20240218

@Ho-ChunHuang-NOAA
Copy link
Contributor Author

@PerryShafran-NOAA The figures look good.

@PerryShafran-NOAA
Copy link
Contributor

Excellent! I think we can merge this then.

Copy link
Contributor

@PerryShafran-NOAA PerryShafran-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 work as expected. Approved for merge.

@PerryShafran-NOAA PerryShafran-NOAA merged commit cecd842 into NOAA-EMC:develop Feb 23, 2024
RoshanShrestha-NOAA pushed a commit to RoshanShrestha-NOAA/EVS that referenced this pull request Mar 12, 2024
* apply AQMv7 changes

* fix file number digit
@Ho-ChunHuang-NOAA Ho-ChunHuang-NOAA deleted the feature/EVSv1.0.8 branch October 23, 2024 18:52
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.

4 participants