Opened 3 years ago

Last modified 2 months ago

#5509 new defect

Video with lower dimensions gets transcoded to higher dimensions

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

Description

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:
https://ibb.co/cb1CAv
https://ibb.co/cxwqxa

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?

Subtickets

Change History (3)

comment:1 Changed 3 years ago by ayleph

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 Changed 2 months ago by Ben Sturmfels

Cc: paroneayea removed

Hi Vijeth,

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

Regards,
Ben

comment:3 Changed 2 months ago by Ben Sturmfels

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

Note: See TracTickets for help on using tickets.