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: Extend blend second pass to more blend formulas v2. #11353

Merged
merged 2 commits into from
Jun 14, 2024

Conversation

lightningterror
Copy link
Contributor

Description of Changes

Cd*(1 + Alpha).
Alpha = As, Ad or Af.
For As or Af case when alpha > 128.
For Ad case when there is no RTA correction.

Rationale behind Changes

Accuracy.

Suggested Testing Steps

Test the listed dumps:
There shouldn't be any diffs or minimal since the alpha isn't higher than 128 in these games.
Found a few where it is higher than 128 but with overlap so won't help in those cases.

image

Cd*(1 + Alpha).
Alpha = As, Ad or Af.
For As or Af case when alpha > 128.
For Ad case when there is no RTA correction.
@lightningterror lightningterror force-pushed the gs_extend_blend_pass2_v2 branch from db356e4 to 097473a Compare June 13, 2024 21:18
@AmyRoxwell
Copy link

Midnight Club dub remix and underground don't seem to have any regressions neither new issues with this PR with all renders
Master:
image
PR:
image

Master:
image
Pr:
image

GSDumps:
Midnight Club 3 - DUB Edition Remix_SLUS-21355_20240614110.zip
Midnight Club 3 - DUB Edition Remix_SLUS-21355_20240614105951.zip

Underground:

Master:
image
Pr:
image

Master:
image
Pr:
image

Master:
image
Pr:
image

GSDumps:
Need for Speed - Underground_SLUS-20811_20240614114342.zip
Need for Speed - Underground_SLUS-20811_20240614113209.zip

@lightningterror lightningterror merged commit 7aea867 into master Jun 14, 2024
22 checks passed
@lightningterror lightningterror deleted the gs_extend_blend_pass2_v2 branch June 14, 2024 19:40
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.

4 participants