+- Selur's Little Message Board (https://forum.selur.net)
+-- Forum: Hybrid - Support (https://forum.selur.net/forum-1.html)
+--- Forum: Problems & Questions (https://forum.selur.net/forum-3.html)
+--- Thread: BasicVSR++ (/thread-3188.html)
Pages:
1
2
RE: BasicVSR++ - andry62 - 19.04.2023
Dear Selur.
I have three GPUs - two TITAN RTX and one 4090RTX.
With version Hybrid_2023.03.17.1 and VapoursynthR61_torch_2023.03.12
BasicVSR++ runs in three threads. With version Hybrid_dev_2023.04.18-205712
no way.
RE: BasicVSR++ - Selur - 19.04.2023
Still even with 24GB RAM on the 4090 RTX, when filtering 4k content with BasicVSR++ you will run out of vram unless you use tiling.
Does the Vapoursynth Preview show an error when you open it?
If it just closes, save the script to a file and see if you can open it without issues with vsViewer.
=> Simply complaining that is does not work will not allow me to figure out what the issue might be.
Cu Selur
RE: BasicVSR++ - Selur - 19.04.2023
Ah, just noticed current torch is 2023.04.16
-> will send the link
Cu Selur
RE: BasicVSR++ - Dan64 - 19.04.2023
Hello Selur,
your dev version is working perfectly. With this update using BasicVSR++ with the LinearMask (that allows to use weighted merge) transform BasicVSR++ in one of the best VSR filters available, especially with model NTIRE 2021 (2), which is more aggressive (so the mask is necessary) in introducing more details in the video.
Thanks,
Dan
RE: BasicVSR++ - Selur - 19.04.2023
Happy, it's working in the way you wanted it too.
RE: BasicVSR++ - andry62 - 19.04.2023
Dear Selur.
Everything worked with VapoursynthR61_torch_2023.04.16.
Regarding LinearMask, I completely agree with Dan64-
additional filter adjustment appears.
Thank you.
RE: BasicVSR++ - Selur - 30.04.2023
As a "head's up": I'm going to remove the 'LinearMask' and replace it with a general 'Merge'-option.
http://www.vapoursynth.com/doc/functions/video/merge.html
LinearMask was a special case of Merge:
core.std.Merge(clipa=filteredClip, clipb=unfilteredClip, weight=[setWeight])Cu Selur
RE: BasicVSR++ - Dan64 - 01.05.2023
Hello Selur,
thanks for the update. It will be possible to automatically set all the planes to the same value ?
Just for curiosity, did you see any advantage in assigning different values to the planes ?
Dan
RE: BasicVSR++ - Selur - 01.05.2023
Atm. I haven't planned to add additional io elements to set all values at the same time.
The general idea is to allow (not for just BasicVSR++) to allow to just for example use a specific plane, this basically adds to all filters the option to restrict them to specific planes.