-
Notifications
You must be signed in to change notification settings - Fork 747
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
[Feat] Support specify gpu for testing and training with gpu-id instead of gpu-ids and gpus #756
Conversation
Codecov Report
@@ Coverage Diff @@
## main #756 +/- ##
==========================================
+ Coverage 84.53% 84.62% +0.08%
==========================================
Files 165 165
Lines 10664 10664
Branches 1622 1622
==========================================
+ Hits 9015 9024 +9
+ Misses 1303 1295 -8
+ Partials 346 345 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
Need to update docs (training.md & test.md)
…ad of gpu-ids and gpus (open-mmlab#756) * support set gpu number for testing and deprecate gpu_ids and gpus with gpu_id * update gpu-ids gpus and gpu-id dostring * add args.gpus process * fix typo Co-authored-by: Tong Gao <gaotongxiao@gmail.com>
…ad of gpu-ids and gpus (open-mmlab#756) * support set gpu number for testing and deprecate gpu_ids and gpus with gpu_id * update gpu-ids gpus and gpu-id dostring * add args.gpus process * fix typo Co-authored-by: Tong Gao <gaotongxiao@gmail.com>
open-mmlab/mmdetection#7019
Motivation
If the number of GPUs is more than 1, there will be an assertion error in MMDP because MMDP does not support the situation with more than one GPU.
Modification
--gpu-ids is changed to --gpu-id.
BC-breaking (Optional)
--gpus-ids and --gpus will be deprecated, please use --gpu-id.