Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-12 Thread Eoff, Ullysses A
> -Original Message-
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of 
> Michael Niedermayer
> Sent: Friday, July 12, 2019 11:21 AM
> To: FFmpeg development discussions and patches 
> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> 
> > On Wed, Jul 10, 2019 at 07:54:32PM +, Eoff, Ullysses A wrote:
> > Although -reinit_filter 0 works in some cases, it is quite
> > verbose.  And I have found many other cases where it
> > doesn't work when we need to specify additional filters
> > (e.g. hwupload, hwdownload when doing qsv or vaapi
> > accelerated decoding with hw output formats).
> 
> can you fix the cases which do not work so the code works with
> -reinit_filter 0 ?
> 

 Linjie's patch should fix it: https://patchwork.ffmpeg.org/patch/13905/

> Thanks
> 
> [...]
> --
> Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> 
> Democracy is the form of government in which you can choose your dictator
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-12 Thread Michael Niedermayer
On Wed, Jul 10, 2019 at 07:54:32PM +, Eoff, Ullysses A wrote:
> > -Original Message-
> > From: Eoff, Ullysses A
> > Sent: Tuesday, July 09, 2019 6:45 PM
> > To: ffmpeg-devel@ffmpeg.org
> > Cc: Eoff, Ullysses A 
> > Subject: [PATCH] avfilter: add rawdump filter
> > 
> > This filter enables raw frames to be dumped to a
> > file before they are sent through the auto-inserted
> > scaler filter and useful when you want unscaled
> > raw frames in an output file.
> > 
> 
> The primary goal that I'm trying to achieve here is
> dump raw "unscaled" yuv to an output file while 
> decoding a multi-resolution video.  Unfortunately,
> the ffmpeg utility auto-inserts a scaler into the filter
> graph, so -f rawvideo doesn't suffice here since the
> decoded data passes through the scaler before it's
> handed off to the rawvideo filter.
> 
> I experimented with the alternative methods suggested
> in earlier feedback.  More specifically, "reinit_filter"
> and "split" were suggested.
> 
> For "split" filter, I couldn't find a way that it could bypass
> the auto-inserted scaler.
> 

> Although -reinit_filter 0 works in some cases, it is quite
> verbose.  And I have found many other cases where it
> doesn't work when we need to specify additional filters
> (e.g. hwupload, hwdownload when doing qsv or vaapi
> accelerated decoding with hw output formats).

can you fix the cases which do not work so the code works with
-reinit_filter 0 ?

Thanks

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Democracy is the form of government in which you can choose your dictator


signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Paul B Mahol
On 7/10/19, Eoff, Ullysses A  wrote:
>> -Original Message-
>> From: Eoff, Ullysses A
>> Sent: Tuesday, July 09, 2019 6:45 PM
>> To: ffmpeg-devel@ffmpeg.org
>> Cc: Eoff, Ullysses A 
>> Subject: [PATCH] avfilter: add rawdump filter
>>
>> This filter enables raw frames to be dumped to a
>> file before they are sent through the auto-inserted
>> scaler filter and useful when you want unscaled
>> raw frames in an output file.
>>
>
> The primary goal that I'm trying to achieve here is
> dump raw "unscaled" yuv to an output file while
> decoding a multi-resolution video.  Unfortunately,
> the ffmpeg utility auto-inserts a scaler into the filter
> graph, so -f rawvideo doesn't suffice here since the
> decoded data passes through the scaler before it's
> handed off to the rawvideo filter.
>
> I experimented with the alternative methods suggested
> in earlier feedback.  More specifically, "reinit_filter"
> and "split" were suggested.
>
> For "split" filter, I couldn't find a way that it could bypass
> the auto-inserted scaler.
>
> Although -reinit_filter 0 works in some cases, it is quite
> verbose.  And I have found many other cases where it
> doesn't work when we need to specify additional filters
> (e.g. hwupload, hwdownload when doing qsv or vaapi
> accelerated decoding with hw output formats).
>
> So, afaict, this patch is the only reliable method I can
> find to intercept the decoded data before it reaches
> the scaler for various use-cases (although the patch still
> needs a little bit of work as pointed out in previous
> replies).
>
> For added context, my organizations team and I contribute
> to both msdk/qsv and vaapi plugins for ffmpeg and
> gstreamer development.  The idea is we want a way to
> validate that direct decoder output is an exact bitmatch
> across every decoder solution that we contribute
> to (to provide a better and more consistent end-user
> experience regardless of end-user choice).  But since
> ffmpeg auto-inserts this "scaler" and not all scaling
> algo's are equal, it poses a challenge to verify.

You can use this in your own branch. I see no valid usecase except in
this testing scenario.

