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

Enhance/roc rel ts plots #84

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

WPringle
Copy link

Adding capabilities for plotting reliability diagram and threat score as well as ROC curve.

In the ROC curve, add the number of observed exceedances and observed negatives onto axes, add AUC stat and the probability where the surrogate gives the best POD - FAR value.

Example output for Florence regression GAHM run, 6-ft exceedance
REL_Florence_2018_48hr_leadtime_6_ft
ROC_Florence_2018_48hr_leadtime_6_ft
TS_Florence_2018_48hr_leadtime_6_ft

SorooshMani-NOAA and others added 5 commits November 21, 2024 08:38
Updating the main repo from the dev repo
…y intervals, allowing to switch off the prob map plotting, adding possibility of user-defined output directory
…nd output netcdf file that includes all the stats for the three plot types
@SorooshMani-NOAA SorooshMani-NOAA self-assigned this Nov 21, 2024
@SorooshMani-NOAA
Copy link
Collaborator

SorooshMani-NOAA commented Nov 21, 2024

Please note the script names should be updated here:

run_ensemble = "stormworkflow.main:main"
hurricane_data = "stormworkflow.prep.hurricane_data:cli"
hurricane_mesh = "stormworkflow.prep.hurricane_mesh:cli"
download_data = "stormworkflow.prep.download_data:cli"
setup_ensemble = "stormworkflow.prep.setup_ensemble:cli"
combine_ensemble = "stormworkflow.post.combine_ensemble:cli"
analyze_ensemble = "stormworkflow.post.analyze_ensemble:cli"
storm_roc_curve = "stormworkflow.post.storm_roc_curve:cli"

I will have to implement a test for script names at some point!

@SorooshMani-NOAA
Copy link
Collaborator

Also the colors are easier on my eyes, thank you!

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.

2 participants