Re: [FFmpeg-devel] [PATCH v3] Add "sar" alias to "aspect" option of video encoders

2016-05-05 Thread Michael Niedermayer
On Thu, May 05, 2016 at 03:37:49PM +0300, Andrey Utkin wrote: > It is impossible to pass "aspect" parameter to encoder from ffmpeg CLI > because option from lavc/options_table.h is eclipsed by option with same > name in ffmpeg_opt.c, which has different meaning (DAR, not SAR). > --- > > v3: added

Re: [FFmpeg-devel] [PATCH] parsing code: add '\r' as whitespace

2016-05-05 Thread Muhammad Faiz
On Fri, May 6, 2016 at 4:14 AM, Michael Niedermayer wrote: > On Thu, May 05, 2016 at 09:49:18PM +0700, Muhammad Faiz wrote: >> for compatibility with platforms that treat it >> as newline > > i think this is ok but it should bump the minor version number of > the libs for which this affects public

Re: [FFmpeg-devel] [PATCH 57/57] support for matrox m703 mpeg-2

2016-05-05 Thread Michael Niedermayer
On Fri, May 06, 2016 at 01:15:02AM +0200, Michael Niedermayer wrote: > On Thu, May 05, 2016 at 08:18:02PM +0200, Piotr Bandurski wrote: > > > > a 10bit file with some smooth 10bit gradients would make it > > > > easier to assign the LSBs correctly. The existing file has half of > > > > them 0 > >

Re: [FFmpeg-devel] [PATCH 1/2] vc2enc: prevent random data

2016-05-05 Thread Rostislav Pehlivanov
On 5 May 2016 at 10:06, Christophe Gisquet wrote: > The slice prefix is 0 in the reference encoder and the decoder ignores it. > Writing 0 there seems like the best temporary solution. > > The padding could have contained uninitialized data, but reference VC2 > encoders put 0xFF there, hence the

Re: [FFmpeg-devel] [PATCH 57/57] support for matrox m703 mpeg-2

2016-05-05 Thread Michael Niedermayer
On Thu, May 05, 2016 at 08:18:02PM +0200, Piotr Bandurski wrote: > > > a 10bit file with some smooth 10bit gradients would make it > > > easier to assign the LSBs correctly. The existing file has half of > > > them 0 > > > > You mean something like this?: > > > > http://bennettfeely.com/gradient

Re: [FFmpeg-devel] [PATCH 57/57] support for matrox m703 mpeg-2

2016-05-05 Thread Michael Niedermayer
On Thu, May 05, 2016 at 10:51:10PM +0200, Piotr Bandurski wrote: > > a 10bit file with some smooth 10bit gradients would make it > > easier to assign the LSBs correctly. The existing file has half of > > them 0 > > > > [...] > > I reencoded this: > > https://samples.mplayerhq.hu/V-codecs/v210/v

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add M101 decoder

