Opened 7 years ago

Last modified 2 years ago

#5509 new defect

Video with lower dimensions gets transcoded to higher dimensions

Reported by: Vijeth Owned by:
Priority: major Milestone: 0.13.0
Component: programming Keywords: transcoding, video
Cc: Boris Bobrov, alyph, thomzane Parent Tickets:


I uploaded a video with (256, 144) dimensions, and changed the max default size (that a video can be transcoded to) to (1920, 1080). And, to my surprise, the video actually got transcoded to those dimensions.

So, when I viewed it on my local machine, there was zero padding (black colour) added to the top and bottom of the video to be in that resolution (dimensions).

These are the images:

I checked skip_transcode() method myself, and I realized the dimensionality check was working fine, but there was a problem with container-formats (not sure if it is a problem).

Basically, the video had 'ISO-MP4' container-format but mediagoblin config container-format string only allowed 'Matroska' container-format, and because of this the video was getting transcoded again (even though all other checks were fine).

Shall we remove this container-format check? Or add more container-formats to the config?

Change History (7)

comment:1 by ayleph, 7 years ago

Why not leave the container format check as it is? That's a configuration option, so anyone can change it on their own instance. I think it's currently set to a reasonable default value. Maybe it would be good to document the video plugin options better so people know that they can change it.

comment:2 by Ben Sturmfels, 4 years ago

Cc: paroneayea removed

Hi Vijeth,

Would you be willing to propose a patch to achieve the behavior you're expecting here?


comment:3 by Ben Sturmfels, 4 years ago

Hmm. I suspect that the issue tracker isn't sending email to the reporter by default. I'll look into this.

in reply to:  3 comment:4 by Ben Sturmfels, 4 years ago

Replying to Ben Sturmfels:

Hmm. I suspect that the issue tracker isn't sending email to the reporter by default. I'll look into this.

Just confirming that the issues tracker has now been updated to send notifications to the reporter by default.

comment:5 by Ben Sturmfels, 4 years ago

Milestone: 0.11.0

See duplicate #5591.

comment:6 by Ben Sturmfels, 3 years ago

Cc: thomzane added

Michael from FSF also reported that uploading an already encoded video triggered a transcode when he wasn't expecting it to. Not sure exactly what format that video was in when uploaded. Seems related to this issue though.

comment:7 by Ben Sturmfels, 2 years ago

Note: See TracTickets for help on using tickets.