-
Notifications
You must be signed in to change notification settings - Fork 443
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
Use torchvision in MRCNN on CUDA #3347
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## releases/1.6.0 #3347 +/- ##
==================================================
- Coverage 80.76% 80.73% -0.03%
==================================================
Files 536 536
Lines 40476 40477 +1
==================================================
- Hits 32689 32681 -8
- Misses 7787 7796 +9 ☔ View full report in Codecov by Sentry. |
@sovrasov - Will it affect the speed of nncf optimization a lot? Should it only be applied to MaskRCNN-ResNet50 which was the offending thing? or all models? |
This PR is for testing purposes, I'll update the implementation later to restrict the scope of the changes. Also, I'm planning to benchmark performance and accuracy with this patch today |
@chuneuny-emily please see the benchmark results mmcv vs torchvision in the PR description. torchvision is slightly slower, but I think this is acceptable. |
Summary
This could mitigate QAT instability on some datasets
Dataset: geti test data from the ticket
Model: ResNet50 MRCNN
How to test
Checklist
License
Feel free to contact the maintainers if that's a concern.