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

Metal: Fix subpixel blending; fix inconsistent blend state #97217

Merged
merged 1 commit into from
Sep 20, 2024

Conversation

stuartcarnie
Copy link
Contributor

Closes #97195

Note

Investigating this issue identified an unrelated blending issue in Metal, which is also fixed.

  • 2D: Reset has_blend state, which was incorrectly carried across batches.
  • Metal: Ensure blend state is applied when pipeline states are changed

Verified

This commit was created on github.com and signed with GitHub’s verified signature.
@stuartcarnie stuartcarnie force-pushed the 97185_subpixel_rendering branch from 6b6eb82 to e826ab9 Compare September 20, 2024 05:16
@AThousandShips AThousandShips added this to the 4.4 milestone Sep 20, 2024
@akien-mga akien-mga changed the title [2D,Metal]: Fix subpixel blending; fix inconsistent blend state in Metal [Metal] Fix subpixel blending; fix inconsistent blend state in Metal Sep 20, 2024
@akien-mga akien-mga changed the title [Metal] Fix subpixel blending; fix inconsistent blend state in Metal Metal: Fix subpixel blending; fix inconsistent blend state in Metal Sep 20, 2024
@akien-mga akien-mga changed the title Metal: Fix subpixel blending; fix inconsistent blend state in Metal Metal: Fix subpixel blending; fix inconsistent blend state Sep 20, 2024
@mihe
Copy link
Contributor

mihe commented Sep 20, 2024

I can confirm that this fixes the blending issue for subpixel antialiasing.

@stuartcarnie
Copy link
Contributor Author

@mihe thanks for confirming 🙇🏻

@akien-mga akien-mga merged commit 34dd5e0 into godotengine:master Sep 20, 2024
19 checks passed
@akien-mga
Copy link
Member

Thanks!

@stuartcarnie stuartcarnie deleted the 97185_subpixel_rendering branch September 20, 2024 19:36
@bruvzg
Copy link
Member

bruvzg commented Sep 25, 2024

Is it fixing #97056 as well?

@stuartcarnie
Copy link
Contributor Author

Is it fixing #97056 as well?

Yes, that is the Metal part of the fix in that PR and I observed those symptoms in the issue

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.

Font subpixel antialiasing results in artifacts
5 participants