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

Support hmdb51 dataset preparation #60

Merged
merged 4 commits into from
Jul 26, 2020
Merged

Conversation

dreamerlin
Copy link
Collaborator

No description provided.

@@ -0,0 +1,20 @@
#! /usr/bin/bash env
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this line looks suspicious. isn't it #!/usr/bin/env bash?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change it to #!/usr/bin/env bash. Same changes in other scripts are made in #62


# sudo apt-get install unrar
unrar x test_train_splits.rar
rm test_train_splits.rar
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it is very likely that user's pc does not have unrar installed. as a consequence, the newly downloaded data will be deleted without decompression. ref https://stackoverflow.com/questions/3474526/stop-on-first-error

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use set -e to make bash exit the script if any command fails

@codecov
Copy link

codecov bot commented Jul 26, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #60   +/-   ##
=======================================
  Coverage   84.67%   84.67%           
=======================================
  Files          73       73           
  Lines        3889     3889           
  Branches      632      632           
=======================================
  Hits         3293     3293           
  Misses        490      490           
  Partials      106      106           
Flag Coverage Δ
#unittests 84.67% <ø> (ø)

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 41e2732...65bdb4c. Read the comment docs.

@innerlee innerlee merged commit 84aa448 into open-mmlab:master Jul 26, 2020
@dreamerlin dreamerlin deleted the hmdb branch August 5, 2020 08:41
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