[Fix] Avoid scope switching when using mmdet inference interface

[Ben-Louis]

Motivation

While inferring top-down models on videos, the scope alternates between 'mmdet' and 'mmpose' repeatedly, which often results in numerous warning messages.

Modification

BC-breaking (Optional)

Use cases (Optional)

Checklist

Before PR:

• I have read and followed the workflow indicated in the CONTRIBUTING.md to create this PR.
• Pre-commit or linting tools indicated in CONTRIBUTING.md are used to fix the potential lint issues.
• Bug fixes are covered by unit tests, the case that causes the bug should be added in the unit tests.
• New functionalities are covered by complete unit tests. If not, please add more unit tests to ensure correctness.
• The documentation has been modified accordingly, including docstring or example tutorials.

After PR:

• CLA has been signed and all committers have signed the CLA in this PR.

[codecov]

Codecov Report

Patch coverage: 84.61% and project coverage change: -0.01 ⚠️

Comparison is base (ffbf595) 82.15% compared to head (2354744) 82.15%.

❗ Current head 2354744 differs from pull request most recent head 102c5f2. Consider uploading reports for the commit 102c5f2 to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##           dev-1.x    #2039      +/-   ##
===========================================
- Coverage    82.15%   82.15%   -0.01%     
===========================================
  Files          224      225       +1     
  Lines        13362    13373      +11     
  Branches      2265     2268       +3     
===========================================
+ Hits         10978    10986       +8     
- Misses        1873     1877       +4     
+ Partials       511      510       -1     

Flag	Coverage Δ
unittests	82.15% <84.61%> (-0.01%)	⬇️

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

Impacted Files	Coverage Δ
mmpose/utils/config_utils.py	80.00% <80.00%> (ø)
...ose/apis/webcam/nodes/model_nodes/detector_node.py	82.35% <100.00%> (ø)
mmpose/utils/__init__.py	100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[ly015]

What about naming the function 'port_mmdet_pipeline' or 'adapt_mmdet_pipeline'?

[Ben-Louis]

Good idea. I think 'adapt_mmdet_pipeline' is preferable

[ly015]

BTW, please also use lazy import for mmdetection in other parts of mmpose package, like https://github.com/open-mmlab/mmpose/blob/dev-1.x/mmpose/apis/inferencers/pose2d_inferencer.py#L7. So the model training/testing and document compiling will not depend on mmdetection.

[ly015]

BTW, please also use lazy import for mmdetection in other parts of mmpose package, like https://github.com/open-mmlab/mmpose/blob/dev-1.x/mmpose/apis/inferencers/pose2d_inferencer.py#L7. So the model training/testing and document compiling will not depend on mmdetection.

Fixed in #2045 . No need to update this PR.