2016-05-05 Thread Michael Niedermayer
On Thu, May 05, 2016 at 10:52:35PM +0200, Piotr Bandurski wrote: > > +AVCodec ff_m101_decoder = { > > +.name = "m101", > > +.long_name = NULL_IF_CONFIG_SMALL("matrox uncompressed sd"), > > "Matrox Uncompressed SD" would look better. changed thx [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add M101 decoder

2016-05-05 Thread Michael Niedermayer
On Thu, May 05, 2016 at 06:57:26PM +, Carl Eugen Hoyos wrote: > Michael Niedermayer niedermayer.cc> writes: > > > TODO: find out which LSB for 10bit go where > > Please also mention ticket #2611. locally changed [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0F

Re: [FFmpeg-devel] [PATCH] parsing code: add '\r' as whitespace

2016-05-05 Thread Michael Niedermayer
On Thu, May 05, 2016 at 09:49:18PM +0700, Muhammad Faiz wrote: > for compatibility with platforms that treat it > as newline i think this is ok but it should bump the minor version number of the libs for which this affects public interface also should be split per lib [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add M101 decoder

2016-05-05 Thread Piotr Bandurski
> +AVCodec ff_m101_decoder = { > +.name = "m101", > +.long_name = NULL_IF_CONFIG_SMALL("matrox uncompressed sd"), "Matrox Uncompressed SD" would look better. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/

Re: [FFmpeg-devel] [PATCH 57/57] support for matrox m703 mpeg-2

2016-05-05 Thread Piotr Bandurski
> a 10bit file with some smooth 10bit gradients would make it > easier to assign the LSBs correctly. The existing file has half of > them 0 > > [...] I reencoded this: https://samples.mplayerhq.hu/V-codecs/v210/v210_720p.avi to M101: https://www.datafilehost.com/d/98759d14 ___

Re: [FFmpeg-devel] [PATCH] Add test for vorbis encode

2016-05-05 Thread Michael Niedermayer
On Thu, May 05, 2016 at 05:03:00PM +, Petru Rares Sincraian wrote: > Here is the patch, > > Sorry. > > On 05/05/16 19:01, Petru Rares Sincraian wrote: > > Hi there, > > > > Here is a patch for the vorbis encoder. I only tried this test on a > > x64 architecture, maybe I need to change the FU

Re: [FFmpeg-devel] [PATCH 3/3] fate: add test for concat with and without automatic filtering.

2016-05-05 Thread James Almer
On 5/5/2016 3:22 PM, Nicolas George wrote: > Le septidi 17 floréal, an CCXXIV, James Almer a écrit : >> Still fails >> >> $ /ffmpeg/build/ffprobe -hide_banner -bitexact -show_streams -show_packets >> -fflags keepside -safe 0 -show_data_hash crc32 >> tests/data/fate/concat-demuxer-convert-lavf-h26

Re: [FFmpeg-devel] [PATCH] added support for hardware assist H264 video encoding for the Raspberry Pi

2016-05-05 Thread Aman Gupta
I confirmed that this patch works on the rpi3, when compiling ffmpeg with `--enable-omx --enable-omx-rpi` and using `ffmpeg -v:c h264_omx` I had to make some minor changes for ffmpeg compatibility which I pushed to my rebase branch. Aman On Thu, Apr 21, 2016 at 10:14 AM, Amancio Hasty wrote: >

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add M101 decoder

2016-05-05 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > TODO: find out which LSB for 10bit go where Please also mention ticket #2611. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 3/3] fate: add test for concat with and without automatic filtering.

2016-05-05 Thread Nicolas George
Le septidi 17 floréal, an CCXXIV, James Almer a écrit : > Still fails > > $ /ffmpeg/build/ffprobe -hide_banner -bitexact -show_streams -show_packets > -fflags keepside -safe 0 -show_data_hash crc32 > tests/data/fate/concat-demuxer-convert-lavf-h264.ffconcat > [concat @ 0102c980] Impossib

Re: [FFmpeg-devel] [PATCH 57/57] support for matrox m703 mpeg-2

2016-05-05 Thread Piotr Bandurski
> > a 10bit file with some smooth 10bit gradients would make it > > easier to assign the LSBs correctly. The existing file has half of > > them 0 > > You mean something like this?: > > http://bennettfeely.com/gradients/img/gradient_24.png If not then please point me to something useful and I wi

Re: [FFmpeg-devel] [PATCH 57/57] support for matrox m703 mpeg-2

2016-05-05 Thread Piotr Bandurski
Dnia 5 maja 2016 19:47 Michael Niedermayer napisał(a): > On Tue, May 03, 2016 at 12:10:22PM +0200, Piotr Bandurski wrote: > > > > > > > > Dnia 3 maja 2016 11:54 Piotr Bandurski napisał(a): > > > > > > > > > > > > > Hi, > > > > > > > as long as we are throwing out questions. > > > > >

Re: [FFmpeg-devel] [PATCH 57/57] support for matrox m703 mpeg-2

2016-05-05 Thread Michael Niedermayer
On Tue, May 03, 2016 at 12:10:22PM +0200, Piotr Bandurski wrote: > > > > Dnia 3 maja 2016 11:54 Piotr Bandurski napisał(a): > > > > > > > > Hi, > > > > > as long as we are throwing out questions. > > > > > > the m70x stuff is similar to m10x . and uses same binary codec: > > > > > > VIDC

[FFmpeg-devel] [PATCH 1/2] avcodec: add M101 decoder

2016-05-05 Thread Michael Niedermayer
TODO: find out which LSB for 10bit go where Signed-off-by: Michael Niedermayer --- libavcodec/Makefile |1 + libavcodec/allcodecs.c |1 + libavcodec/avcodec.h|1 + libavcodec/codec_desc.c |7 +++ libavcodec/m101.c | 120

[FFmpeg-devel] [PATCH 2/2] avformat/riff: add M101

2016-05-05 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/riff.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/riff.c b/libavformat/riff.c index 70baea1..ccbe608 100644 --- a/libavformat/riff.c +++ b/libavformat/riff.c @@ -419,6 +419,8 @@ const AVCodecTag ff_codec_bmp_tags[] = {

Re: [FFmpeg-devel] [PATCH] Bonk decoder and demuxer

2016-05-05 Thread Piotr Bandurski
> On 5/5/16, Piotr Bandurski wrote: > >> > > > > > > > > Improved version attached. > >> > > > > >> > > > I found another bug, this time in lossy mode - the file encoded as > >> > > > "joint stereo" decodes after a few seconds into noise: > >> > > > > >> > > > https://www.datafilehost.com/d/c2e8b3

Re: [FFmpeg-devel] [PATCH] Add test for vorbis encode

2016-05-05 Thread Petru Rares Sincraian
Here is the patch, Sorry. On 05/05/16 19:01, Petru Rares Sincraian wrote: > Hi there, > > Here is a patch for the vorbis encoder. I only tried this test on a > x64 architecture, maybe I need to change the FUZZ parameter. > > Can anybody test in another architecture? > > > Many thanks, > Petru Ra

[FFmpeg-devel] [PATCH] Add test for vorbis encode

2016-05-05 Thread Petru Rares Sincraian
Hi there, Here is a patch for the vorbis encoder. I only tried this test on a x64 architecture, maybe I need to change the FUZZ parameter. Can anybody test in another architecture? Many thanks, Petru Rares. ___ ffmpeg-devel mailing list ffmpeg-devel@

Re: [FFmpeg-devel] [PATCH] Bonk decoder and demuxer

2016-05-05 Thread Paul B Mahol
On 5/5/16, Piotr Bandurski wrote: >> > > > > > > > > Improved version attached. >> > > > >> > > > I found another bug, this time in lossy mode - the file encoded as >> > > > "joint stereo" decodes after a few seconds into noise: >> > > > >> > > > https://www.datafilehost.com/d/c2e8b332 >> > > >> >

Re: [FFmpeg-devel] [PATCH] Bonk decoder and demuxer

2016-05-05 Thread Piotr Bandurski
> > > > > > > > > Improved version attached. > > > > > > > > I found another bug, this time in lossy mode - the file encoded as > > > > "joint stereo" decodes after a few seconds into noise: > > > > > > > > https://www.datafilehost.com/d/c2e8b332 > > > > > > here are a few more samples: > > >

Re: [FFmpeg-devel] [PATCH 3/3] fate: add test for concat with and without automatic filtering.

2016-05-05 Thread James Almer
On 5/5/2016 6:58 AM, Nicolas George wrote: > Le decadi 10 floréal, an CCXXIV, James Almer a écrit : >> I think this needs to be $(TARGET_SAMPLES). > > Fixed. > >> You should be able to simplify this using FATE_FFPROBE and >> FATE_SAMPLES_FFPROBE. >> It would also remove the need to specify the f

[FFmpeg-devel] [PATCH] parsing code: add '\r' as whitespace

2016-05-05 Thread Muhammad Faiz
for compatibility with platforms that treat it as newline Signed-off-by: Muhammad Faiz --- libavfilter/graphparser.c | 2 +- libavutil/avstring.c | 2 +- libavutil/bprint.c| 2 +- libavutil/opt.c | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/libavf

Re: [FFmpeg-devel] [PATCH] lavd/avfoundation: use AVCodecParameters

2016-05-05 Thread Thilo Borgmann
Am 05.05.16 um 15:36 schrieb Richard Kern: > >> On May 5, 2016, at 9:35 AM, Derek Buitenhuis >> wrote: >> >> On 5/5/2016 2:08 PM, Rick Kern wrote: >>> Fixes "Could not find codec parameters for stream" error (#5494) >>> >>> Signed-off-by: Rick Kern >>> --- >>> libavdevice/avfoundation.m | 26 ++

Re: [FFmpeg-devel] [PATCH] lavd/avfoundation: use AVCodecParameters

2016-05-05 Thread Richard Kern
> On May 5, 2016, at 9:35 AM, Derek Buitenhuis > wrote: > > On 5/5/2016 2:08 PM, Rick Kern wrote: >> Fixes "Could not find codec parameters for stream" error (#5494) >> >> Signed-off-by: Rick Kern >> --- >> libavdevice/avfoundation.m | 26 +- >> 1 file changed, 13 inser

Re: [FFmpeg-devel] [PATCH] lavd/avfoundation: use AVCodecParameters

2016-05-05 Thread Derek Buitenhuis
On 5/5/2016 2:08 PM, Rick Kern wrote: > Fixes "Could not find codec parameters for stream" error (#5494) > > Signed-off-by: Rick Kern > --- > libavdevice/avfoundation.m | 26 +- > 1 file changed, 13 insertions(+), 13 deletions(-) Patch looks good to me. I assume you have

Re: [FFmpeg-devel] [PATCH] Bonk decoder and demuxer

2016-05-05 Thread Piotr Bandurski
> > > > > > > Improved version attached. > > > > > > I found another bug, this time in lossy mode - the file encoded as "joint > > > stereo" decodes after a few seconds into noise: > > > > > > https://www.datafilehost.com/d/c2e8b332 > > > > here are a few more samples: > > > > https://www.data

[FFmpeg-devel] [PATCH] lavd/avfoundation: use AVCodecParameters

2016-05-05 Thread Rick Kern
Fixes "Could not find codec parameters for stream" error (#5494) Signed-off-by: Rick Kern --- libavdevice/avfoundation.m | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/libavdevice/avfoundation.m b/libavdevice/avfoundation.m index 763e675..8132278

[FFmpeg-devel] [PATCH v3] Add "sar" alias to "aspect" option of video encoders

2016-05-05 Thread Andrey Utkin
It is impossible to pass "aspect" parameter to encoder from ffmpeg CLI because option from lavc/options_table.h is eclipsed by option with same name in ffmpeg_opt.c, which has different meaning (DAR, not SAR). --- v3: added doc update doc/codecs.texi | 3 +++ libavcodec/opti

Re: [FFmpeg-devel] [PATCH] vf_colorspace: add floyd-steinberg dithering option to full conversion.

2016-05-05 Thread Michael Niedermayer
On Tue, May 03, 2016 at 01:53:42PM -0400, Ronald S. Bultje wrote: > --- > doc/filters.texi | 13 > libavfilter/colorspacedsp.c | 12 > libavfilter/colorspacedsp.h | 6 ++ > libavfilter/colorspacedsp_template.c | 128 > +++

Re: [FFmpeg-devel] [RFC]lavc/adpcm: Support baby monitor adpcm_ms

2016-05-05 Thread Carl Eugen Hoyos
Paul B Mahol gmail.com> writes: > >> There is clearly overread if you ignore > >> first two bytes. > > > > How can I reproduce the overread? > > By decoding file, looking at ffmpeg output. I did that and I don't see an overread: What do I miss? Carl Eugen _

Re: [FFmpeg-devel] [PATCH] fate: remove "-v 0" from ffprobe tests.

2016-05-05 Thread Michael Niedermayer
On Thu, May 05, 2016 at 11:59:19AM +0200, Nicolas George wrote: > The FATE scripts separate stdout from stderr. > Having a first idea about what went wrong in the log file > is more convenient. > > Signed-off-by: Nicolas George > --- > tests/fate-run.sh | 16 > 1 file changed, 8

[FFmpeg-devel] [PATCH] fate: remove "-v 0" from ffprobe tests.

2016-05-05 Thread Nicolas George
The FATE scripts separate stdout from stderr. Having a first idea about what went wrong in the log file is more convenient. Signed-off-by: Nicolas George --- tests/fate-run.sh | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/tests/fate-run.sh b/tests/fate-run.

Re: [FFmpeg-devel] [PATCH 3/3] fate: add test for concat with and without automatic filtering.

2016-05-05 Thread Nicolas George
Le decadi 10 floréal, an CCXXIV, James Almer a écrit : > I think this needs to be $(TARGET_SAMPLES). Fixed. > You should be able to simplify this using FATE_FFPROBE and > FATE_SAMPLES_FFPROBE. > It would also remove the need to specify the ffprobe$(PROGSSUF)$(EXESUF) > dependency > for all thes

Re: [FFmpeg-devel] [PATCH 02/12] lavf: update auto-bsf to new BSF API

2016-05-05 Thread Nicolas George
Le decadi 10 floréal, an CCXXIV, Rodger Combs a écrit : > There's definitely going to be some trickiness around this if we end up > having 1:N filters, or filters that actually requiring flushing. > Otherwise, though, this should work fine, and we can address multiple-bsf > or more-advanced BSF cas

[FFmpeg-devel] [PATCH 0/2] Fix VC-2 encoder v2

2016-05-05 Thread Christophe Gisquet
Improved text in comments and commit message. Regenerated checksums after fix for bitexactness in auxiliary data due to version string. Christophe Gisquet (2): vc2enc: prevent random data vc2: fate tests libavcodec/vc2enc.c | 5 + tests/fate/vcodec.mak

[FFmpeg-devel] [PATCH 2/2] vc2: fate tests

2016-05-05 Thread Christophe Gisquet
--- tests/fate/vcodec.mak | 17 - tests/ref/vsynth/vsynth1-vc2-420p | 4 tests/ref/vsynth/vsynth1-vc2-420p10 | 4 tests/ref/vsynth/vsynth1-vc2-420p12 | 4 tests/ref/vsynth/vsynth1-vc2-422p | 4 tests/ref/vsynth/vsynth1-v

[FFmpeg-devel] [PATCH 1/2] vc2enc: prevent random data

2016-05-05 Thread Christophe Gisquet
The slice prefix is 0 in the reference encoder and the decoder ignores it. Writing 0 there seems like the best temporary solution. The padding could have contained uninitialized data, but reference VC2 encoders put 0xFF there, hence the memset value. Overall this allows producing bistreams with n

Re: [FFmpeg-devel] [PATCH 2/2] avformat/tee: Use ref instead copy in write_packet

2016-05-05 Thread Nicolas George
Le quartidi 14 floréal, an CCXXIV, sebechlebsky...@gmail.com a écrit : > From: Jan Sebechlebsky > > Replace av_copy_packet and deprecated av_dup_packet by > creating reference using av_packet_ref. > > Signed-off-by: Jan Sebechlebsky > --- > This should be effectively the same as calling av_pac

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat/tee: Fix TeeSlave.bsfs pointer array size

2016-05-05 Thread Nicolas George
Le quintidi 15 floréal, an CCXXIV, sebechlebsky...@gmail.com a écrit : > From: Jan Sebechlebsky > > TeeSlave.bsfs is array of pointers to AVBitStreamFilterContext, > so element size should be really size of a pointer, not size > of TeeSlave structure. > > Signed-off-by: Jan Sebechlebsky Ok for

Re: [FFmpeg-devel] [FFmpeg-cvslog] Merge commit '2758cdedfb7ac61f8b5e4861f99218b6fd43491d'

2016-05-05 Thread wm4
On Thu, 5 May 2016 02:26:09 +0200 Michael Niedermayer wrote: > > This is quite a hack, and we should not add new APIs to support it, in > > my opinion. I think you will have a hard time finding support in an open > > source project for adding or re-adding APIs to make internal forks easier > > to

[FFmpeg-devel] [PATCH] avfilter/graphparser: remove '\n' from parse_filter

2016-05-05 Thread Muhammad Faiz
this allow a filter to be written like this: aformat = sample_fmts = fltp|flt: sample_rates = 44100|44800 Signed-off-by: Muhammad Faiz --- libavfilter/graphparser.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/graphparser.c b/libavfilter/graphparse