After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 769147 - playbin: Add videoflip/glvideoflip or similar to the video-sink
playbin: Add videoflip/glvideoflip or similar to the video-sink
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
1.8.2
Other All
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on: 768687
Blocks: 765309
 
 
Reported: 2016-07-25 11:51 UTC by Andy Robinson
Modified: 2018-11-03 11:48 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
2 second video of a door, portrait mode. (2.40 MB, video/quicktime)
2016-07-25 11:51 UTC, Andy Robinson
Details

Description Andy Robinson 2016-07-25 11:51:48 UTC
Created attachment 332099 [details]
2 second video of a door, portrait mode.

I attach a 2-second video taken on an iPhone 4S. It's a portrait mode video of a normal upright door. I can double-click it on Windows 10 or Mac 10.0 and it plays correctly as portrait mode. However GStreamer displays it in landscape mode with the door on its side. I've tried:
gst-launch-1.0 filesrc location=door_rotn.mov ! decodebin ! videoconvert ! autovideosink
Linux GST 1.2.4
Windows GST 1.8.2
Mac GST 1.8.2

Obviously it's easy enough to put a rotation into the GStreamer pipeline to correct it, but it shouldn't be necessary.

Andy Robinson
Comment 1 Sebastian Dröge (slomo) 2016-07-25 12:07:28 UTC
With videoflip method=auto or the same with glvideoflip, this already does the right thing. It's just not in playbin yet.

If you do a manual pipeline, you will have to add videoflip or glvideoflip yourself though.
Comment 2 Andy Robinson 2016-07-25 13:56:30 UTC
Thanks for this. Is there any way to determine the rotation required and then I can specify it to videoflip explicitly? This would have 2 advantages for me:

1) It might well work on GST 1.2.4 which does not have the "automatic" videoflip method. And there is lots of 1.2.4 still out there, e.g. Ubuntu 14.04 LTS.

2) I already offer my users the option of flipping, so if I use "automatic" videoflip then I would have to follow it with a second videoflip for user-requested flipping. I presume it would be more efficient to combine the two operations in a single flip.
Comment 3 Sebastian Dröge (slomo) 2016-07-25 13:58:49 UTC
The rotation is given in a tag, GST_TAG_IMAGE_ORIENTATION
Comment 4 Andy Robinson 2016-07-25 14:05:38 UTC
Thanks again, I'll try it.

Incidentally I think my 2nd point above is invalid - I don't think you can execute a 90 degree rotation followed by a horizontal or vertical flip in one operation can you? I don't think any of the available flip modes do this.
Comment 5 Andy Robinson 2016-07-25 14:11:29 UTC
Actually I think they do, sorry I'll shut up.
Comment 6 Matthew Waters (ystreet00) 2018-05-06 11:00:39 UTC
Are you planning to work on this?
Comment 7 GStreamer system administrator 2018-11-03 11:48:20 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gst-plugins-base/issues/279.