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

About the save_tf_record() #14

Closed
happyday521 opened this issue Dec 29, 2018 · 2 comments
Closed

About the save_tf_record() #14

happyday521 opened this issue Dec 29, 2018 · 2 comments

Comments

@happyday521
Copy link

Hi! Thanks for releasing the code! I am having some trouble preparing my own dataset.
In function read_videos_and_save_tf_records():
clipboard
I think one sequence corresponds to the frames of one video file(for example 1.avi) ,and the function
save_tf_record() save these frames to a .tfrecord file(e.g 1.tfrecord).

However, in the function save_tf_record():
clipboard1
It seems one sequence corresponds to >= 1 video files. I am confused by it,

Can you help me? Thanks!

@alexlee-gk
Copy link
Owner

That's right, the function save_tf_record can save multiple sequences corresponding to multiple files. However, this function was always just called with a single sequence.

This turned out to be quite inefficient. I have updated the master branch with several improvements, one of which is in the preprocessing of the KTH dataset. In the new version, you should see that sequences_per_file=128 video sequences are saved in each tfrecord file.

Also, after you pull my changes, make sure to pip install -r requirements.txt again since there is one new dependency.

@happyday521
Copy link
Author

Get it. Thanks!

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

No branches or pull requests

2 participants