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 bug in distributed sampler #4257

Merged
merged 3 commits into from
Dec 10, 2020
Merged

Conversation

LXXXXR
Copy link
Contributor

@LXXXXR LXXXXR commented Dec 9, 2020

No description provided.

@codecov
Copy link

codecov bot commented Dec 9, 2020

Codecov Report

Merging #4257 (3ea7d2a) into master (bec3fc1) will increase coverage by 0.00%.
The diff coverage is 80.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4257   +/-   ##
=======================================
  Coverage   62.69%   62.70%           
=======================================
  Files         232      232           
  Lines       17603    17607    +4     
  Branches     2945     2945           
=======================================
+ Hits        11037    11041    +4     
  Misses       6027     6027           
  Partials      539      539           
Flag Coverage Δ
unittests 62.66% <80.00%> (+<0.01%) ⬆️

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

Impacted Files Coverage Δ
mmdet/models/dense_heads/yolo_head.py 34.04% <ø> (ø)
mmdet/datasets/samplers/distributed_sampler.py 33.33% <50.00%> (+3.92%) ⬆️
mmdet/models/necks/fpn.py 94.04% <100.00%> (+0.22%) ⬆️

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 bec3fc1...fe86407. Read the comment docs.

@hellock
Copy link
Member

hellock commented Dec 10, 2020

@ZwwWayne can have a double check.

@ZwwWayne ZwwWayne merged commit 111e7b0 into open-mmlab:master Dec 10, 2020
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