Ticket #9861 (closed Patch - Feature: fixed)
Opened 2 years ago
Last modified 10 months ago
[PATCH] Rotate IPhone videos for proper playback
| Reported by: | Dave Badia <dbadia@…> | Owned by: | stichnot |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.26 |
| Component: | MythTV - Video Playback | Version: | Master Head |
| Severity: | medium | Keywords: | |
| Cc: | Ticket locked: | no |
Description
IPhone videos can appear upside down in myth depending on the orientation of the phone when the video was captured. This patch checks the MOV metadata to detect the orientation and flips the frames if necessary. The MOV metadata is read by ffmpeg so this upstream ffmpeg patch is required to be merged into myth: http://git.videolan.org/?p=ffmpeg.git;a=commit;h=d3cef0a85b7d3fd133a3349341646fe15aeb0030
The flip is executed differently depending on the decoding/rendering method:
VDPAU: Simple swap of coordinates as suggested by the nvidia documentation Software: Added a new filter called vflip. This filter will be automatically applied when needed. XVMC: Didn't bother since it's going away
Test both active types and it's working well for me
This patch can be safely merged into myth prior to the ffmpeg patch - it just won't execute until that's done.
Attachments
Change History
comment:1 Changed 2 years ago by robertm
- Owner changed from robertm to janne
- Type changed from Patch - Bug Fix to Patch - Feature
- Component changed from Plugin - MythVideo to MythTV - Video Playback
This doesn't actually have anything to do with mythvideo, changing component and type (since it's not a bug fix either)
comment:2 Changed 16 months ago by beirdo
- Owner changed from janne to markk
- Status changed from new to assigned
Sending this somewhere where it might get some attention.
Changed 16 months ago by Dave Badia <dbadia@…>
- Attachment mov-flip-both2 added
new patch that applies cleanly to master
comment:4 Changed 11 months ago by stichnot
- Owner set to stichnot
- Status changed from assigned to accepted
Dave, do you have an updated patch that you can upload?
Changed 11 months ago by Dave Badia <dbadia@…>
- Attachment mov-flip-both3 added
New patch that applies cleanly to master
comment:5 Changed 10 months ago by Dave Badia <dbadia@…>
- Status changed from accepted to closed
- Resolution set to fixed
