-
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: add support for scaling #14698
Conversation
I've refactored options a bit to avoid hardcoded scaling size. @billxc: please take a look if that works for you. |
Download the artifacts for this pull request: |
1f1f4ce
to
ec73f8e
Compare
ec73f8e
to
0f2707f
Compare
looks good for me, thanks for the work!@kasper93 |
Adds `--vf=d3d11vpp=scale` to scale video using d3d11 video processor. Co-authored-by: xc <[email protected]>
Adds `--vf=d3d11vpp=scaling-mode` to control which extensions should be enabled. Fixes: mpv-player#11390 Co-authored-by: xc <[email protected]>
0f2707f
to
371ce82
Compare
For my 4K screen, when using the super resolution feature, to achieve an output resolution of 4K, the scale value should ideally be set to 2.0 for a 1080p video and 3.0 for a 720p video. Could you add a feature to automatically set the scale values? |
This can be done with lua script, although it would be nice to have native support.
Update: Improved the script, now automatically adapts to different hwdec or swdec. |
I couldn't make it work. Isn't below commands in mpv.config enough? vf-add=d3d11vpp |
|
Same as |
I meet with an issue with d3d11vpp filter.
|
I added lines below still no luck. Already tried with different resolutions. Mpv registers d3d11vpp as filter but it doesn't trigger super resolution status in Nvidia Control panel. In case of conflict i tried it with fresh install with no other configuration other than below. vf-add=d3d11vpp |
Frame download doesn't work currently, because they are not allocated by ffmpeg and we don't have proper context to use for that. Will see what can be done about it. |
When using this, I always get two warning messages
I could eliminate the first warning by specifying the mode like The second warning seems to come from Line 190 in e3a9ce2
|
I can't for the life of me get this to actually work. I have this in my mpv.conf:
Also tried without the first line. I don't get any logging even saying it's trying to enable it. Though it shows that it's set in the info section, it doesn't actually do anything and Nvidia reports it is inactive. |
RTX Video Super Resolution will not be active when a game is using NVIDIA Image Scaling (NIS), Dynamic Super Resolution (DSR) or Deep Learning Dynamic Super Resolution (DLDSR) |
Well none of those are the case for me, and other applications have no issues enabling it |
Why? |
Because I saw that elsewhere. But as I said I tried without that line and it still did nothing |
Have you read documentation? Do you have hardware decoding enabled?
|
Ah I was missing |
No. I don't have hardware to test this, so you have to wait for someone else to implement or at least poc how to configure this. |
Okay thank you! |
Well, I found actually it could work with swdec if the source is yuv420p10. |
Just add format=nv12, any pixfmt will work with d3d11vpp |
That's not the issue here. I think autoconvert is broken in some place. yuv420p8 should also work without users manual setting. |
Fixed by #14746
Not really broken, autoconvert will not inject software on cpu format conversion before hwupload. So it currently fails with
and while it could automatically do the conversion, I think it is better to let users explicitly state it, to avoid possibly slow on CPU conversion. Currently if you want to have this filter work with software input/output you can do
Yes, because there is direct upload format compatibility in this case. Also I need to update this filter to output also rgb10a2 when needed. |
autoconvert is also now fixed in #14746 |
Is there a way to disable upscaling part for Nvidia VSR? I'm only interested in artifact reduction, since upscaling is only slightly better than lanczos. |
Apparently, some processing is still desired by users even at a 1.0 scaling factor. Fixes: mpv-player#14698 (comment)
@Damole-wer: Try #15071, should help in this case. |
Apparently, some processing is still desired by users even at a 1.0 scaling factor. Fixes: mpv-player#14698 (comment)
Apparently, some processing is still desired by users even at a 1.0 scaling factor. Fixes: mpv-player#14698 (comment)
No description provided.