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

GS/HW: Make Haunting Ground render fix invalidate depth as well #8978

Merged
merged 1 commit into from Jun 15, 2023

Conversation

stenzek
Copy link
Member

@stenzek stenzek commented Jun 14, 2023

Description of Changes

Fixes depth leaking into the image.

Game's an ass. See the comment in the source.

Note to myself: my clear PR currently doesn't negate the need for this fix... need to dig further into why.

Rationale behind Changes

Closes #8976.

Suggested Testing Steps

Test Haunting Ground. The 3 dumps I have seem okay.

@EnekoT2001
Copy link

EnekoT2001 commented Jun 15, 2023

I tested it just now and while the depth leakage is fixed, the haunting ground logo appearing on screen after starting a new game still shows up, not sure if that'd need to be a different PR or not.

Also by checking more scenes out, I found another bug possibly linked to the logo getting burned on screen but not sure, this one doesn't disappear after switching to software renderer and back so maybe is completely unrelated.

In some scenes, (this case being "the great flood") this happens, not sure how to even explain it apart from "the screen is very white", it also occurs on the main build so it wasn't introduced by this PR.
Haunting Ground_SLES-52877_20230615022919.png

It feels like the game logo getting burned on screen after starting a new game imo (also still present)

Haunting Ground_SLES-52877_20230614135335.png

Here are gsdumps of both (the one for the white screen one I mentioned is a multi-frame gs dump, for some reason when making a single frame gsdump on that one it'd give me a corrupted archive)

HG_white_bug.zip

HG_logo_bug.zip

Thank you again for figuring out how to fix the depth leakage issue btw, that was fast, I really should've checked more cutscenes/gameplay before submitting the report, sorry.

@stenzek
Copy link
Member Author

stenzek commented Jun 15, 2023

More pain in the ass stuff. It's nothing too complicated, just abusing GS local memory.

All three dumps are fine now. If you open them, it'll flash the junk on the screen for a couple of frames - that's just due to the forced preload hack. Ingame should be fine.

The proper solution, non-hacky to this requires my clear PR (#8961), as well as proper overlapping colour/depth targets in the TC (which I haven't done yet).

@EnekoT2001
Copy link

Thank you so much! I've checked and all of the bugs I've mentioned are gone now. I also checked more stuff like the game over screen, other cutscenes, and even gameplay from different points and I can't find any other issues I think it should be all fine now! Thanks again!

@stenzek stenzek merged commit 85b41e2 into PCSX2:master Jun 15, 2023
12 checks passed
@stenzek stenzek deleted the haunting-ground branch June 15, 2023 13:06
@mantis14
Copy link

i play xbsx2 on xbox, how do i use them?

@stenzek
Copy link
Member Author

stenzek commented Jun 16, 2023

i play xbsx2 on xbox, how do i use them?

Why are you asking us? If you're not aware, this fork violates our license, my own copyright, and abused several of our developers over the course of months last year, and our attempts to resolve it were met with silence and more crap-flinging.

Please have the respect to not discuss it here, thanks.

@mantis14
Copy link

mantis14 commented Jun 16, 2023 via email

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.

[BUG]: Haunting Ground Texture cache regression
3 participants