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

Retest tsn and tsm video ckpts #50

Merged
merged 4 commits into from
Jul 24, 2020

Conversation

SuX97
Copy link
Collaborator

@SuX97 SuX97 commented Jul 24, 2020

Retest tsn and tsm video ckpts, modify the test pipeline to make sense.

@SuX97
Copy link
Collaborator Author

SuX97 commented Jul 24, 2020

@dreamerlin Please inform me the path in aliyun of tsm ckpts and logs to fill in the TODO blank.

@SuX97 SuX97 force-pushed the re-test_video_ckpts branch from b89b09a to b3d009f Compare July 24, 2020 06:07
@codecov
Copy link

codecov bot commented Jul 24, 2020

Codecov Report

Merging #50 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #50   +/-   ##
=======================================
  Coverage   84.95%   84.95%           
=======================================
  Files          73       73           
  Lines        3874     3874           
  Branches      627      627           
=======================================
  Hits         3291     3291           
  Misses        482      482           
  Partials      101      101           
Flag Coverage Δ
#unittests 84.95% <ø> (ø)

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


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 dd84015...805154a. Read the comment docs.

@SuX97 SuX97 force-pushed the re-test_video_ckpts branch from b3d009f to 12ea89e Compare July 24, 2020 09:41
minor
@SuX97 SuX97 force-pushed the re-test_video_ckpts branch from a8c1bce to 805154a Compare July 24, 2020 11:46
@SuX97
Copy link
Collaborator Author

SuX97 commented Jul 24, 2020

@dreamerlin Ready to be merged.

@innerlee innerlee merged commit 553c59e into open-mmlab:master Jul 24, 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.

2 participants