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

[Jun Li]

On Wed, May 8, 2019 at 12:32 AM Jun Li <junli1026 at gmail.com> wrote:

>
>
> 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
>

New version is sent out to address the issue.