Skip to content
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 train.py/test.py #343

Merged
merged 7 commits into from
Mar 10, 2021
Merged

Refactor train.py/test.py #343

merged 7 commits into from
Mar 10, 2021

Conversation

wHao-Wu
Copy link
Contributor

@wHao-Wu wHao-Wu commented Mar 8, 2021

[Enhance] Refactoring the train.py/test.py

  • Changing details in train.py

    • Re-seting gpu_ids with distributed training mode
    • Adding "cfg-options" and deprecating "options" in arguments
    • Adding support for "custom_imports"
  • Changing details in test.py

    • Changing the import path of test.py
      • Switching wrap_fp16_model from path "mmcv" to path "mmdet"
    • Adding support for "custom_imports"
    • Adding "cfg-options" and "eval-options", and deprecating "options" in argument
    • Adding support when test dataset is concatenated
    • Changing op when args.fuse_conv_bn is true
    • Changing interface in evaluation
  • Verifying refactor above with SECOND model

@codecov
Copy link

codecov bot commented Mar 8, 2021

Codecov Report

Merging #343 (6370199) into master (e366272) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #343   +/-   ##
=======================================
  Coverage   49.69%   49.69%           
=======================================
  Files         174      174           
  Lines       11758    11758           
  Branches     1838     1838           
=======================================
  Hits         5843     5843           
  Misses       5555     5555           
  Partials      360      360           
Flag Coverage Δ
unittests 49.69% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e366272...f3aa185. Read the comment docs.

tools/train.py Outdated Show resolved Hide resolved
tools/train.py Outdated Show resolved Hide resolved
@ZwwWayne ZwwWayne merged commit 8b131f5 into open-mmlab:master Mar 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants