Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tile error with Rife 4.26 on non standard dimension videos #112

Open
zelenooki87 opened this issue Oct 12, 2024 · 2 comments
Open

Tile error with Rife 4.26 on non standard dimension videos #112

zelenooki87 opened this issue Oct 12, 2024 · 2 comments

Comments

@zelenooki87
Copy link

zelenooki87 commented Oct 12, 2024

Failed to evaluate the script:
Python exception: vsmlrt.RIFEMerge: tile size must be divisible by 64 (192, 160)

Traceback (most recent call last):
File "src\\cython\\vapoursynth.pyx", line 3365, in vapoursynth._vpy_evaluate
File "src\\cython\\vapoursynth.pyx", line 3366, in vapoursynth._vpy_evaluate
File "C:\Users\admin\AppData\Local\Temp\tempPreviewVapoursynthFile11_44_02_151.vpy", line 63, in 
clip = vsmlrt.RIFE(clip, multi=Fraction(2), model=426, backend=Backend.ORT_DML(fp16=False,device_id=0)) # new fps: 28.96
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:/Program Files/Hybrid/64bit/vs-mlrt/vsmlrt.py", line 1215, in RIFE
output0 = RIFEMerge(
^^^^^^^^^^
File "C:/Program Files/Hybrid/64bit/vs-mlrt/vsmlrt.py", line 1049, in RIFEMerge
raise ValueError(
ValueError: vsmlrt.RIFEMerge: tile size must be divisible by 64 (192, 160)

Latest Hybrid dev and vsmlrt addon. doe snot matter if I use Directml or TensorRT
Input file dimensions are realy small
Width : 176 pixels
Height : 144 pixels

rife 4.25 and lower doesnt have this problem with such small input dimensions.

@WolframRhodium
Copy link
Contributor

The exception is raised because rife 4.26 upgrades the architecture. (hzwer/Practical-RIFE#109 (comment))

You should not expect the limitation on tile size to be stable for the v1 implementation. The v2 implementation handles this automatically.

@flowreen
Copy link

@zelenooki87
In order to temporarily fix the following error:
"Failed to evaluate the script:
Python exception: vsmlrt.RIFEMerge: tile size must be divisible by 64"
on the "v1" variants until the SVP developer updates their code, you can open
C:\Program Files (x86)\SVP 4\script\generate.js
and replace this:

    var pw = Math.floor((media.dst_w-1)/32+1)*32 - media.dst_w;
    var ph = Math.floor((media.dst_h-1)/32+1)*32 - media.dst_h;

with this:

    var pw = Math.floor((media.dst_w-1)/64+1)*64 - media.dst_w;
    var ph = Math.floor((media.dst_h-1)/64+1)*64 - media.dst_h;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants