Skip to content

Add warning for files with corrupt containers #3961

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jun 4, 2021
Merged

Add warning for files with corrupt containers #3961

merged 2 commits into from
Jun 4, 2021

Conversation

@bjuncek
Copy link
Contributor

@bjuncek bjuncek commented Jun 4, 2021

Resolves #3282

@bjuncek bjuncek mentioned this pull request Jun 4, 2021
Closed
datumbox
datumbox approved these changes Jun 4, 2021
View reviewed changes
Copy link
Contributor

@datumbox datumbox left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks!

@datumbox datumbox merged commit c6c5a5f into pytorch:master Jun 4, 2021
facebook-github-bot pushed a commit that referenced this pull request Jun 10, 2021
@fmassa @facebook-github-bot
[fbsync] Add warning for files with corrupt containers (#3961)
33496a1 
Summary: Co-authored-by: Vasilis Vryniotis <[email protected]>

Reviewed By: NicolasHug

Differential Revision: D29027310

fbshipit-source-id: f7b96ef1e5cfd5b37bfd9a449487ccc0eb4cfb21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@datumbox datumbox datumbox approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cla signed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add verbose mode for dataset constructors to print directory scan stats + add a warning when video loading fails

3 participants

@bjuncek @datumbox @facebook-github-bot