MVTools without idx

[Reel.Deel]

1) avisynth.org.ru appears to be dead? It loads "Default Web Site Page".

Fizick's plugins are also hosted here: http://www.avisynth.nl/users/fizick/

Edit: the DL link still points to avisynth.org.ru, here's the correct DL link for MVTools: www.avisynth.nl/users/fizick/mvtools/mvtools-v2.5.11.22.zip

[jackoneill]

Fizick's plugins are also hosted here: http://www.avisynth.nl/users/fizick/

Edit: the DL link still points to avisynth.org.ru, here's the correct DL link for MVTools: www.avisynth.nl/users/fizick/mvtools/mvtools-v2.5.11.22.zip

Ah, thanks.

[Fizick]

jackoneill, sorry I can not give you answer about MVDepan right now. It is old code. And it is quite possible that the code is correct (by result) :)


And yes, I now see some problem with my main site. May be cheap hosting limit is exceeded.

[jackoneill]

jackoneill, sorry I can not give you answer about MVDepan right now. It is old code. And it is quite possible that the code is correct (by result) :)


Oh well. I noticed another thing though: the "zerow" parameter appears to be ignored. The variable "zeroWeight" at line 531 shadows the class member.

[jackoneill]

Another question about MVDepan: if the "range" parameter is greater than zero, it will calculate data (motionx, motiony, motionrot, motionzoom) for several frames in a single GetFrame call. However, it uses just one frame from the mask clip in the calculations for all those frames. Is this a bug?

[jackoneill]

While integrating the recent changes into the VapourSynth plugin, I found several bugs.

MVBlockFPS.cpp
--------------

- At lines 861 and 862 the pointers pDst[1], pDst[2], pSrc[1], pSrc[2], pRef[1], and pRef[2] should each have nWidth_B/2 added.

- At lines 865, 866, and 867 the pitches need to be multiplied by nHeight_B, not nHeight-nHeight_B.

- Short2Bytes is not called before returning from MVBlockFps::GetFrame. Frames are returned uninitialised when overlap is used.

- There is a line missing after line 730.

MaskFun.cpp
-----------

- In function MakeSADMaskTime MaskPitch should be used, not nBlkX, when writing to Mask.



Older things I hadn't noticed or forgot about:

MVFlowBlur.cpp
--------------

At line 74 it should be nVPadding instead of nHPadding.

The arrays MaskSmallB, MaskSmallF, MaskFullYB, MaskFullYF, MaskFullUVB, and MaskFullUVF are never used.



And also: shouldn't FlowFps reject vector clips with negative delta? And in Degrain2 and Degrain3 only the first two vector clips are checked for negative delta.