>
> Regards,
> U. Artie
>
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Eoff, Ullysses A
> -Original Message-
> From: Eoff, Ullysses A
> Sent: Tuesday, July 09, 2019 6:45 PM
> To: ffmpeg-devel@ffmpeg.org
> Cc: Eoff, Ullysses A 
> Subject: [PATCH] avfilter: add rawdump filter
> 
> This filter enables raw frames to be dumped to a
> file before they are sent through the auto-inserted
> scaler filter and useful when you want unscaled
> raw frames in an output file.
> 

The primary goal that I'm trying to achieve here is
dump raw "unscaled" yuv to an output file while 
decoding a multi-resolution video.  Unfortunately,
the ffmpeg utility auto-inserts a scaler into the filter
graph, so -f rawvideo doesn't suffice here since the
decoded data passes through the scaler before it's
handed off to the rawvideo filter.

I experimented with the alternative methods suggested
in earlier feedback.  More specifically, "reinit_filter"
and "split" were suggested.

For "split" filter, I couldn't find a way that it could bypass
the auto-inserted scaler.

Although -reinit_filter 0 works in some cases, it is quite
verbose.  And I have found many other cases where it
doesn't work when we need to specify additional filters
(e.g. hwupload, hwdownload when doing qsv or vaapi
accelerated decoding with hw output formats).

So, afaict, this patch is the only reliable method I can
find to intercept the decoded data before it reaches
the scaler for various use-cases (although the patch still
needs a little bit of work as pointed out in previous
replies).

For added context, my organizations team and I contribute
to both msdk/qsv and vaapi plugins for ffmpeg and
gstreamer development.  The idea is we want a way to
validate that direct decoder output is an exact bitmatch
across every decoder solution that we contribute
to (to provide a better and more consistent end-user
experience regardless of end-user choice).  But since
ffmpeg auto-inserts this "scaler" and not all scaling
algo's are equal, it poses a challenge to verify.

Regards,
U. Artie

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Gyan



On 10-07-2019 10:15 PM, Eoff, Ullysses A wrote:

-Original Message-
From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of Gyan
Sent: Wednesday, July 10, 2019 7:33 AM
To: ffmpeg-devel@ffmpeg.org
Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter



On 10-07-2019 07:51 PM, Eoff, Ullysses A wrote:

-Original Message-
From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of 
Hendrik Leppkes
Sent: Wednesday, July 10, 2019 4:13 AM
To: FFmpeg development discussions and patches 
Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

...
I believe using -reinit_filter 0 with ffmpeg can achieve such a
result., as long as you make sure all filtering done (preferably none)
is capable of on-the-fly changes.


Excellent!  I wouldn't have thought to try -reinit_filter 0 for such a 
scenario.  I
just tried it and it achieves the exact result we're looking for.  But it's 
unfortunate
that it generates a warning message for every frame processed.  Is there a 
simple
way to silence that message, yet still get other verbose logging (I don't want 
to
flood our automation logs)?

No. You'll have to upgrade the level in libavfilter/buffersrc.c


I see.  Would anyone object if I sent a patch to upgrade the level to 
AV_LOG_DEBUG?
Yes, I would. A video stream with changing frame properties is atypical, 
so the user should be warned about it. Your specific use case 
specifically seems to involve regularly, and perhaps purposely, varying 
prop changes, so I understand you find these messages to be bloat. An 
acceptable middle ground would be to add three additional state 
variables (called, say, last_w, last_h, last_pix_fmt) which prints the 
warning only when the props change relative to the last time it changed.


Gyan
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Eoff, Ullysses A
> -Original Message-
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of Gyan
> Sent: Wednesday, July 10, 2019 7:33 AM
> To: ffmpeg-devel@ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> 
> 
> 
> On 10-07-2019 07:51 PM, Eoff, Ullysses A wrote:
> >> -Original Message-
> >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of 
> >> Hendrik Leppkes
> >> Sent: Wednesday, July 10, 2019 4:13 AM
> >> To: FFmpeg development discussions and patches 
> >> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> >>
> >> ...
> >> I believe using -reinit_filter 0 with ffmpeg can achieve such a
> >> result., as long as you make sure all filtering done (preferably none)
> >> is capable of on-the-fly changes.
> >>
> > Excellent!  I wouldn't have thought to try -reinit_filter 0 for such a 
> > scenario.  I
> > just tried it and it achieves the exact result we're looking for.  But it's 
> > unfortunate
> > that it generates a warning message for every frame processed.  Is there a 
> > simple
> > way to silence that message, yet still get other verbose logging (I don't 
> > want to
> > flood our automation logs)?
> 
> No. You'll have to upgrade the level in libavfilter/buffersrc.c
> 

I see.  Would anyone object if I sent a patch to upgrade the level to 
AV_LOG_DEBUG?

