-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[Feature] Support calculate confusion matrix and plot it. #1287
Conversation
Codecov ReportBase: 0.02% // Head: 86.88% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## dev-1.x #1287 +/- ##
============================================
+ Coverage 0.02% 86.88% +86.86%
============================================
Files 121 166 +45
Lines 8217 13580 +5363
Branches 1368 2158 +790
============================================
+ Hits 2 11799 +11797
+ Misses 8215 1422 -6793
- Partials 0 359 +359
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
cmap: str = 'viridis', | ||
classes: Optional[List[str]] = None, | ||
colorbar: bool = True, | ||
show: bool = True): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would prefer to add a save path parameter here so that the result image can be saved on the command line when using linux
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The method will output the figure, users can save the figure manually if they want.
Hi, about the third method: use the command line tool. I can't find confusion_matrix.py in Analysis_tools. @mzr1996 |
I have moved it to the |
0d45608
to
01fb19f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Hi, the github codebase is not updated. Another problem is, I used the latest version of mmcla, when I run confusion_matrix.py, an error occured--cannot import name 'ConfusionMatrix' from 'mmcls.evaluation'. I found that 'ConfusionMatrix' is not defined, is there a script for 'ConfusionMatrix'? @mzr1996 |
I think you can pull the latest |
@mzr1996 hi , when I used the third way to get confusion matrix, the following error occured: |
Modification
Add the
ConfusionMatrix
metric and a command-line tool to calculate/plot the confusion matrix.Use cases (Optional)
1. Basic Usage
2. Use with Evalutor
3. Use command-line tool
Checklist
Before PR:
After PR: