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

rsx: Reimplement projected texturing to play nice with the subimage views #14149

Merged
merged 6 commits into from
Jul 9, 2023

Conversation

kd-11
Copy link
Contributor

@kd-11 kd-11 commented Jul 8, 2023

#14142

Testing needed:

  • Resident Evil: Code Veronica X
  • TLOU
  • Uncharted 2 & 3

@kd-11 kd-11 changed the title rsx: Reimplement projected texturing rsx: Reimplement projected texturing to play nice with the subimage views Jul 8, 2023
@Linear524
Copy link

Hi KD-11 !
I tried this PR and unfortunately RPCS3 crashes when trying to load main menu :(
Logs shows this:
E RSX: ERROR: 0:633: 'coord' : undeclared identifier
ERROR: 0:633: 'xyz' : vector swizzle selection out of range
ERROR: 0:633: 'w' : vector swizzle selection out of range
ERROR: 0:633: '[]' : scalar integer expression required
ERROR: 0:633: '_texcoord_xform' : no matching overloaded function found
ERROR: 0:633: 'texture' : no matching overloaded function found
ERROR: 0:633: '' : compilation terminated
ERROR: 7 compilation errors. No code generated.

E RSX:
F {RSX Worker 2} SIG: Thread terminated due to fatal error: Failed to compile fragment shader
(in file C:\Users\ContainerAdministrator\AppData\Local\Temp\cirrus-ci-build\rpcs3\Emu\RSX\VK\VKProgramPipeline.cpp:29[:25], in function compile)
E RSX: ERROR: 0:632: 'coord' : undeclared identifier
ERROR: 0:632: 'xyz' : vector swizzle selection out of range
ERROR: 0:632: 'w' : vector swizzle selection out of range
ERROR: 0:632: '[]' : scalar integer expression required
ERROR: 0:632: '_texcoord_xform' : no matching overloaded function found
ERROR: 0:632: 'texture' : no matching overloaded function found
ERROR: 0:632: '' : compilation terminated
ERROR: 7 compilation errors. No code generated.

E RSX:
F {RSX Worker 10} SIG: Thread terminated due to fatal error: Failed to compile fragment shader
(in file C:\Users\ContainerAdministrator\AppData\Local\Temp\cirrus-ci-build\rpcs3\Emu\RSX\VK\VKProgramPipeline.cpp:29[:25], in function compile)
E RSX: ERROR: 0:634: 'coord' : undeclared identifier
ERROR: 0:634: 'xyz' : vector swizzle selection out of range
ERROR: 0:634: 'w' : vector swizzle selection out of range
ERROR: 0:634: '[]' : scalar integer expression required
ERROR: 0:634: '_texcoord_xform' : no matching overloaded function found
ERROR: 0:634: 'texture' : no matching overloaded function found
ERROR: 0:634: '' : compilation terminated
ERROR: 7 compilation errors. No code generated.

E RSX:
F {RSX Worker 1} SIG: Thread terminated due to fatal error: Failed to compile fragment shader
(in file C:\Users\ContainerAdministrator\AppData\Local\Temp\cirrus-ci-build\rpcs3\Emu\RSX\VK\VKProgramPipeline.cpp:29[:25], in function compile)

@Megamouse Megamouse added the RSX label Jul 8, 2023
@kd-11
Copy link
Contributor Author

kd-11 commented Jul 8, 2023

@Linear524 Retest

@solarmystic
Copy link

solarmystic commented Jul 8, 2023

Still getting the same fatal error and rpcs3 crash after booting up TLOU using the latest PR build unfortunately.

F {RSX [0x008590c]} SIG: Thread terminated due to fatal error: Failed to compile fragment shader (in file C:\Users\ContainerAdministrator\AppData\Local\Temp\cirrus-ci-build\rpcs3\Emu\RSX\VK\VKProgramPipeline.cpp:29[:25], in function compile)

Logfile:-
RPCS3.log

@kd-11
Copy link
Contributor Author

kd-11 commented Jul 8, 2023

@davewuk Thanks. The PR breaks some things from reports on discord, I'll update when the found issues are fixed.

@readywer
Copy link

readywer commented Jul 8, 2023

GT5 with this pr.
Képernyőfelvétel (360)
Képernyőfelvétel (361)

@solarmystic
Copy link

TLOU dynamic shadow fixed as of the latest pr commit on my end.

image

@kd-11 kd-11 marked this pull request as ready for review July 9, 2023 10:14
@kd-11
Copy link
Contributor Author

kd-11 commented Jul 9, 2023

Try with an older build from like a month ago and see if the problem is fixed there with the same settings.

@kd-11
Copy link
Contributor Author

kd-11 commented Jul 9, 2023

@davewuk Is there anything that worked better (with regards to shadows) on the older build? Verify without using any upscaling. If this PR produces correct results, we can proceed with the merge.

@kd-11
Copy link
Contributor Author

kd-11 commented Jul 9, 2023

Thanks for confirming. I'll proceed and merge this work then.

@kd-11 kd-11 merged commit a98b4ad into RPCS3:master Jul 9, 2023
5 checks passed
@Linear524
Copy link

Uncharted series (2 and 3) was always partially broken. As well as TLoU post processing effects and SPU MLAA.
But anyway - I suggest to never use scaling and use FSR upscaling instead... With native 720p. This is the only way to see all intended graphical effects without those "skipping patches"... Just try to set WCB+RCB+WDB+RDB without MLAA skipping patch and watch the result...

@Linear524
Copy link

Without WCB+RCB+WDB+RDB and without working MLAA you will miss:

  • Color Balance and Dynamic Exposure effects
  • DoF effects
  • Bloom control
  • Morphological Antialiasing
  • Proper explosion FXs
  • Vignette effects
  • Film Grain effect
  • Proper reflections on cubemaps
  • etc

@Linear524
Copy link

@davewuk

  • I don't have to, because I saw this for years and you can try it by yourself

@kd-11
Copy link
Contributor Author

kd-11 commented Jul 12, 2023

Don't add discussion comments to closed PRs.

@RPCS3 RPCS3 locked as spam and limited conversation to collaborators Jul 12, 2023
@kd-11 kd-11 deleted the subimage-views-fix1 branch August 9, 2024 20:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants