Skip to content

[Fix] Use len(result) as batch size#987

Merged
ZwwWayne merged 4 commits intomasterfrom
wangruohui-patch-1
Apr 27, 2021
Merged

[Fix] Use len(result) as batch size#987
ZwwWayne merged 4 commits intomasterfrom
wangruohui-patch-1

Conversation

@wangruohui
Copy link
Copy Markdown
Collaborator

fix #985

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 24, 2021

Codecov Report

Merging #987 (2b92d48) into master (5a99f58) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 2b92d48 differs from pull request most recent head 6333c56. Consider uploading reports for the commit 6333c56 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master     #987   +/-   ##
=======================================
  Coverage   65.27%   65.27%           
=======================================
  Files         151      151           
  Lines        9619     9619           
  Branches     1741     1741           
=======================================
  Hits         6279     6279           
  Misses       3017     3017           
  Partials      323      323           
Flag Coverage Δ
unittests 65.27% <100.00%> (ø)

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

Impacted Files Coverage Δ
mmcv/engine/test.py 26.88% <100.00%> (ø)

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 5a99f58...6333c56. Read the comment docs.

@zhouzaida zhouzaida requested a review from dreamerlin April 25, 2021 02:26
@zhouzaida
Copy link
Copy Markdown
Collaborator

@dreamerlin

@zhouzaida zhouzaida changed the title use len(result) as batch size [Fix] Use len(result) as batch size Apr 27, 2021
@ZwwWayne ZwwWayne merged commit 5d75c7a into master Apr 27, 2021
@hellock hellock deleted the wangruohui-patch-1 branch April 27, 2021 14:13
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.

Progress bar does not forward correctly

4 participants