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: Regression fixes and GIF Fix for Puzzle Quest/Motocross Mania 3 #9773

Merged
merged 3 commits into from
Aug 11, 2023

Conversation

refractionpcsx2
Copy link
Member

@refractionpcsx2 refractionpcsx2 commented Aug 11, 2023

Description of Changes

Stops it halfing the height on a shuffle by accident.
Fixes problem where it could try and read a null RT
Adjusted how GIF Intermittent packets split.

Rationale behind Changes

Was too horny with the halving, so in some cases the actual size got halved, not just the doubled shuffle size.
Regression from #9745
null RT was possible if there was no RT but it was still reading a texture, possibly in an Alpha tested Z write situation.
Adjusted GIF intermittent packet timing to be a bit kinder to games without sacrificing speed.

Suggested Testing Steps

Test Onimusha 3, Wallace & Gromit Project Zoo, Puzzle Quest, Motocross Mania 3. Make sure nothing is broken.

Fixes #6733 Puzzle Quest flickering graphics
Fixes Motocross Mania 3 flickering textures
Fixes Onimusha 3 bad height on shuffle, plus some invisible bad height changes in Haunting Ground and Sniper Elite.

Onimusha 3:
Master:
image
PR:
image

Motocross Mania 3:

Master:
image
PR:
image

Puzzle Quest:
Master:
image
PR:
image

@refractionpcsx2 refractionpcsx2 changed the title GS/HW: Stop errantly halfing normal height GS/HW: Regression fixes and GIF Fix for Puzzle Quest/Motocross Mania 3 Aug 11, 2023
@refractionpcsx2 refractionpcsx2 merged commit 2a0f001 into master Aug 11, 2023
22 checks passed
@refractionpcsx2 refractionpcsx2 deleted the gs_shufflehalf branch August 11, 2023 23:28
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]: Puzzle Quest - Challenge of the Warlords
3 participants