-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Customize runner type #4570
Customize runner type #4570
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4570 +/- ##
==========================================
- Coverage 64.31% 64.29% -0.02%
==========================================
Files 250 250
Lines 19249 19286 +37
Branches 3252 3264 +12
==========================================
+ Hits 12380 12400 +20
- Misses 6223 6234 +11
- Partials 646 652 +6
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
configs/faster_rcnn/faster_rcnn_r50_caffe_fpn_mstrain_90000it_coco.py
Outdated
Show resolved
Hide resolved
configs/faster_rcnn/faster_rcnn_r50_caffe_fpn_mstrain_90000it_coco.py
Outdated
Show resolved
Hide resolved
LGTM after resolving comments. Please @jshilong also have a look. |
May also enrich the PR message to describe the supported feature and migration code in detail. Thus other users could understand the PR more clearly as this modification is important. |
Add support to customize runner type ('EpochBasedRunner', 'IterBasedRunner') and merge the 'total_epoch' attribute in the config with the newly added runner attribute as follows: