Video upload failed when video is portrait.

Videos being uploaded that are portrait fail every time no matter what.

 

tried various video files in portrait shot and uploaded directly from: iPhone [4s, 5, (ios8)] [5s, 6, 6 plus, 6s (ios9)]; galaxy s3, s4, s5; note 3, 4; ipad 3, mini 1, 4, pro 1, and kindle hdx... all failed

 

re encoded from sony vegas, vlc, and ffmpeg and all have the same failed result.

 

tried true portrait videos and rotated videos via metadata both fail

 

even had a 6s (without a case) slip out of my hand and scratch my laptop screen in the process of testing.

 

repeated any test in landscape and it works with no problem.

 

ruled out all hardware, format, codex, and encapsulation.

 

poking around the code, its setup to only accept 4:3 and 16:9 videos

 

how would i go about adding a portrait aspect ratio?

Quote · 23 Dec 2015

Hi. Please let us know what version you are using. I'd like to reproduce this problem if it's 7.2.1

Quote · 23 Dec 2015

 

Hi. Please let us know what version you are using. I'd like to reproduce this problem if it's 7.2.1

 totally forget to state that, its 7.2.1

Quote · 23 Dec 2015

Vertical videos process on my 7.1.6 site so this may be related to the new HTML5 uploader.  Test in both the HTML5 uploader and the regular uploader.

Geeks, making the world a better place
Quote · 23 Dec 2015

 

poking around the code, its setup to only accept 4:3 and 16:9 videos

 The encoder, ffmpeg, will encode the video to those formats even if vertical/portrait.  It will have black boxes on either side.  It is not the format that is preventing the encoding as ffmpeg can handle the videos.

Geeks, making the world a better place
Quote · 23 Dec 2015

tried html5 and regular uploader to the same result. upload fails, creates temp file, no error output. dose the same in my 7.2.1 completely stock sandbox.

 

monitoring the server processes during testing and ffmpeg is never engaged with these files, the failure is after the temp file is created and before ffmpeg touchs it to make a thumbnail. No other files, thumbs, db entries are created. Just the xxxx_temp is created in the data dir

Quote · 23 Dec 2015

I suggest you start looking at your server.

Geeks, making the world a better place
Quote · 23 Dec 2015

I'm surprised no one else has run into / noticed this... If you peep your "/flash/modules/video/files/" folder I'm sure you'll see all the "xxxx_temp" files chilling in there representing content you lost and the members it annoyed.

 

With all dolphin bugs - its normally solved with less then ten lines of code! 

https://www.boonex.com/forums/topic/Adding-portrait-video-support-to-7-2-x.htm

Quote · 23 Dec 2015

Thank you for the report and the fix.

There is an issue for this in GitHub:

https://github.com/boonex/dolphin.pro/issues/220

The temporary fix is in the post above from @arkane220 or just disable square resizing in modules/boonex/videos/classes/BxVideosConfig.php file:

'browse' => array('postfix' => THUMB_FILE_NAME . IMAGE_EXTENSION, 'image' => true, 'w' => 240, 'h' => 240, 'square' => false),
'browse2x' => array('postfix' => THUMB_FILE_NAME . '_2x' . IMAGE_EXTENSION, 'image' => true, 'w' => 480, 'h' => 480, 'square' => false),
Rules → http://www.boonex.com/terms
Quote · 24 Dec 2015

Hi Alex,

Just so we understand this a bit better. I've gone through the temporary fix code.

So does this code allow for the feature to be "enabled" and supported, while the square resizing disables the feature?

 

Thank you for the report and the fix.

There is an issue for this in GitHub:

https://github.com/boonex/dolphin.pro/issues/220

The temporary fix is in the post above from @arkane220 or just disable square resizing in modules/boonex/videos/classes/BxVideosConfig.php file:

'browse' => array('postfix' => THUMB_FILE_NAME . IMAGE_EXTENSION, 'image' => true, 'w' => 240, 'h' => 240, 'square' => false),
'browse2x' => array('postfix' => THUMB_FILE_NAME . '_2x' . IMAGE_EXTENSION, 'image' => true, 'w' => 480, 'h' => 480, 'square' => false),

 

Quote · 24 Dec 2015

 

 

Hi Alex,

Just so we understand this a bit better. I've gone through the temporary fix code.

So does this code allow for the feature to be "enabled" and supported, while the square resizing disables the feature?

basically do one or the other... haven't tried both not sure of the result.

 

if you just fix the thumbnail hard stop it will stretched the video since there is no aspect control in the stock encoding script and if you added to maintain aspect ratio it will pad it with side bars (i think)


anyhow... i didn't want any video padding (letter box) or stretching.

 

first, tweaked the thumbnail creation in the functions.inc.php and that stretched the video to 16:9 and looked horrible.

 

second,  went through all the files related to video uploading, handing, and encoding.

 

my fix will encode the video in its native vertical 9:16 aspect ratio and make a square 1:1 thumbnail.

 

>75% of my users visit the site from their mobile phone so i wanted it to play properly as if someone send it to you via text.

 

since my fix and posting something about it in my forums i had some 62 vertical videos uploaded with no issues and no fails. all videos create the square thumbnail, encode correctly to 360x640, and play on my phone with no issue at full screen with no letter boxing holding it vertical.

 

note: i have seen some stretching in the thumbnails but not worried about that right now.

Quote · 24 Dec 2015

Yes, my fix is just for the thumbnail fix, portrait video still may have problems, like stretching.

Thank you @arkane220 for the detailed explanation.

Rules → http://www.boonex.com/terms
Quote · 26 Dec 2015

Ok.I've reverted your change of "false" and will just await 7.2.2.

 

Yes, my fix is just for the thumbnail fix, portrait video still may have problems, like stretching.

Thank you @arkane220 for the detailed explanation.

 

Quote · 28 Dec 2015
 
 
Below is the legacy version of the Boonex site, maintained for Dolphin.Pro 7.x support.
The new Dolphin solution is powered by UNA Community Management System.