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

recursively remove pretrained step for testing #695

Merged
merged 5 commits into from
Mar 9, 2021

Conversation

dreamerlin
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented Mar 8, 2021

Codecov Report

Merging #695 (22bded5) into master (ee34d95) will decrease coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #695      +/-   ##
==========================================
- Coverage   84.64%   84.59%   -0.05%     
==========================================
  Files         130      130              
  Lines        9346     9357      +11     
  Branches     1575     1578       +3     
==========================================
+ Hits         7911     7916       +5     
- Misses       1038     1042       +4     
- Partials      397      399       +2     
Flag Coverage Δ
unittests 84.58% <ø> (-0.05%) ⬇️

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

Impacted Files Coverage Δ
mmaction/models/backbones/resnet3d_slowfast.py 82.65% <0.00%> (-1.96%) ⬇️
mmaction/models/recognizers/base.py 64.54% <0.00%> (+0.46%) ⬆️

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 ee34d95...22bded5. Read the comment docs.

@dreamerlin dreamerlin requested a review from congee524 March 8, 2021 14:16
@dreamerlin dreamerlin changed the title remove pretrained step for testing recursively remove pretrained step for testing Mar 9, 2021
@innerlee innerlee merged commit f8481fa into open-mmlab:master Mar 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants