Skip to content

Desktop: Fix Vello rendering colors with doubly-applied gamma by converting to linear in the shader#2998

Merged
TrueDoctor merged 3 commits intomasterfrom
fix-gamma
Aug 5, 2025
Merged

Desktop: Fix Vello rendering colors with doubly-applied gamma by converting to linear in the shader#2998
TrueDoctor merged 3 commits intomasterfrom
fix-gamma

Conversation

@TrueDoctor
Copy link
Copy Markdown
Member

Vello renders it's values to an RgbaUnorm texture
but if we try to use this in the main rendering
pipeline which renders to an Srgb surface gamma
mapping is applied twice.

Closes #2971

@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) August 5, 2025 09:13 Inactive
Vello renders it's values to an RgbaUnorm texture
but if we try to use this in the main rendering
pipeline which renders to an Srgb surface gamma
mapping is applied twice.
@Keavon Keavon changed the title Fix gamma issue by applying tone mapping in the shader Desktop: Fix Vello rendering colors with doubly-applied gamma by applying tone mapping in the shader Aug 5, 2025
@Keavon Keavon changed the title Desktop: Fix Vello rendering colors with doubly-applied gamma by applying tone mapping in the shader Desktop: Fix Vello rendering colors with doubly-applied gamma by converting to linear in the shader Aug 5, 2025
Comment thread desktop/src/render/composite_shader.wgsl Outdated
@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) August 5, 2025 09:41 Inactive
@TrueDoctor TrueDoctor enabled auto-merge (squash) August 5, 2025 09:48
@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) August 5, 2025 09:52 Inactive
@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) August 5, 2025 10:00 Inactive
@TrueDoctor TrueDoctor merged commit 8b4adf4 into master Aug 5, 2025
4 checks passed
@TrueDoctor TrueDoctor deleted the fix-gamma branch August 5, 2025 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix Vello canvas rendering not being displayed in sRGB

3 participants