Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-13 Thread Vittorio Giovara
On Sat, Oct 8, 2016 at 1:39 PM, Michael Niedermayer wrote: > On Fri, Oct 07, 2016 at 10:38:22PM -0400, Vittorio Giovara wrote: >> On Fri, Oct 7, 2016 at 8:38 PM, Michael Niedermayer >> wrote: >> > On Fri, Oct 07, 2016 at 03:31:46PM -0400, Vittorio Giovara wrote: >> >> This matrix needs to be appl

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-08 Thread Michael Niedermayer
On Fri, Oct 07, 2016 at 10:38:22PM -0400, Vittorio Giovara wrote: > On Fri, Oct 7, 2016 at 8:38 PM, Michael Niedermayer > wrote: > > On Fri, Oct 07, 2016 at 03:31:46PM -0400, Vittorio Giovara wrote: > >> This matrix needs to be applied after all others have (currently only > >> display matrix from

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-07 Thread Vittorio Giovara
On Fri, Oct 7, 2016 at 8:38 PM, Michael Niedermayer wrote: > On Fri, Oct 07, 2016 at 03:31:46PM -0400, Vittorio Giovara wrote: >> This matrix needs to be applied after all others have (currently only >> display matrix from trak), but cannot be handled in movie box, since >> streams are not allocat

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-07 Thread Michael Niedermayer
On Sat, Oct 08, 2016 at 02:38:27AM +0200, Michael Niedermayer wrote: > On Fri, Oct 07, 2016 at 03:31:46PM -0400, Vittorio Giovara wrote: > > This matrix needs to be applied after all others have (currently only > > display matrix from trak), but cannot be handled in movie box, since > > streams are

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-07 Thread Michael Niedermayer
On Fri, Oct 07, 2016 at 03:31:46PM -0400, Vittorio Giovara wrote: > This matrix needs to be applied after all others have (currently only > display matrix from trak), but cannot be handled in movie box, since > streams are not allocated yet. > > So store it in main context and if not identity, app

[FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-07 Thread Vittorio Giovara
This matrix needs to be applied after all others have (currently only display matrix from trak), but cannot be handled in movie box, since streams are not allocated yet. So store it in main context and if not identity, apply it when appropriate, handling the case when trak display matrix is identi

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-07 Thread Carl Eugen Hoyos
2016-10-06 22:40 GMT+02:00 Vittorio Giovara : >>> https://www.dropbox.com/s/w2uu37o11rvoz1q/moviedispmat.mp4?dl=1 >> what is the intended / correct SAR / DAR for this sample ? > > without the patch Stream #0:0(und): Video: h264 (High) (avc1 / > 0x31637661), yuv420p, 540x576 [SAR 1:1 DAR 15:16]

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-06 Thread Michael Niedermayer
On Wed, Oct 05, 2016 at 05:25:33PM -0400, Vittorio Giovara wrote: > This matrix needs to be applied after all others have (currently only > display matrix from trak), but cannot be handled in movie box, since > streams are not allocated yet. > > So store it in main context and if not identity, app

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-06 Thread compn
On Thu, 6 Oct 2016 16:40:12 -0400 Vittorio Giovara wrote: > > what is the intended / correct SAR / DAR for this sample ? > > without the patch Stream #0:0(und): Video: h264 (High) (avc1 / > 0x31637661), yuv420p, 540x576 [SAR 1:1 DAR 15:16], 102 kb/s, 25 fps, > 25 tbr, 12800 tbn, 50 tbc (defa

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-06 Thread Vittorio Giovara
On Thu, Oct 6, 2016 at 4:07 PM, Michael Niedermayer wrote: > On Thu, Oct 06, 2016 at 12:12:07AM -0400, Vittorio Giovara wrote: >> On Wed, Oct 5, 2016 at 10:07 PM, Michael Niedermayer >> wrote: >> > On Wed, Oct 05, 2016 at 05:25:33PM -0400, Vittorio Giovara wrote: >> >> This matrix needs to be app

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-06 Thread Michael Niedermayer
On Thu, Oct 06, 2016 at 12:12:07AM -0400, Vittorio Giovara wrote: > On Wed, Oct 5, 2016 at 10:07 PM, Michael Niedermayer > wrote: > > On Wed, Oct 05, 2016 at 05:25:33PM -0400, Vittorio Giovara wrote: > >> This matrix needs to be applied after all others have (currently only > >> display matrix fro

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-05 Thread Vittorio Giovara
On Wed, Oct 5, 2016 at 10:07 PM, Michael Niedermayer wrote: > On Wed, Oct 05, 2016 at 05:25:33PM -0400, Vittorio Giovara wrote: >> This matrix needs to be applied after all others have (currently only >> display matrix from trak), but cannot be handled in movie box, since >> streams are not alloca

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-05 Thread Michael Niedermayer
On Wed, Oct 05, 2016 at 05:25:33PM -0400, Vittorio Giovara wrote: > This matrix needs to be applied after all others have (currently only > display matrix from trak), but cannot be handled in movie box, since > streams are not allocated yet. > > So store it in main context and if not identity, app

[FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-05 Thread Vittorio Giovara
This matrix needs to be applied after all others have (currently only display matrix from trak), but cannot be handled in movie box, since streams are not allocated yet. So store it in main context and if not identity, apply it when appropriate, handling the case when trak display matrix is identi