Skip to content

[Fix]Fix lvis classwise test error.#6845

Merged
ZwwWayne merged 4 commits intoopen-mmlab:dev-v2.20.0from
Czm369:lvis_classwise
Dec 22, 2021
Merged

[Fix]Fix lvis classwise test error.#6845
ZwwWayne merged 4 commits intoopen-mmlab:dev-v2.20.0from
Czm369:lvis_classwise

Conversation

@Czm369
Copy link
Collaborator

@Czm369 Czm369 commented Dec 21, 2021

Motivation

Fix lvis classwise test error.

Modification

Add [] for catId to to be compatible with lvis API.

@codecov
Copy link

codecov bot commented Dec 21, 2021

Codecov Report

Merging #6845 (653860b) into dev-v2.20.0 (deca532) will increase coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##           dev-v2.20.0    #6845   +/-   ##
============================================
  Coverage        62.02%   62.02%           
============================================
  Files              323      323           
  Lines            25744    25744           
  Branches          4363     4363           
============================================
+ Hits             15967    15969    +2     
+ Misses            8929     8928    -1     
+ Partials           848      847    -1     
Flag Coverage Δ
unittests 62.01% <0.00%> (+0.01%) ⬆️

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

Impacted Files Coverage Δ
mmdet/datasets/lvis.py 14.86% <0.00%> (ø)
mmdet/core/bbox/samplers/random_sampler.py 80.55% <0.00%> (+5.55%) ⬆️

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 deca532...653860b. Read the comment docs.

@ZwwWayne ZwwWayne requested a review from hhaAndroid December 22, 2021 02:34
@ZwwWayne ZwwWayne merged commit 1efcc0a into open-mmlab:dev-v2.20.0 Dec 22, 2021
@AronLin AronLin mentioned this pull request Dec 27, 2021
ZwwWayne pushed a commit that referenced this pull request Jul 18, 2022
* [Fix]Fix lvis classwise test error.

* self.areaRngLbl = ['all', 'small', 'medium', 'large']

* pre-commit
ZwwWayne pushed a commit to ZwwWayne/mmdetection that referenced this pull request Jul 19, 2022
* [Fix]Fix lvis classwise test error.

* self.areaRngLbl = ['all', 'small', 'medium', 'large']

* pre-commit
@Czm369 Czm369 deleted the lvis_classwise branch August 20, 2022 12:38
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