Re: [FFmpeg-devel] [GSOC][PATCH 1/4] factoring obmc out of snow

2016-08-21 Thread Michael Niedermayer
On Sun, Aug 21, 2016 at 09:32:31AM +0300, Станислав Долганов wrote: > Sorry, I forgot to return ok from encode init. Here updated patch. > > 2016-08-20 23:44 GMT+03:00 Michael Niedermayer : > > > On Sat, Aug 20, 2016 at 11:01:29PM +0300, Станислав Долганов wrote: > > >

Re: [FFmpeg-devel] [GSOC][PATCH 1/4] factoring obmc out of snow

2016-08-20 Thread Michael Niedermayer
On Sat, Aug 20, 2016 at 11:01:29PM +0300, Станислав Долганов wrote: > Fixed it. breaks fate (this worked with the previous patch) make: *** [fate-vsynth1-snow] Error 139 make: *** [fate-vsynth1-snow-hpel] Error 139 make: *** [fate-vsynth1-snow-ll] Error 139 make: *** [fate-filter-mcdeint-fast]

Re: [FFmpeg-devel] [GSOC][PATCH 1/4] factoring obmc out of snow

2016-08-20 Thread Michael Niedermayer
On Fri, Aug 19, 2016 at 05:07:01PM +0300, Станислав Долганов wrote: > New fixes heres another crash: http://ffmpeg.org/~michael/snow-chroma-bug-fuzzed2.avi zzuf -c -s116 -r0.001 ./ffmpeg_g -i snow-chroma-bug.avi -f null - #0 ff_emu_edge_vfix15_mmx () at libavcodec/x86/videodsp.asm:340 #1

Re: [FFmpeg-devel] [GSOC][PATCH 1/4] factoring obmc out of snow

2016-08-18 Thread Michael Niedermayer
On Thu, Aug 18, 2016 at 02:48:40PM +0300, Станислав Долганов wrote: > Seem like this fixes it. still crashes zzuf -c -s0: -r0.01 ./ffmpeg -i snow-chroma-bug.avi -f null - ... [avi @ 0x3e20e40] Something went wrong during header parsing, I will ignore it and try to continue anyway. Truncating

Re: [FFmpeg-devel] [GSOC][PATCH 1/4] factoring obmc out of snow

2016-08-18 Thread Michael Niedermayer
On Wed, Aug 17, 2016 at 02:07:20PM +0300, Станислав Долганов wrote: > Hello, > > I'm sending the patch set with implementation of GSoC project -- FFV1 P > frame support. The current FFV1 uses the same OBMC code as the Snow codec. > Also new median_me_mp function has appeared. > > I'm attaching

Re: [FFmpeg-devel] [GSOC][PATCH 1/4] factoring obmc out of snow

2016-08-17 Thread Michael Niedermayer
On Wed, Aug 17, 2016 at 07:50:22PM +0200, Michael Niedermayer wrote: > On Wed, Aug 17, 2016 at 02:07:20PM +0300, Станислав Долганов wrote: > > Hello, > > > > I'm sending the patch set with implementation of GSoC project -- FFV1 P > > frame support. The current FFV1 uses the same OBMC code as the

Re: [FFmpeg-devel] [GSOC][PATCH 1/4] factoring obmc out of snow

2016-08-17 Thread Michael Niedermayer
On Wed, Aug 17, 2016 at 02:07:20PM +0300, Станислав Долганов wrote: > Hello, > > I'm sending the patch set with implementation of GSoC project -- FFV1 P > frame support. The current FFV1 uses the same OBMC code as the Snow codec. > Also new median_me_mp function has appeared. > > I'm attaching

Re: [FFmpeg-devel] [GSOC][PATCH 1/4] factoring obmc out of snow

2016-08-17 Thread Станислав Долганов
> Thanks for the report! Typically, people will measure a few coding points > per file (instead of one) so you can plot filesize vs. quality (in whatever > metric) and see relative improvement between clips (old vs. new) over > interpolated points. See e.g. a graph like this: > >

Re: [FFmpeg-devel] [GSOC][PATCH 1/4] factoring obmc out of snow

2016-08-17 Thread Ronald S. Bultje
Hi Stanislav, On Wed, Aug 17, 2016 at 7:07 AM, Станислав Долганов < stanislav.dolga...@gmail.com> wrote: > Hello, > > I'm sending the patch set with implementation of GSoC project -- FFV1 P > frame support. The current FFV1 uses the same OBMC code as the Snow codec. > Also new median_me_mp