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

[WinRenderer] Fix HQ scallers, to avoid unnecessary scaling at Stage1… #8280

Merged
merged 2 commits into from Oct 29, 2015

Conversation

afedchin
Copy link
Member

… (with DXVA processor and color shader). This also fixes HQ scallers for TAB/SBS modes.

@da-anda ping

@afedchin afedchin added Type: Fix non-breaking change which fixes an issue v16 Jarvis Component: Video labels Oct 23, 2015
@afedchin afedchin added this to the Jarvis 16.0-alpha4 milestone Oct 23, 2015
@afedchin
Copy link
Member Author

jenkins build this please

@afedchin
Copy link
Member Author

build error in unrelated.

@da-anda
Copy link
Member

da-anda commented Oct 24, 2015

I tried to reproduce the users issue and it seems partly fixed with this. HQ scalers are behaving correctly now. When switching form HQ scales to bicubic I noticed some shifting of the video in the rect of the right eye in SBS mode - the video itself didn't exceed the mask though - it only seems to be aligned differently within the mask than on HQ scalers.

However, using "Render method: Software", DXVA2 HW accell and "Video scaling method: DXVA" (in VideoOSD) is still having the clipping issue of the right eye.

edit: It doesn't seem to matter which render mode - happens with all.

@afedchin
Copy link
Member Author

there is a difference between DXVA scaling and others and the difference is not 3D related. A little clipping/shifting (about half of pixel) occurs also on normal videos and stereo movies for both eyes. This issue exists in dx9 version also. I don't know exactly why it occurs.

@fritsch
Copy link
Member

fritsch commented Oct 25, 2015

Pixelratio not 1.0? If that's the case - 1920 width will be scaled whenever 1920 * pixelratio < 1919 :-)

@da-anda
Copy link
Member

da-anda commented Oct 25, 2015

here a short screen capture of the issue with DXVA scaling vs HQ-scaler (every other scaler works like lanzcos3) https://youtu.be/HG3gNy2ToaE

@fritsch
Copy link
Member

fritsch commented Oct 25, 2015

I'd prefer a debuglog ...

@afedchin
Copy link
Member Author

@da-anda I have no issue with dxva https://youtu.be/VsIdnVYCRe0 I think @fritsch is right debuglog would be helpfull

@da-anda
Copy link
Member

da-anda commented Oct 26, 2015

@afedchin have you zoomed the video? (can't read russian)
here is a debuglog with component debugging of video component turned on - but I don't think it's of any help as it only shows the shaders loaded when I cycled through them http://xbmclogs.com/podh2jozc

@afedchin
Copy link
Member Author

@da-anda of course, before test I have setted zoom to 1.52 instead of normal 1.0
updated. check this again please

@da-anda
Copy link
Member

da-anda commented Oct 26, 2015

@afedchin just tested - still having that issue. Maybe related to my old AMD GPU?

@afedchin
Copy link
Member Author

I've noticed that you tested with sbs file. I always tested with tab file. I need to do some testing with sbs file.

@afedchin
Copy link
Member Author

I've tested with SBS file and still no issue. Perhaps your driver ignores scissor on some stage. I've added a temporary commit which perhaps solves the issue, but with no warranty. @da-anda test it please. thanks.

@afedchin afedchin added the WIP PR that is still being worked on label Oct 26, 2015
Anton Fedchin added 2 commits October 28, 2015 18:24
… (with DXVA processor and color shader). This also fixes HQ scallers for TAB/SBS modes.
…me drivers which works incorrectly with DXVA processor and deferred context.
@afedchin
Copy link
Member Author

jenkins build this please

@MartijnKaijser
Copy link
Member

ignore droid failures

@afedchin afedchin removed the WIP PR that is still being worked on label Oct 29, 2015
@afedchin
Copy link
Member Author

got positive feedback from user. this can be merged before a4 is released.

MartijnKaijser added a commit that referenced this pull request Oct 29, 2015
[WinRenderer] Fix HQ scallers, to avoid unnecessary scaling at Stage1…
@MartijnKaijser MartijnKaijser merged commit 17877f8 into xbmc:master Oct 29, 2015
@afedchin afedchin deleted the fix_dx_hqscallers branch October 29, 2015 19:04
@afedchin afedchin restored the fix_dx_hqscallers branch December 14, 2015 08:06
@afedchin afedchin deleted the fix_dx_hqscallers branch December 14, 2015 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants