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?
|
Hi. Please let us know what version you are using. I'd like to reproduce this problem if it's 7.2.1 |
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
|
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 |
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 |
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
|
I suggest you start looking at your server. Geeks, making the world a better place |
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
|
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 |
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),
|
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.
|
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 |
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.
|