> Gyan
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Eoff, Ullysses A
> -Original Message-
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of 
> Nicolas George
> Sent: Wednesday, July 10, 2019 7:53 AM
> To: FFmpeg development discussions and patches 
> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> 
> Eoff, Ullysses A (12019-07-10):
> > For the record, this "static" is a problem if multiple rawdump filters are
> > specified in the graph.  The initial reason for this was to open the file 
> > for
> > writing on the first init (to overwrite any pre-existing file), then open 
> > for
> > appending on successive reinit's (e.g. frame resolution changes).  Thus,
> > would need a better solution to achieve the same result without
> > influencing other instances of the filter.
> 
> The variable needs to be in the filter's private context, nothing more.
> 

Each time init is called (i.e. for reinit on resolution change), the private 
context is a
new instance.  On first init we want mode="w" for each rawdump filter in the
graph, but on reinit we want mode="a".  How do we achieve this when every
call to init passes a new private context instance.  I am not very familiar with
ffmpeg's programming API's, so maybe I'm missing something?

> Regards,
> 
> --
>   Nicolas George
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Eoff, Ullysses A
> -Original Message-
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of 
> Nicolas George
> Sent: Wednesday, July 10, 2019 8:05 AM
> To: FFmpeg development discussions and patches 
> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> 
> Eoff, Ullysses A (12019-07-10):
> > Agree.  There are many ways to specify format.  But there are some formats
> > which one might want to dump that some hw drivers can't output directly
> > without conversion.
> 
> In that case, your goal to avoid conversion is moot.
> 

The goal is to avoid scaling ;-)

> Regards,
> 
> --
>   Nicolas George
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Nicolas George
Eoff, Ullysses A (12019-07-10):
> Agree.  There are many ways to specify format.  But there are some formats
> which one might want to dump that some hw drivers can't output directly
> without conversion. 

In that case, your goal to avoid conversion is moot.

Regards,

-- 
  Nicolas George


signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Eoff, Ullysses A
> -Original Message-
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of Lynne
> Sent: Wednesday, July 10, 2019 7:52 AM
> To: FFmpeg development discussions and patches 
> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> 
> 
> 
> 
> Jul 10, 2019, 3:26 PM by ullysses.a.e...@intel.com:
> 
> >> -Original Message-
> >> From: Paul B Mahol [mailto:one...@gmail.com]
> >> Sent: Wednesday, July 10, 2019 7:20 AM
> >> To: Eoff, Ullysses A 
> >> Cc: FFmpeg development discussions and patches 
> >> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> >>
> >> On 7/10/19, Eoff, Ullysses A  wrote:
> >> >> -Original Message-
> >> >> From: Paul B Mahol [mailto:one...@gmail.com]
> >> >> Sent: Wednesday, July 10, 2019 3:25 AM
> >> >> To: FFmpeg development discussions and patches 
> >> >> Cc: Eoff, Ullysses A 
> >> >> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> >> >>
> >> >> On 7/10/19, Fu, Linjie  wrote:
> >> >> >> -Original Message-
> >> >> >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf
> >> >> >> Of Paul B Mahol
> >> >> >> Sent: Wednesday, July 10, 2019 15:55
> >> >> >> To: FFmpeg development discussions and patches  >> >> >> de...@ffmpeg.org>
> >> >> >> Cc: Eoff, Ullysses A 
> >> >> >> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> >> >> >>
> >> >> >> On 7/10/19, U. Artie Eoff  wrote:
> >> >> >> > This filter enables raw frames to be dumped to a
> >> >> >> > file before they are sent through the auto-inserted
> >> >> >> > scaler filter and useful when you want unscaled
> >> >> >> > raw frames in an output file.
> >> >> >> >
> >> >> >>
> >> >> >> Why?
> >> >> >>
> >> >> >> -f rawvideo doesn't work for you?
> >> >> >
> >> >> > unscaled raw video could be dumped through this filter in resolution
> >> >> > changing cases.
> >> >> >
> >> >>
> >> >> Maybe, I still fail how this filter is useful, when user can use split
> >> >> filter to dump video
> >> >> at any point in graph.
> >> >>
> >> >
> >> > If there are already other ways to achieve the same result without this
> >> > filter then that's great!  Essentially, I'm trying to achieve exactly 
> >> > what
> >> > Linjie stated so we can compare raw results across different decoder
> >> > solutions.  And the forced insertion of the auto scaler in ffmpeg makes
> >> > that difficult for multi-resolution cases.
> >>
> >> Please provide examples of filtergraph where you are supposed to use
> >> this "filter".
> >>
> >
> > ffmpeg -v verbose -hwaccel vaapi -vaapi_device /dev/dri/renderD128 \
> >  -i input.h264 -vf 'format=nv12,rawdump=file=./ffdump.yuv' -f null -
> >
> 
> You can make the decoder output nv12 directly via -hwaccel_output_format nv12

Agree.  There are many ways to specify format.  But there are some formats
which one might want to dump that some hw drivers can't output directly
without conversion. 

> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Eoff, Ullysses A
> -Original Message-
> From: Nicolas George [mailto:geo...@nsup.org]
> Sent: Wednesday, July 10, 2019 7:37 AM
> To: FFmpeg development discussions and patches 
> Cc: Eoff, Ullysses A 
> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> 
> U. Artie Eoff (12019-07-09):
> > +FILE *rawdump_file;
> 
> Independently from the question of whether this filter is useful or not,
> I think the dump should have used AVIO.
> 

Thanks for the tip!  Even though this patch is redundant and
unnecessary (based on feedback)... I will have a look at AVIO for
future changes/patches.

Cheers,
U. Artie

> Regards,
> 
> --
>   Nicolas George
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Nicolas George
Eoff, Ullysses A (12019-07-10):
> For the record, this "static" is a problem if multiple rawdump filters are
> specified in the graph.  The initial reason for this was to open the file for
> writing on the first init (to overwrite any pre-existing file), then open for
> appending on successive reinit's (e.g. frame resolution changes).  Thus,
> would need a better solution to achieve the same result without
> influencing other instances of the filter.

The variable needs to be in the filter's private context, nothing more.

Regards,

-- 
  Nicolas George


signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Lynne



Jul 10, 2019, 3:26 PM by ullysses.a.e...@intel.com:

>> -Original Message-
>> From: Paul B Mahol [mailto:one...@gmail.com]
>> Sent: Wednesday, July 10, 2019 7:20 AM
>> To: Eoff, Ullysses A 
>> Cc: FFmpeg development discussions and patches 
>> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
>>
>> On 7/10/19, Eoff, Ullysses A  wrote:
>> >> -Original Message-
>> >> From: Paul B Mahol [mailto:one...@gmail.com]
>> >> Sent: Wednesday, July 10, 2019 3:25 AM
>> >> To: FFmpeg development discussions and patches 
>> >> Cc: Eoff, Ullysses A 
>> >> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
>> >>
>> >> On 7/10/19, Fu, Linjie  wrote:
>> >> >> -Original Message-
>> >> >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf
>> >> >> Of Paul B Mahol
>> >> >> Sent: Wednesday, July 10, 2019 15:55
>> >> >> To: FFmpeg development discussions and patches > >> >> de...@ffmpeg.org>
>> >> >> Cc: Eoff, Ullysses A 
>> >> >> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
>> >> >>
>> >> >> On 7/10/19, U. Artie Eoff  wrote:
>> >> >> > This filter enables raw frames to be dumped to a
>> >> >> > file before they are sent through the auto-inserted
>> >> >> > scaler filter and useful when you want unscaled
>> >> >> > raw frames in an output file.
>> >> >> >
>> >> >>
>> >> >> Why?
>> >> >>
>> >> >> -f rawvideo doesn't work for you?
>> >> >
>> >> > unscaled raw video could be dumped through this filter in resolution
>> >> > changing cases.
>> >> >
>> >>
>> >> Maybe, I still fail how this filter is useful, when user can use split
>> >> filter to dump video
>> >> at any point in graph.
>> >>
>> >
>> > If there are already other ways to achieve the same result without this
>> > filter then that's great!  Essentially, I'm trying to achieve exactly what
>> > Linjie stated so we can compare raw results across different decoder
>> > solutions.  And the forced insertion of the auto scaler in ffmpeg makes
>> > that difficult for multi-resolution cases.
>>
>> Please provide examples of filtergraph where you are supposed to use
>> this "filter".
>>
>
> ffmpeg -v verbose -hwaccel vaapi -vaapi_device /dev/dri/renderD128 \
>  -i input.h264 -vf 'format=nv12,rawdump=file=./ffdump.yuv' -f null -
>

You can make the decoder output nv12 directly via -hwaccel_output_format nv12
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Eoff, Ullysses A
> From: Eoff, Ullysses A
> +static av_cold int rawdump_init(AVFilterContext *avctx)
> +{
> +RawDumpContext *ctx = avctx->priv;
> +static char *mode = "w";

For the record, this "static" is a problem if multiple rawdump filters are
specified in the graph.  The initial reason for this was to open the file for
writing on the first init (to overwrite any pre-existing file), then open for
appending on successive reinit's (e.g. frame resolution changes).  Thus,
would need a better solution to achieve the same result without
influencing other instances of the filter.

> +
> +ctx->rawdump_file = fopen(ctx->rawdump_file_str, mode);
> +
> +if (ctx->rawdump_file == NULL) {
> +int err = AVERROR(errno);
> +char buf[128];
> +av_strerror(err, buf, sizeof(buf));
> +av_log(ctx, AV_LOG_ERROR, "Failed to open %s for dumping: %s\n",
> +   ctx->rawdump_file_str, buf);
> +return err;
> +}
> +
> +av_log(ctx, AV_LOG_DEBUG, "Opened %s (mode:%s) for dumping.\n",
> +   ctx->rawdump_file_str, mode);
> +
> +mode = "a";
> +
> +return 0;
> +}
> +
> +static av_cold void rawdump_uninit(AVFilterContext *avctx)
> +{
> +RawDumpContext *ctx = avctx->priv;
> +
> +if (ctx->rawdump_file != NULL)
> +fclose(ctx->rawdump_file);
> +}
> +
> +#define OFFSET(x) offsetof(RawDumpContext, x)
> +#define FLAGS AV_OPT_FLAG_FILTERING_PARAM|AV_OPT_FLAG_VIDEO_PARAM
> +static const AVOption rawdump_options[] = {
> +{"file", "Set file where to dump raw frames", OFFSET(rawdump_file_str), 
> AV_OPT_TYPE_STRING, {.str=NULL}, 0, 0, FLAGS },
> +{ NULL }
> +};
> +
> +AVFILTER_DEFINE_CLASS(rawdump);
> +
> +static const AVFilterPad rawdump_inputs[] = {
> +{
> +.name = "default",
> +.type = AVMEDIA_TYPE_VIDEO,
> +.filter_frame = rawdump_filter_frame,
> +},
> +{ NULL }
> +};
> +
> +static const AVFilterPad rawdump_outputs[] = {
> +{
> +.name = "default",
> +.type = AVMEDIA_TYPE_VIDEO,
> +},
> +{ NULL }
> +};
> +
> +AVFilter ff_vf_rawdump = {
> +.name  = "rawdump",
> +.description   = NULL_IF_CONFIG_SMALL("Dump frames to file"),
> +.init  = rawdump_init,
> +.uninit= rawdump_uninit,
> +.query_formats = rawdump_query_formats,
> +.priv_size = sizeof(RawDumpContext),
> +.priv_class= _class,
> +.inputs= rawdump_inputs,
> +.outputs   = rawdump_outputs,
> +};
> --
> 2.20.1

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Nicolas George
U. Artie Eoff (12019-07-09):
> +FILE *rawdump_file;

Independently from the question of whether this filter is useful or not,
I think the dump should have used AVIO.

Regards,

-- 
  Nicolas George


signature.asc
Description: PGP signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Gyan



On 10-07-2019 07:51 PM, Eoff, Ullysses A wrote:

-Original Message-
From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of 
Hendrik Leppkes
Sent: Wednesday, July 10, 2019 4:13 AM
To: FFmpeg development discussions and patches 
Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

...
I believe using -reinit_filter 0 with ffmpeg can achieve such a
result., as long as you make sure all filtering done (preferably none)
is capable of on-the-fly changes.


Excellent!  I wouldn't have thought to try -reinit_filter 0 for such a 
scenario.  I
just tried it and it achieves the exact result we're looking for.  But it's 
unfortunate
that it generates a warning message for every frame processed.  Is there a 
simple
way to silence that message, yet still get other verbose logging (I don't want 
to
flood our automation logs)?


No. You'll have to upgrade the level in libavfilter/buffersrc.c

Gyan
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Paul B Mahol
On 7/10/19, Eoff, Ullysses A  wrote:
>> -Original Message-
>> From: Paul B Mahol [mailto:one...@gmail.com]
>> Sent: Wednesday, July 10, 2019 7:20 AM
>> To: Eoff, Ullysses A 
>> Cc: FFmpeg development discussions and patches 
>> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
>>
>> On 7/10/19, Eoff, Ullysses A  wrote:
>> >> -Original Message-
>> >> From: Paul B Mahol [mailto:one...@gmail.com]
>> >> Sent: Wednesday, July 10, 2019 3:25 AM
>> >> To: FFmpeg development discussions and patches
>> >> 
>> >> Cc: Eoff, Ullysses A 
>> >> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
>> >>
>> >> On 7/10/19, Fu, Linjie  wrote:
>> >> >> -Original Message-
>> >> >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On
>> >> >> Behalf
>> >> >> Of Paul B Mahol
>> >> >> Sent: Wednesday, July 10, 2019 15:55
>> >> >> To: FFmpeg development discussions and patches > >> >> de...@ffmpeg.org>
>> >> >> Cc: Eoff, Ullysses A 
>> >> >> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
>> >> >>
>> >> >> On 7/10/19, U. Artie Eoff  wrote:
>> >> >> > This filter enables raw frames to be dumped to a
>> >> >> > file before they are sent through the auto-inserted
>> >> >> > scaler filter and useful when you want unscaled
>> >> >> > raw frames in an output file.
>> >> >> >
>> >> >>
>> >> >> Why?
>> >> >>
>> >> >> -f rawvideo doesn't work for you?
>> >> >
>> >> > unscaled raw video could be dumped through this filter in resolution
>> >> > changing cases.
>> >> >
>> >>
>> >> Maybe, I still fail how this filter is useful, when user can use split
>> >> filter to dump video
>> >> at any point in graph.
>> >>
>> >
>> > If there are already other ways to achieve the same result without this
>> > filter then that's great!  Essentially, I'm trying to achieve exactly
>> > what
>> > Linjie stated so we can compare raw results across different decoder
>> > solutions.  And the forced insertion of the auto scaler in ffmpeg makes
>> > that difficult for multi-resolution cases.
>>
>> Please provide examples of filtergraph where you are supposed to use
>> this "filter".
>>
>
> ffmpeg -v verbose -hwaccel vaapi -vaapi_device /dev/dri/renderD128 \
>   -i input.h264 -vf 'format=nv12,rawdump=file=./ffdump.yuv' -f null -
>
>> >
>> > Could you point me to an example of using split to achieve this?
>>
>> See documentation of split filter.
>
> Thanks.  I will take a look and experiment with it.

