search

pinterf / mvtools

mvtools plugin for avisynth
155 stars 17 forks source link

Potential bug in CrossSearch #61

Closed adworacz closed 6 months ago

adworacz commented 6 months ago

https://github.com/pinterf/mvtools/blob/d8bdff7e02c15a28dcc6e9ef2ebeaa9d16cc1f56/Sources/PlaneOfBlocks.cpp#L1583

I believe this should be mvy - j instead of mvy + j. This would align with the horizontal check above, and I believe matches this case from x264: https://github.com/MasterNobody/x264/blob/master/encoder/me.c#L171-L174

adworacz commented 6 months ago

Oh wow - apparently I found this same bug previously. I thought I had a sense of dejavu: https://github.com/pinterf/mvtools/issues/56

Resolving.