-
Notifications
You must be signed in to change notification settings - Fork 9.5k
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
[Refactor]: Unified parameter initialization #4750
Conversation
|
@MeowZheng @jshilong Please review the one-stage algorithm first, I tagged todo where I couldn't be sure. |
Please set |
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.
Many thanks for your nice work, it looks good.
|
Codecov Report
@@ Coverage Diff @@
## master #4750 +/- ##
==========================================
- Coverage 66.03% 64.85% -1.19%
==========================================
Files 261 261
Lines 20306 19915 -391
Branches 3466 3371 -95
==========================================
- Hits 13410 12916 -494
- Misses 6156 6251 +95
- Partials 740 748 +8
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Will recent detectors be modified?
|
No modification for the time being, if you think it is necessary, we can consider doing it in advance. |
Thanks. My understanding is that modification is not essential (except for backbones). |
Motivation
Refactoring model initialization methods based MMCV. For more information, please refer to open-mmlab/mmcv#780
Dependency
Models
Models that have been refactored as follow:
All models have been checked and passed!