-
Notifications
You must be signed in to change notification settings - Fork 1
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
plot new CRPS_EMP_FAIR stat #421
Comments
@JohnHalleyGotway Can this statistic be calculated using agg logic? If yes, what is the formula? |
@TatianaBurek, as seen on this line of code, just like CRPS_EMP, the CRPS_EMP_FAIR statistics is aggregated as a weighted mean, where the weight is defined by the number of matched pairs listed in the TOTAL column. |
@JohnHalleyGotway @venitahagerty I am confused. Which statistics do we need |
@TatianaBurek, yes, I see why you'd be confused. I think @venitahagerty accidentally added an extra 'S' in the statistic that she added to the database. The new statistics is named CRPS_EMP_FAIR with only one 'S'. For reference, here are the related issues:
And I do see a copy/paste error in the description in the MET User's Guide that likely caused the confusion. The ECNT Table describes it as:
I'll remove the word Skill from that description. So @venitahagerty can you please update METdataio to correctly store this statistic as CRPS_EMP_FAIR? |
Sorry about that. Just fixed it in the schema in the develop branch in
METdataio.
…On Wed, Nov 2, 2022 at 8:18 AM johnhg ***@***.***> wrote:
@TatianaBurek <https://github.com/TatianaBurek>, yes, I see why you'd be
confused. I think @venitahagerty <https://github.com/venitahagerty>
accidentally added an extra 'S' in the statistic that she added to the
database. The new statistics is named CRPS_EMP_FAIR with only one 'S'.
For reference, here are the related issues:
- dtcenter/MET#2206 <dtcenter/MET#2206>
added CRPS_EMP_FAIR to the ECNT line type
<https://met.readthedocs.io/en/develop/Users_Guide/ensemble-stat.html#id2>
.
- dtcenter/METdataio#119
<dtcenter/METdataio#119> updated the
database for this new statistics, but the PR dtcenter/METdataio#126
<dtcenter/METdataio#126> added the wrong name.
And I do see a copy/paste error in the description in the MET User's
Guide, that likely caused the confusion. The ECNT Table
<https://met.readthedocs.io/en/develop/Users_Guide/ensemble-stat.html#id2>
describes it as:
The Continuous Ranked Probability Skill Score (empirical distribution) adjusted by subtracting 1/2(m) times the mean absolute difference of the ensemble members (m is the ensemble size)
I'll remove the word *Skill* from that description.
So @venitahagerty <https://github.com/venitahagerty> can you please
update METdataio to correctly store this statistic as CRPS_EMP_FAIR?
—
Reply to this email directly, view it on GitHub
<#421 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AJGI4XQMPAJJWEKJXZUBCKDWGJZ2BANCNFSM6AAAAAAQ6WKH7U>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
Venita Hagerty, PhD
Research Associate
CIRA at NOAA, GSL/AVID
Voicemail: 307-217-6697
|
@venitahagerty I don't see this change in develop... Did you do "push" |
Change is now in develop branch
…On Wed, Nov 2, 2022 at 8:42 AM Tatiana Burek ***@***.***> wrote:
@venitahagerty <https://github.com/venitahagerty> I don't see this change
in develop... Did you do "push"
—
Reply to this email directly, view it on GitHub
<#421 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AJGI4XS2MZFPE5WZ7HDJA4DWGJ4WJANCNFSM6AAAAAAQ6WKH7U>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
Venita Hagerty, PhD
Research Associate
CIRA at NOAA, GSL/AVID
Voicemail: 307-217-6697
|
Describe the Enhancement
MET development version 11.0.0-beta3 made several changes to the "stat" output.
Include CRPS_EMP_FAIR to plotting as a simple stat and agg stat
issues:
dtcenter/METdataio#119
dtcenter/METcalcpy#215
Time Estimate
Estimate the amount of work required here.
Issues should represent approximately 1 to 3 days of work.
Sub-Issues
Consider breaking the enhancement down into sub-issues.
Relevant Deadlines
List relevant project deadlines here or state NONE.
Funding Source
2799991
Define the Metadata
Assignee
Labels
Projects and Milestone
Define Related Issue(s)
Consider the impact to the other METplus components.
Enhancement Checklist
See the METplus Workflow for details.
Branch name:
feature_<Issue Number>_<Description>
Pull request:
feature <Issue Number> <Description>
Select: Reviewer(s) and Linked issues
Select: Repository level development cycle Project for the next official release
Select: Milestone as the next official version
The text was updated successfully, but these errors were encountered: