vapoursynth-preview icon indicating copy to clipboard operation
vapoursynth-preview copied to clipboard

Published 20 hours ago • verified publisher icon Endilll

Compat formats will be killed off

Open myrsloik opened this issue 4 years ago • 10 comments

The compat formats will be removed in future builds of VS meaning that you'll have to use RGB24 as output and pack the image yourself. You can get the beta builds with this behavior here: https://forum.doom9.org/showthread.php?t=183070

myrsloik avatar Aug 01 '21 12:08 myrsloik

Thank you for letting me know. Can Python API in those beta builds be considered stable? Can you give any estimates if it's not? Again, any estimates for proper release with APIv4? I'm interested not just in compat formats removed, but in audio support as well.

Endilll avatar Aug 01 '21 15:08 Endilll

Can Python API in those beta builds be considered stable?

Yes, there are no further changes planned. Overall there are very few changes on the Python side apart from the minimum necessary due to core functionality differences.

Again, any estimates for proper release with APIv4?

A few weeks to a month I guess. Very few bugs reported so far and scripts/applications seem to get updated reasonably fast.

myrsloik avatar Aug 01 '21 15:08 myrsloik

Good news, thank you. I'll make sure to update vapoursynth-preview, and revisit audio support proof-of-concept I made against some early builds year and a half ago.

Endilll avatar Aug 01 '21 15:08 Endilll

Do you have an update on this? The breaking changes to vapoursynth have been out for a few weeks and are now pushed to the Arch repos. vspreview is completely broken with the new version.

shssoichiro avatar Oct 02 '21 10:10 shssoichiro

@shssoichiro As temporary workaround you may use my patch (with libp2p plug-in).

DJATOM avatar Oct 02 '21 10:10 DJATOM

I'm still receiving the same error with that patch. To be specific, the error I'm getting is module 'vapoursynth' has no attribute 'AlphaOutputTuple' in core/types.py line 697.

shssoichiro avatar Oct 02 '21 10:10 shssoichiro

I'm trying to do some debugging locally. Looks like AlphaOutputTuple was renamed to VideoOutputTuple, but after fixing that, I hit another error. I'll keep going, hopefully eventually I'll have something working.

Is there a full migration guide from API3 to API4? I searched for one but could not find anything of the sort.

shssoichiro avatar Oct 02 '21 11:10 shssoichiro

@DJATOM It looks like these are the remaining fixes to get it to work: https://github.com/shssoichiro/vapoursynth-preview/commit/5e8eb1ff1f930068f49a98d48a6180319a331a55

shssoichiro avatar Oct 02 '21 11:10 shssoichiro

Ah, indeed. I forgot to update for latest changes. Will make new patch soon.

DJATOM avatar Oct 02 '21 11:10 DJATOM

Here comes updated patch.

DJATOM avatar Oct 02 '21 11:10 DJATOM