Skip to content

vello_gpu: Add the probe to the wgpu backend - #2008

Merged
LaurenzV merged 4 commits into
mainfrom
laurenz/wgpu-probe
Oct 21, 2026
Merged

LaurenzV merged 4 commits into
mainfrom
laurenz/wgpu-probe

Conversation

@LaurenzV

Copy link
Copy Markdown
Collaborator

Should be self-explanatory, the same thing we have for WebGL already, but now for the wgpu backend. I've tried this on a couple of devices with wgpu via the browser and it seems to work fine. I have not tried running this natively yet, but we can fix issues as they come up.

@laurenz-canva
laurenz-canva requested a review from grebmeg October 19, 2026 12:30
Add `Renderer::probe`, the wgpu counterpart to `WebGlRenderer::probe`.
It draws the shared probe scene into an offscreen texture with the
renderer's target format, so the probe exercises the pipelines the
application actually renders with, and copies the result into a
readback buffer that is mapped on submit.

`WgpuPendingProbe::try_finish` polls the device without blocking and
compares the result against the bundled reference images. Since the wgpu
renderer receives its depth view per render, the caller states whether
it uses a depth buffer. `Rgba8Unorm` and `Bgra8Unorm` render targets are
supported; other formats return `WgpuProbeError::UnsupportedFormat`.
Comment thread vello_gpu/src/render/wgpu/probe.rs
@LaurenzV
LaurenzV enabled auto-merge October 21, 2026 07:40
@LaurenzV
LaurenzV added this pull request to the merge queue Oct 21, 2026
Merged via the queue into main with commit af7208f Oct 21, 2026
18 checks passed
@LaurenzV
LaurenzV deleted the laurenz/wgpu-probe branch October 21, 2026 08:17
HigherOrderLogic pushed a commit to HigherOrderLogic/vl that referenced this pull request Oct 27, 2026
Should be self-explanatory, the same thing we have for WebGL already,
but now for the wgpu backend. I've tried this on a couple of devices
with wgpu via the browser and it seems to work fine. I have not tried
running this natively yet, but we can fix issues as they come up.
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.

2 participants