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

Removing Open CV dependencies #1679

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

ishaanagw
Copy link
Contributor

No description provided.

@ishaanagw
Copy link
Contributor Author

Partly fixes - #1477

Copy link
Contributor

@guarin guarin left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! I believe the videos are not correctly saved, left some comments :)

for frame in frames:
out.write(frame)
out.release()
out = torchvision.io.write_video(path, self.frames_over_videos, frames)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
out = torchvision.io.write_video(path, self.frames_over_videos, frames)
out = torchvision.io.write_video(filename=path, video_array=frames, fps=1)

self.frames_over_videos is a list of videos and we want to save every video individually.

for frame in self.frames:
out.write(frame)
out.release()
out = torchvision.io.write_video(path, self.frames, n_frames_per_video)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
out = torchvision.io.write_video(path, self.frames, n_frames_per_video)
out = torchvision.io.write_video(filename=path, video_array=self.frames, fps=1)

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