-
Notifications
You must be signed in to change notification settings - Fork 3k
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
vf_d3d11vpp: always insert filter for non-standard scaling modes #15071
Conversation
Apparently, some processing is still desired by users even at a 1.0 scaling factor. Fixes: mpv-player#14698 (comment)
Download the artifacts for this pull request: |
Thank you for the commit. |
Are you sure they are not doing super resolution by always scaling up and downscaling? There is no trigger, we set the same filter mode and version. Closing this PR as it doesn't work as expected. |
Not sure how nvdia actually handling the native res's VSR.
|
Let me explain my point of view. After reading this blog post about the first version of VSR, i was assuming that upscaling by any factor is done using bi-cubic filtering. After that, other processing takes place. Therefore i thought we can skip the upscaling part, as it seems separate and done just to enlarge the image. Let's compare VSR in chrome and mpv. I'll be using this video at 720p SDR in h264 codec. To see what i'm talking about, there are two archives for comparing mpv and chrome VSR and for scaling factors, with all video screenshots done by ShareX. No shaders were enabled. I'm comparing two screenshots by setting video-zoom to at least 2 in mpv and switch them back and forth, It's just much more convenient to do it locally than in a browser. When upscaling from 720 to 1080 here, for some reason mpv is different, and has more details despite scaling factor being the same. Here output is 720p, So to summarize, while I agree that VSR at scale factor of 2 can be higher quality (because it is able to fit in extra details in higher resolution), most of the upgrade comes from de-artifacting, quality of which seems to be different between chrome and mpv. Thank you for your time. Hopefully, I explained everything I needed. |
Apparently, some processing is still desired by users even at a 1.0 scaling factor.
Fixes: #14698 (comment)