[FFmpeg-devel] [PATCH] FFV1 improvments

[Michael Niedermayer]

On Mon, Oct 11, 2010 at 01:25:35PM +0200, Michael Niedermayer wrote:
> On Mon, Oct 11, 2010 at 09:59:16AM +0000, Loren Merritt wrote:
> > On Sun, 10 Oct 2010, Jason Garrett-Glaser wrote:
> >
> >> On Sun, Oct 10, 2010 at 2:05 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> >>> On Sun, Oct 10, 2010 at 01:19:57PM -0700, Jason Garrett-Glaser wrote:
> >>>> On Sun, Oct 10, 2010 at 12:25 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> >>>>> Hi
> >>>>>
> >>>>> Will apply patch-set below soon.
> >>>>> bikesheds will be politely ignored
> >>>>
> >>>> Looks good;
> >>>
> >>> commited
> >>>
> >>>
> >>>> what are the plans for version 2?
> >>>
> >>> the immedeate plan for ffv1.2 is to add rectangular slice based multithreading
> >>> this should make the code much faster without extra delay
> >>
> >> Another thing you might want to try (since you're changing things...)
> >> is Loren's median5 predictor; testing in FFV2 showed that it beat the
> >> standard median predictor by a few % in terms of order-0 entropy.  I
> >> don't have it, but I assume he can post it here.
> >
> > median5(l, tr, l+t-tl, l*2-ll, t*2-tt)
> > also, median3(l, tr, l+t-tl) is better than median3(l, t, l+t-tl)
> 
> Thanks, ill look into trying/adding them to ffv1

Tried on foreman:
18726558 pred_lor_median3.avi
18655092 pred_old_median.avi
18649580 pred_lor_median5.avi
18609236 pred_T_median5.avi

Patches for the test attached
Iam somewhat uncertain if the added complexity is worth the 0.25% gain
maybe we can find a better predictor where the gain is bigger


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The educated differ from the uneducated as much as the living from the
dead. -- Aristotle 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-loren-median3.patch
Type: text/x-patch
Size: 875 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101024/e6a34682/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-loren-median5.patch
Type: text/x-patch
Size: 2234 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101024/e6a34682/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-L-median5.patch
Type: text/x-patch
Size: 802 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101024/e6a34682/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101024/e6a34682/attachment.pgp>