The resolution changing support is big hack, and should not be relied upon.
Use some other tool. Like gstreamer.

>
>>
>> >
>> > U. Artie
>> >
>> >>
>> >> > IMHO, it will be great if there are some options available to
>> >> > disable/enable
>> >> > the auto scaling filter
>> >> > in pipeline, thus the whole stream doesn't need to be scaled into
>> >> > the
>> >> > same
>> >> > size of the first frame.
>> >> > The difference in scale algorithms will make it hard to compare the
>> >> > output
>> >> > with other decoder.
>> >> >
>> >> > - linjie
>> >> > ___
>> >> > ffmpeg-devel mailing list
>> >> > ffmpeg-devel@ffmpeg.org
>> >> > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>> >> >
>> >> > To unsubscribe, visit link above, or email
>> >> > ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
>> >
>
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Eoff, Ullysses A
> -Original Message-
> From: Paul B Mahol [mailto:one...@gmail.com]
> Sent: Wednesday, July 10, 2019 7:20 AM
> To: Eoff, Ullysses A 
> Cc: FFmpeg development discussions and patches 
> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> 
> On 7/10/19, Eoff, Ullysses A  wrote:
> >> -Original Message-
> >> From: Paul B Mahol [mailto:one...@gmail.com]
> >> Sent: Wednesday, July 10, 2019 3:25 AM
> >> To: FFmpeg development discussions and patches 
> >> Cc: Eoff, Ullysses A 
> >> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> >>
> >> On 7/10/19, Fu, Linjie  wrote:
> >> >> -Original Message-
> >> >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf
> >> >> Of Paul B Mahol
> >> >> Sent: Wednesday, July 10, 2019 15:55
> >> >> To: FFmpeg development discussions and patches  >> >> de...@ffmpeg.org>
> >> >> Cc: Eoff, Ullysses A 
> >> >> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> >> >>
> >> >> On 7/10/19, U. Artie Eoff  wrote:
> >> >> > This filter enables raw frames to be dumped to a
> >> >> > file before they are sent through the auto-inserted
> >> >> > scaler filter and useful when you want unscaled
> >> >> > raw frames in an output file.
> >> >> >
> >> >>
> >> >> Why?
> >> >>
> >> >> -f rawvideo doesn't work for you?
> >> >
> >> > unscaled raw video could be dumped through this filter in resolution
> >> > changing cases.
> >> >
> >>
> >> Maybe, I still fail how this filter is useful, when user can use split
> >> filter to dump video
> >> at any point in graph.
> >>
> >
> > If there are already other ways to achieve the same result without this
> > filter then that's great!  Essentially, I'm trying to achieve exactly what
> > Linjie stated so we can compare raw results across different decoder
> > solutions.  And the forced insertion of the auto scaler in ffmpeg makes
> > that difficult for multi-resolution cases.
> 
> Please provide examples of filtergraph where you are supposed to use
> this "filter".
> 

ffmpeg -v verbose -hwaccel vaapi -vaapi_device /dev/dri/renderD128 \
  -i input.h264 -vf 'format=nv12,rawdump=file=./ffdump.yuv' -f null -

> >
> > Could you point me to an example of using split to achieve this?
> 
> See documentation of split filter.

Thanks.  I will take a look and experiment with it.

> 
> >
> > U. Artie
> >
> >>
> >> > IMHO, it will be great if there are some options available to
> >> > disable/enable
> >> > the auto scaling filter
> >> > in pipeline, thus the whole stream doesn't need to be scaled into the
> >> > same
> >> > size of the first frame.
> >> > The difference in scale algorithms will make it hard to compare the
> >> > output
> >> > with other decoder.
> >> >
> >> > - linjie
> >> > ___
> >> > ffmpeg-devel mailing list
> >> > ffmpeg-devel@ffmpeg.org
> >> > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >> >
> >> > To unsubscribe, visit link above, or email
> >> > ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
> >
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Eoff, Ullysses A
> -Original Message-
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of 
> Hendrik Leppkes
> Sent: Wednesday, July 10, 2019 4:13 AM
> To: FFmpeg development discussions and patches 
> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> 
> On Wed, Jul 10, 2019 at 12:13 PM Fu, Linjie  wrote:
> >
> > > -Original Message-
> > > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf
> > > Of Paul B Mahol
> > > Sent: Wednesday, July 10, 2019 15:55
> > > To: FFmpeg development discussions and patches  > > de...@ffmpeg.org>
> > > Cc: Eoff, Ullysses A 
> > > Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> > >
> > > On 7/10/19, U. Artie Eoff  wrote:
> > > > This filter enables raw frames to be dumped to a
> > > > file before they are sent through the auto-inserted
> > > > scaler filter and useful when you want unscaled
> > > > raw frames in an output file.
> > > >
> > >
> > > Why?
> > >
> > > -f rawvideo doesn't work for you?
> >
> > unscaled raw video could be dumped through this filter in resolution 
> > changing cases.
> >
> > IMHO, it will be great if there are some options available to 
> > disable/enable the auto scaling filter
> > in pipeline, thus the whole stream doesn't need to be scaled into the same 
> > size of the first frame.
> > The difference in scale algorithms will make it hard to compare the output 
> > with other decoder.
> >
> 
> I believe using -reinit_filter 0 with ffmpeg can achieve such a
> result., as long as you make sure all filtering done (preferably none)
> is capable of on-the-fly changes.
> 

Excellent!  I wouldn't have thought to try -reinit_filter 0 for such a 
scenario.  I
just tried it and it achieves the exact result we're looking for.  But it's 
unfortunate
that it generates a warning message for every frame processed.  Is there a 
simple
way to silence that message, yet still get other verbose logging (I don't want 
to
flood our automation logs)?

U. Artie

> - Hendrik
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Paul B Mahol
On 7/10/19, Eoff, Ullysses A  wrote:
>> -Original Message-
>> From: Paul B Mahol [mailto:one...@gmail.com]
>> Sent: Wednesday, July 10, 2019 3:25 AM
>> To: FFmpeg development discussions and patches 
>> Cc: Eoff, Ullysses A 
>> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
>>
>> On 7/10/19, Fu, Linjie  wrote:
>> >> -Original Message-
>> >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf
>> >> Of Paul B Mahol
>> >> Sent: Wednesday, July 10, 2019 15:55
>> >> To: FFmpeg development discussions and patches > >> de...@ffmpeg.org>
>> >> Cc: Eoff, Ullysses A 
>> >> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
>> >>
>> >> On 7/10/19, U. Artie Eoff  wrote:
>> >> > This filter enables raw frames to be dumped to a
>> >> > file before they are sent through the auto-inserted
>> >> > scaler filter and useful when you want unscaled
>> >> > raw frames in an output file.
>> >> >
>> >>
>> >> Why?
>> >>
>> >> -f rawvideo doesn't work for you?
>> >
>> > unscaled raw video could be dumped through this filter in resolution
>> > changing cases.
>> >
>>
>> Maybe, I still fail how this filter is useful, when user can use split
>> filter to dump video
>> at any point in graph.
>>
>
> If there are already other ways to achieve the same result without this
> filter then that's great!  Essentially, I'm trying to achieve exactly what
> Linjie stated so we can compare raw results across different decoder
> solutions.  And the forced insertion of the auto scaler in ffmpeg makes
> that difficult for multi-resolution cases.

Please provide examples of filtergraph where you are supposed to use
this "filter".

>
> Could you point me to an example of using split to achieve this?

See documentation of split filter.

>
> U. Artie
>
>>
>> > IMHO, it will be great if there are some options available to
>> > disable/enable
>> > the auto scaling filter
>> > in pipeline, thus the whole stream doesn't need to be scaled into the
>> > same
>> > size of the first frame.
>> > The difference in scale algorithms will make it hard to compare the
>> > output
>> > with other decoder.
>> >
>> > - linjie
>> > ___
>> > ffmpeg-devel mailing list
>> > ffmpeg-devel@ffmpeg.org
>> > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>> >
>> > To unsubscribe, visit link above, or email
>> > ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
>
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Eoff, Ullysses A
> -Original Message-
> From: Paul B Mahol [mailto:one...@gmail.com]
> Sent: Wednesday, July 10, 2019 3:25 AM
> To: FFmpeg development discussions and patches 
> Cc: Eoff, Ullysses A 
> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> 
> On 7/10/19, Fu, Linjie  wrote:
> >> -Original Message-
> >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf
> >> Of Paul B Mahol
> >> Sent: Wednesday, July 10, 2019 15:55
> >> To: FFmpeg development discussions and patches  >> de...@ffmpeg.org>
> >> Cc: Eoff, Ullysses A 
> >> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> >>
> >> On 7/10/19, U. Artie Eoff  wrote:
> >> > This filter enables raw frames to be dumped to a
> >> > file before they are sent through the auto-inserted
> >> > scaler filter and useful when you want unscaled
> >> > raw frames in an output file.
> >> >
> >>
> >> Why?
> >>
> >> -f rawvideo doesn't work for you?
> >
> > unscaled raw video could be dumped through this filter in resolution
> > changing cases.
> >
> 
> Maybe, I still fail how this filter is useful, when user can use split
> filter to dump video
> at any point in graph.
> 

If there are already other ways to achieve the same result without this
filter then that's great!  Essentially, I'm trying to achieve exactly what
Linjie stated so we can compare raw results across different decoder
solutions.  And the forced insertion of the auto scaler in ffmpeg makes
that difficult for multi-resolution cases.

Could you point me to an example of using split to achieve this?

U. Artie

> 
> > IMHO, it will be great if there are some options available to disable/enable
> > the auto scaling filter
> > in pipeline, thus the whole stream doesn't need to be scaled into the same
> > size of the first frame.
> > The difference in scale algorithms will make it hard to compare the output
> > with other decoder.
> >
> > - linjie
> > ___
> > ffmpeg-devel mailing list
> > ffmpeg-devel@ffmpeg.org
> > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >
> > To unsubscribe, visit link above, or email
> > ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Hendrik Leppkes
On Wed, Jul 10, 2019 at 12:13 PM Fu, Linjie  wrote:
>
> > -Original Message-
> > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf
> > Of Paul B Mahol
> > Sent: Wednesday, July 10, 2019 15:55
> > To: FFmpeg development discussions and patches  > de...@ffmpeg.org>
> > Cc: Eoff, Ullysses A 
> > Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> >
> > On 7/10/19, U. Artie Eoff  wrote:
> > > This filter enables raw frames to be dumped to a
> > > file before they are sent through the auto-inserted
> > > scaler filter and useful when you want unscaled
> > > raw frames in an output file.
> > >
> >
> > Why?
> >
> > -f rawvideo doesn't work for you?
>
> unscaled raw video could be dumped through this filter in resolution changing 
> cases.
>
> IMHO, it will be great if there are some options available to disable/enable 
> the auto scaling filter
> in pipeline, thus the whole stream doesn't need to be scaled into the same 
> size of the first frame.
> The difference in scale algorithms will make it hard to compare the output 
> with other decoder.
>

I believe using -reinit_filter 0 with ffmpeg can achieve such a
result., as long as you make sure all filtering done (preferably none)
is capable of on-the-fly changes.

- Hendrik
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Paul B Mahol
On 7/10/19, Fu, Linjie  wrote:
>> -Original Message-
>> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf
>> Of Paul B Mahol
>> Sent: Wednesday, July 10, 2019 15:55
>> To: FFmpeg development discussions and patches > de...@ffmpeg.org>
>> Cc: Eoff, Ullysses A 
>> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
>>
>> On 7/10/19, U. Artie Eoff  wrote:
>> > This filter enables raw frames to be dumped to a
>> > file before they are sent through the auto-inserted
>> > scaler filter and useful when you want unscaled
>> > raw frames in an output file.
>> >
>>
>> Why?
>>
>> -f rawvideo doesn't work for you?
>
> unscaled raw video could be dumped through this filter in resolution
> changing cases.
>

Maybe, I still fail how this filter is useful, when user can use split
filter to dump video
at any point in graph.


> IMHO, it will be great if there are some options available to disable/enable
> the auto scaling filter
> in pipeline, thus the whole stream doesn't need to be scaled into the same
> size of the first frame.
> The difference in scale algorithms will make it hard to compare the output
> with other decoder.
>
> - linjie
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Fu, Linjie
> -Original Message-
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf
> Of Paul B Mahol
> Sent: Wednesday, July 10, 2019 15:55
> To: FFmpeg development discussions and patches  de...@ffmpeg.org>
> Cc: Eoff, Ullysses A 
> Subject: Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter
> 
> On 7/10/19, U. Artie Eoff  wrote:
> > This filter enables raw frames to be dumped to a
> > file before they are sent through the auto-inserted
> > scaler filter and useful when you want unscaled
> > raw frames in an output file.
> >
> 
> Why?
> 
> -f rawvideo doesn't work for you?

unscaled raw video could be dumped through this filter in resolution changing 
cases.

IMHO, it will be great if there are some options available to disable/enable 
the auto scaling filter
in pipeline, thus the whole stream doesn't need to be scaled into the same size 
of the first frame. 
The difference in scale algorithms will make it hard to compare the output with 
other decoder.

- linjie
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter: add rawdump filter

2019-07-10 Thread Paul B Mahol
On 7/10/19, U. Artie Eoff  wrote:
> This filter enables raw frames to be dumped to a
> file before they are sent through the auto-inserted
> scaler filter and useful when you want unscaled
> raw frames in an output file.
>

Why?

-f rawvideo doesn't work for you?
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".