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

[Regression] TLoU and GT6 bugs after #14104 #14142

Closed
Linear524 opened this issue Jul 6, 2023 · 21 comments
Closed

[Regression] TLoU and GT6 bugs after #14104 #14142

Linear524 opened this issue Jul 6, 2023 · 21 comments

Comments

@Linear524
Copy link

Quick summary

TLoU and GT6 have bugs after #14104
TLoU shadowmaps are screwed (projected shadows from dynamic light sources).
GT6 have weird green pixelated garbage at lower right corner.
Screensots attached.

Details

111
112
113
114
115
116

Build with regression

#14104

Attach two log files

RPCS3.log

Attach capture files for visual issues

https://drive.google.com/file/d/1eKeW9HAsyOLNFaEnsmFNYMFd_mlLtG5N/view?usp=sharing

System configuration

SYS: RPCS3 v0.0.28-15306-d41fe80b Alpha | master
SYS: Intel(R) Xeon(R) CPU E5-2690 v4 @ 2.60GHz | 14 Threads | 31.83 GiB RAM | TSC: 2.594GHz | AVX+ | FMA3 | TSX disabled by default
SYS: Operating system: Windows, Major: 10, Minor: 0, Build: 18362, Service Pack: none, Compatibility mode: 0
SYS: Current Time: 2023-07- 5T23:24:13
RSX: Found vulkan-compatible GPU: 'NVIDIA GeForce GTX 1080' running on driver 531.61.0.0
SYS: Firmware version: 4.87

Other details

I use WCB + RCB + WDB + RDB and native 720p mode.
For TLoU I use only RSX lighting patch and no other patches.

@AniLeo
Copy link
Member

AniLeo commented Jul 6, 2023

Probably fixed by 14140, try again

@solarmystic
Copy link

solarmystic commented Jul 6, 2023

Unable to reproduce this in TLOU using the latest master (v0.0.28-15311) which has #14140 already merged, it is quite possible that this issue has already been resolved in that PR for TLOU.
image

@Linear524
Copy link
Author

2364
2365
2366
Sorry, but it still broken...
14140 have not fixed this...
@solarmystic
On your screen I see no dynamic shadows at all, that's why you don't see that issue...
Maybe you are using some MLAA skipping pathes or resolution scaling ?
Also I'm using all accuracy settings, like RCB+RDB+WCB+WDB to reproduce all effects...

@Megamouse
Copy link
Contributor

Can you please upload a new log with logging enabled?
It seems you had set "Disable all logging" in your global config.

@Linear524
Copy link
Author

Sure, here it is:
RPCS3.log

@Linear524
Copy link
Author

Also:
here is the screenshots from version 15277, which have all projected shadows propertly rendered
BCES01585_screenshot_2023_07_07_17_54_05
BCES01585_screenshot_2023_07_07_17_54_09
BCES01585_screenshot_2023_07_07_17_56_07
BCES01585_screenshot_2023_07_07_17_57_04
BCES01585_screenshot_2023_07_07_17_57_28

Please look how Sarah casts her shadows and how dynamic light sources are working... They are broken in latest revisions.

@Linear524
Copy link
Author

And here is RSX capture from v.15277 with working shadows and lighting:
https://drive.google.com/file/d/1_HRCP0ht3ZmSeHMOc9tUkads8uOWTNFV/view?usp=sharing

@kd-11
Copy link
Contributor

kd-11 commented Jul 7, 2023

@Linear524 Let's start with the easy one - explain how to reproduce the green blotch on GT6. I'm not able to reproduce that one.

@Linear524
Copy link
Author

I'm just using my regular max-accuracy settings:
WCB+RCB+WDB+RDB, also I using Antialiasing on Auto, Shader Quality always on Ultra on every game, Zcull Accuracy on Precise, and Native 720p resolution without any tweaks...
All latest builds shows this green pixelated mess on lower right corner for GT6...

@kd-11
Copy link
Contributor

kd-11 commented Jul 7, 2023

Could be a pascal-specific problem. Which track did you test with?

@Linear524
Copy link
Author

Every track, but mostly Nurburgring, second option

@kd-11
Copy link
Contributor

kd-11 commented Jul 7, 2023

Then it's probably pascal (maybe all nvidia?) rounding mode issue in shaders. I'll try and reproduce on the nvidia card later although it is a newer model.

@kd-11 kd-11 self-assigned this Jul 7, 2023
@kd-11
Copy link
Contributor

kd-11 commented Jul 8, 2023

Verified the tlou bug, it's caused by projected coordinate scaling with the new setup. We need some more work around that special case.

@kd-11 kd-11 added the RSX label Jul 8, 2023
@Clarke2131
Copy link

i've noticed something, on build 15277 tlou works fine but when i tested on 15302 and later builds the marks on the screen appears

issue https://youtu.be/jZq87fz0gLc
works fine https://youtu.be/hfzT6jznRrk

my specs are: ryzen 5 5600, 32GB, RX570 (23.5.2)

RPCS3 15277.log
RPCS3 15302.log

1 0 0 28-15277
2 0 0 28-15277
2 0 0 28-15302
1 0 0 28-15302

@kd-11
Copy link
Contributor

kd-11 commented Jul 8, 2023

TLOU shadow issue should be fixed by #14149

@kd-11
Copy link
Contributor

kd-11 commented Jul 8, 2023

@Clarke2131 There have been many fixes since 15302 specifically to fix those artifacts. The issue is only relevant if it still happens after 15311

@Clarke2131
Copy link

fair enough my bad 😅, here is the 15318

RPCS3.log

FPS_ 29 99 _ Vulkan _ 0 0 28-15318 _ The Last of Us™  BCUS98174  07_07_2023 10_39_13 PM

FPS_ 29 92 _ Vulkan _ 0 0 28-15318 _ The Last of Us™  BCUS98174  07_07_2023 10_40_34 PM

@readywer
Copy link

readywer commented Jul 8, 2023

The same pr caused a regression on GT5 too. But it only happens during b-spec default view and in the race menu where you see the track form above.
Képernyőfelvétel (342)
Képernyőfelvétel (341)

@kd-11
Copy link
Contributor

kd-11 commented Jul 9, 2023

@Linear524 The PR got improvements and was merged. Please retest.

@Linear524
Copy link
Author

Thanks KD-11 ! :)
TLoU shadows fixed )
Tonight I will retest GT6 and close this issue :)

@Linear524
Copy link
Author

Hi KD-11 ! )
GT6 still have those green pixelated garbage on lower left part of screen, but if I disable WCB+RCB+WDB+RDB and enable Force CPU Blit instead = those green garbage is gone ! A lot of PostProcessing FX is gone too without WCB, but hey - it is still playable :>
So I'm sure it is related to overall broken WCB function and I'm also sure it will be gone once you finish #10208...
I'm closing this current issue.

P.S. - is it possible for you to complete #10208 this year ?)) I know it is not so simple, but a lot of titles will benefit from this fix :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants