Skip to content

fix: match raster Gaussian radius cap in compute GSplat renderer#8573

Merged
mvaligursky merged 1 commit intomainfrom
mv-compute-radius-cap
Apr 2, 2026
Merged

fix: match raster Gaussian radius cap in compute GSplat renderer#8573
mvaligursky merged 1 commit intomainfrom
mv-compute-radius-cap

Conversation

@mvaligursky
Copy link
Copy Markdown
Contributor

@mvaligursky mvaligursky commented Apr 2, 2026

Align the compute renderer's maximum projected Gaussian extent with the raster renderer's per-eigenaxis cap at vmin pixels.

Changes:

  • The raster path caps each eigenaxis half-extent at vmin = min(1024, min(width, height)) pixels. The compute path was using 2 * vmin, allowing large splats to retain their full natural Gaussian falloff while the raster path steepened them. This made large, mostly-transparent splats appear significantly more opaque in the compute renderer — up to ~2x per pixel — with the discrepancy increasing at lower resolutions as more splats exceeded the raster cap.
  • Changed maxRadius in computeSplatCov from 2 * vmin to vmin to match.
  • Aligned the tile bounding-box radius clamp in computeSplatTileEval accordingly.

The raster path caps each eigenaxis half-extent at vmin pixels. The compute
path was using 2*vmin, letting large transparent splats retain their full
natural Gaussian falloff while the raster path steepened them. This caused
up to ~2x per-pixel opacity discrepancy, worsening at lower resolutions.

Align maxRadius and tile bounding-box clamp to vmin to match.

Made-with: Cursor
@mvaligursky mvaligursky self-assigned this Apr 2, 2026
@mvaligursky mvaligursky added the area: graphics Graphics related issue label Apr 2, 2026
@mvaligursky mvaligursky merged commit d2e5c9b into main Apr 2, 2026
8 checks passed
@mvaligursky mvaligursky deleted the mv-compute-radius-cap branch April 2, 2026 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: graphics Graphics related issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant