update install script with lates mmdet dist to fix yolox input size i…#1123
update install script with lates mmdet dist to fix yolox input size i…#1123fabiogeraci wants to merge 1 commit intoairctic:masterfrom
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1123 +/- ##
==========================================
- Coverage 85.48% 85.28% -0.20%
==========================================
Files 305 305
Lines 6675 6675
==========================================
- Hits 5706 5693 -13
- Misses 969 982 +13
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
|
Hi @fabiogeraci we've been wanting to upgrade the mmdet to 2.20.0 for a very long time now.. This change however breaks another model from mmdet (VFNet). Since there are many IceVision users who relied on VFNet we were reluctant to make this change. We are now planning for an upgrade that will include this and will be out very soon! Thank you again for submitting this PR. |
|
Thanks for the reply |
I modified the install script in order to fix the input_size issue with yolox, as pointed out by @dnth