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

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

Merged
merged 4 commits into from
Apr 27, 2021
Merged

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

merged 4 commits into from
Apr 27, 2021

Conversation

wangruohui
Copy link
Member

fix #985

@codecov
Copy link

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
mmcv/engine/test.py Outdated Show resolved Hide resolved
@zhouzaida
Copy link
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