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

Just another formats/colors conversion issue #49

Closed
chainikdn opened this issue May 25, 2021 · 7 comments
Closed

Just another formats/colors conversion issue #49

chainikdn opened this issue May 25, 2021 · 7 comments
Labels
bug Something isn't working

Comments

@chainikdn
Copy link
Contributor

AVSF ver. 1.0.0
sample 10-bit video: https://mega.nz/file/wJMDjYwI#ZIdVbSxUh3oiPNuiypE__KyT08r6X2OR5TFl1rQ6wkE
no SVP
minimal avs script:

AvsFilterSource()
# ConvertBits(8)

case 1

  1. play video, note selected formats: "in P016 out P016"
  2. uncomment "ConvertBits", reload script
  3. image corrupted

case 2

  1. play video (with ConvertBits already uncommented), everything seems OK
  2. BUT - "input YV24 output YV24" - is it an expected behavior?
@chainikdn chainikdn added the bug Something isn't working label May 25, 2021
@CrendKing
Copy link
Owner

I can repro case 1. Interestingly, I can only repro with this specific video file. My other P010 and YV24 files are fine. I'll look into this.

For case 2, I get NV12 (correctly) instead of YV24, even though YV24 is selected in settings.

@CrendKing
Copy link
Owner

@chainikdn
Copy link
Contributor Author

chainikdn commented Jun 1, 2021

fixed image corruption, yes

for the case 2 - ok, now it tells "Connected with types: in NV12 out NV12"
again, is this expected? shouldn't it feed the script with 10-bit source in any case?
I mean, it must be "P010 -> NV12", isn't it?

BTW, the build artifacts look strange.

@CrendKing
Copy link
Owner

I mean, it must be "P010 -> NV12", isn't it?

Fixed typo in 06d06c3

BTW, the build artifacts look strange.

Care to elaborate?

@chainikdn
Copy link
Contributor Author

"Release x64" -- 21.9 MB ?

@CrendKing
Copy link
Owner

2 pdb files are included, each with ~10MB, 21.9MB seems fine. The downloaded zip file is only 5MB though.

@chainikdn
Copy link
Contributor Author

ok, this one looks fixed

FYI the user reports that another issue (which I can't reproduce on my laptop) was also fixed:
AvisynthFilterIssue02

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants