-
Notifications
You must be signed in to change notification settings - Fork 129
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
Add Jupyter notebook example of noisy label detection for detection tasks #1011
Add Jupyter notebook example of noisy label detection for detection tasks #1011
Conversation
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## releases/1.3.0 #1011 +/- ##
==================================================
- Coverage 78.59% 78.58% -0.01%
==================================================
Files 237 237
Lines 27063 27074 +11
Branches 5397 5401 +4
==================================================
+ Hits 21269 21275 +6
- Misses 4530 4534 +4
- Partials 1264 1265 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
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.
LossDynamicAnalyzer
seems like too much OTX dependent.
We might need refactoring in near future.
210f894
to
f967f23
Compare
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
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!
I left a minor suggestion.
Summary
Visualizer
andLossDynamicsAnalyzer
to support detection tasks.How to test
I added relevant tests for this change.
Checklist
License
Feel free to contact the maintainers if that's a concern.