[FFmpeg-devel] [PATCH v4 2/2] fftools/ffmpeg: Add stream metadata from first frame's metadata

Jun Li junli1026 at gmail.com
Wed May 8 10:32:05 EEST 2019


On Wed, May 8, 2019 at 12:09 AM Gyan <ffmpeg at gyani.pro> wrote:

>
>
> On 08-05-2019 12:25 PM, Jun Li wrote:
> > On Tue, May 7, 2019 at 11:40 PM Gyan <ffmpeg at gyani.pro> wrote:
> >
> >
> >> Also, is there a chance that there may be multiple sources for
> >> orientation data available for a given stream? If yes, what's the
> >> interaction? It looks like you append a new SD element.
> >>
> > Thanks Gyan for review !
> > Nicolas George asked the same question before. :)
> >
> > Yes, this patch can't handle the case every frame has its own
> orientation.
> >  From a technical perspective, it is absolutely possible, for example a
> > motion jpeg stream with different orientation value
> > on every frame. I think an ideal solution for this case is a filter doing
> > transformation per orientation every frame.
>
> I'm not referring to dynamic per-frame orientation.
>
> I'm wondering about a scenario where the container has orientation
> metadata and so does the packet payload (which can only be accessed by
> the decoder). Is there any possibility of that happening? What if they
> are different?
>
> Gyan
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".


Good point !
I do not know the answer. Let's keep it open for a while to see anyone in
the community has the answer.
If no one reply the question, I am going to add some logic to skip the sd
whenever the side_data is already set by by demuxer from container level.
Thanks Gyan !

Best Regards,
Jun


More information about the ffmpeg-devel mailing list