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

ColorYUV(analyze=true) #256

Closed
TbtBI opened this issue Jan 26, 2022 · 6 comments
Closed

ColorYUV(analyze=true) #256

TbtBI opened this issue Jan 26, 2022 · 6 comments

Comments

@TbtBI
Copy link

TbtBI commented Jan 26, 2022

ColorBars(pixel_type="yv12")
#propDelete("_ColorRange")
ColorYUV(analyze=true)
propshow()

Hi, if there is no frame prop for the color range, coloryuv(analyze=true) is returning a clip with wrong levels and wrong color range property. If there is frame prop, coloryuv(analyze=true) is returning a clip with correct levels and frame prop for the color range. You can test with the above script.

@pinterf
Copy link

pinterf commented Jan 26, 2022

Hi!, good observing.
ColorYUV assumes PC range originally, as Avisynth Wiki says:
"[gamma] will operate on TV range instead of the default PC range when the input clip is known to be in TV range: .."

So when no information is given, the clip is assumed to be a full-range clip - ColorYUV speciality. If the _ColorRange property does not exist, it will be set to ColorYUV's default.

One would argue that when no conversion happens (no gamma is applied on the video or gamma is zero), the _ColorRange would stay undefined?

@TbtBI
Copy link
Author

TbtBI commented Jan 26, 2022

I never seen ColorYUV(analyze=true) to change the output in the period before the frame properties. I would expect the same behavior now too.

@pinterf
Copy link

pinterf commented Jan 27, 2022

True, will be fixed soon

@TbtBI
Copy link
Author

TbtBI commented Jan 27, 2022

Thanks.

@pinterf
Copy link

pinterf commented Jan 30, 2022

Hopefully fixed in 7b30966

@TbtBI
Copy link
Author

TbtBI commented Feb 3, 2022

It's ok now.

@TbtBI TbtBI closed this as completed Feb 3, 2022
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

2 participants