-
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
fix (dpool): directly return empty if rois's length is 0 #2099
Conversation
tests/test_dpool.py
Outdated
ModulatedDeformRoIPoolingPack) | ||
|
||
|
||
def test_dpool_empty_cpu(): |
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.
CPU inference for DCN is currently not supported, thus this may be unnecessary.
tests/test_dpool.py
Outdated
def test_dpool_empty_cpu(): | ||
""" | ||
CommandLine: | ||
xdoctest -m tests/test_dpool.py test_dpool_empty_cpu |
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.
No need to use xdoctest to perform unit tests. It is mainly used to test the cases in docstrings.
tests/test_dpool.py
Outdated
assert data.shape[0] == rois.shape[0] | ||
|
||
|
||
def test_dpool_empty_gpu(): |
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.
Non-empty inputs may also be tested, which is unavailable on travis but available locally.
* jon/dev/fix_fpn2: (33 commits) Fix FPN upscale fix analyze log (open-mmlab#2150) Fix a documentation error in GETTING_STARTED.md (open-mmlab#2149) add optimizer registry (open-mmlab#2139) Update version to 1.1 (open-mmlab#2144) Fix IOU assigners when ignore_of_thr > 0 and no pred boxes (open-mmlab#2135) reset worker_seed (open-mmlab#2111) Fix issue with list of metrics in CustomDataset.evaluate (open-mmlab#2128) Code Release: CARAFE: Content-Aware ReAssembly of FEatures (ICCV 2019) (open-mmlab#1583) fixed test package (open-mmlab#2127) add an argument format-only to handle the json formating (open-mmlab#2114) fix (get_cls_results): use np.empty for empty bbox rather than np.arrary (open-mmlab#2116) fix (dpool): directly return empty if rois's length is 0 (open-mmlab#2099) fix workflow problem (open-mmlab#2103) Uint8 fix (open-mmlab#2105) Fix device bug (open-mmlab#2098) fix test ddp initialize (open-mmlab#2100) set FORCE_CUDA (open-mmlab#2097) Speed up sampler (open-mmlab#2094) Use official DDP to implement MMDDP (open-mmlab#2090) ...
* tag 'v1.1.0': (29 commits) Update version to 1.1 (open-mmlab#2144) Fix IOU assigners when ignore_of_thr > 0 and no pred boxes (open-mmlab#2135) reset worker_seed (open-mmlab#2111) Fix issue with list of metrics in CustomDataset.evaluate (open-mmlab#2128) Code Release: CARAFE: Content-Aware ReAssembly of FEatures (ICCV 2019) (open-mmlab#1583) fixed test package (open-mmlab#2127) add an argument format-only to handle the json formating (open-mmlab#2114) fix (get_cls_results): use np.empty for empty bbox rather than np.arrary (open-mmlab#2116) fix (dpool): directly return empty if rois's length is 0 (open-mmlab#2099) fix workflow problem (open-mmlab#2103) Uint8 fix (open-mmlab#2105) Fix device bug (open-mmlab#2098) fix test ddp initialize (open-mmlab#2100) set FORCE_CUDA (open-mmlab#2097) Speed up sampler (open-mmlab#2094) Use official DDP to implement MMDDP (open-mmlab#2090) log meta (open-mmlab#2086) fix pad_val not used in class Pad when pad to a fixed size (open-mmlab#2093) remove cython docs (open-mmlab#2091) remove new_tensor (open-mmlab#2092) ...
…2099) * fix (dpool): directly return empty if rois's length is 0 * fix (dpool): also skip empty when no_trans=True * reformat (test/test_dpool.py): reformat to pass flake8 * fix (test_dpool): skip gpu test when GPU is not avaliable * rm (test_dpool.py): rm test_dpool since it is not very necessary in CI
…2099) * fix (dpool): directly return empty if rois's length is 0 * fix (dpool): also skip empty when no_trans=True * reformat (test/test_dpool.py): reformat to pass flake8 * fix (test_dpool): skip gpu test when GPU is not avaliable * rm (test_dpool.py): rm test_dpool since it is not very necessary in CI
* ttfnet infer * fix ttfnet deploy * clean code * add comment, fix config * add comment, update modelzoo * fix norm resize, update modelzoo
…ter than prediction_length + 1 (open-mmlab#2099)
Fix #2072