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

generate error messages on Transcode failures #17

Open
Ironman28 opened this issue Apr 29, 2015 · 4 comments
Open

generate error messages on Transcode failures #17

Ironman28 opened this issue Apr 29, 2015 · 4 comments

Comments

@Ironman28
Copy link

This clip failed to transcode and then did not respond to a manual transcode command from me:
http://www.criticalcommons.org/Members/mlangford/clips/chinatown-sequence-analysis
When clips fail to transcode, should there be a warning or explanation (e.g., file type not allowed; file size over limit, etc.)?
I also note that this clip appears with an image icon - not sure if this is related

@Ironman28
Copy link
Author

Okay, so I just had to delete that clip for the user, who admitted that it may have been a bad original file. However, the other clip that did not transcode automatically is still there:
http://www.criticalcommons.org/Members/MCIMR/clips/love-and-duration

One difference seems to be that the deleted clip threw a MIME error message, whereas this one might work after doing a manual transcode (i.e., it did not generate a Mime error)

@mgogoulos
Copy link
Member

As you can see on the screenshot the file (love-and-duration) is 906 bytes size, it is definitely not a valid video file. Most probably the user uploaded the wrong file, or somehow upload was corrupted. I agree that the system should warn about the fact that it tried to transcode it and has failed for xyz reason. This is one of the most critical tickets that exist in Plumi in my opinion. It is a common scenario that transcoding will fail (wrong video upload, corrupted video, etc) so the user has to be notified in order to upload the correct video! Should be fixed in plumi.content package of Plumi

cc

@ahelme ahelme added this to the spring 2015 - urgent priority milestone Jun 16, 2015
@ahelme ahelme added the bug label Jun 16, 2015
@ahelme ahelme changed the title Transcode failure without warning generate error messages on Transcode failures Jun 16, 2015
@ahelme
Copy link
Member

ahelme commented Jun 16, 2015

so instead we should generate a decent error message for these transcode failures, and an email notification to Steve (Manager role) and the user should be generated

@ahelme
Copy link
Member

ahelme commented Oct 15, 2015

This would be great for Plumi, it's true!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants