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

Video clips in portrait mode seem to use height as width and vice versa #3483

Closed
alansharkey opened this issue May 11, 2020 · 2 comments
Closed
Labels
duplicate This issue is a duplicate and should be closed

Comments

@alansharkey
Copy link

WHen I import a video clip which I took in portrait mode, it chops the top and bottom of the video off as if it was taking the width parameter as the height and the height as the width.

I have verified this with a few phones and also the latest (May 11th) build.

Alan

@alansharkey
Copy link
Author

An example file is here https://www.dropbox.com/s/azdtx5ugaqfxujw/Alan.mp4?dl=1

Alan

@ferdnyc ferdnyc added the duplicate This issue is a duplicate and should be closed label May 13, 2020
@ferdnyc
Copy link
Contributor

ferdnyc commented May 13, 2020

Yeah, the issue is that those dimensions are correct for the way nearly all phones implement portrait video.

When a phone-like device shoots video in portrait mode, what it stores is a landscape video (1920 wide × 1080 tall, for example), with a 90° rotation embedded in the metadata.

Libopenshot recognizes and applies the rotation, but it treats the video as having its unrotated dimensions, which causes issues as detailed in OpenShot/libopenshot#322 (along with #2969 + others, in this repo).

@ferdnyc ferdnyc closed this as completed May 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue is a duplicate and should be closed
Projects
None yet
Development

No branches or pull requests

2 participants