[develop] Add precipitation-type verification for MET #757
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
DESCRIPTION OF CHANGES:
As part of the (soon-to-end 😢) DTC UFS SRW project, one of our final deliverables was to add precipitation-type verification to MET. This PR modifies the Metplus config files to output the relevant ptype statistics for PB2NC and PointStat.
The precipitation-type variables used are described in this METplus documentation page. This method uses the reported "PRWE" (present weather) field compared to categorical precipitation variables output from the weather model.
Type of change
TESTS CONDUCTED:
Ran verification WE2E tests (
MET_verification
,MET_ensemble_verification
, andMET_verification_only_vx
) I did not make any statistical plots, but I confirmed that the appropriate output was contained in thepoint_stat*.stat
files. Some of them even had meaningful statistics computed for metrics such as FHO, which isn't a given for such simple test cases! Example line here:4 NA NA NA 0.05 CNT 4069 0.012486 0.012367 0.012604 4 >=1.0 >=161&&<=163 NA NA FHO 539 0.19109 0.13729 0.58998
Ran fundamental tests on Hera as well, but the only changes are to the metplus config files so there should be no impact on the rest of the app.
DEPENDENCIES:
None
DOCUMENTATION:
None
ISSUE:
None
CHECKLIST