Skip to content

Remove deprecated gsplatCustomizeVS shader chunk#8422

Merged
mvaligursky merged 1 commit into
mainfrom
mv-remove-gsplat-customize
Jan 30, 2026
Merged

Remove deprecated gsplatCustomizeVS shader chunk#8422
mvaligursky merged 1 commit into
mainfrom
mv-remove-gsplat-customize

Conversation

@mvaligursky

@mvaligursky mvaligursky commented Jan 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Removes the deprecated gsplatCustomizeVS shader chunk and its backward compatibility layer
  • Removes old covariance-based API (modifyCenter, modifyCovariance, modifyColor) in favor of the newer scale-based gsplatModifyVS API (modifySplatCenter, modifySplatRotationScale, modifySplatColor)
  • Removes unused covariance helper functions (gsplatExtractSize, gsplatApplyUniformScale, gsplatMakeRound)
  • Updates validation to warn users that gsplatCustomizeVS has been removed

related to #8246

Technical Details

The old customization API worked with covariance matrices (covA, covB), while the new API works directly with rotation quaternions and scale vectors, providing better performance and a simpler interface.

Migration Guide

Old (removed) New
modifyCenter(center) modifySplatCenter(center)
modifyCovariance(orig, mod, covA, covB) modifySplatRotationScale(orig, mod, rotation, scale)
modifyColor(center, color) modifySplatColor(center, color)
gsplatExtractSize(covA, covB) gsplatGetSizeFromScale(scale)
gsplatMakeRound(covA, covB, size) gsplatMakeSpherical(scale, size)
gsplatApplyUniformScale(covA, covB, s) scale *= s

Files Changed

  • Deleted src/scene/shader-lib/glsl/chunks/gsplat/vert/gsplatCustomize.js
  • Deleted src/scene/shader-lib/wgsl/chunks/gsplat/vert/gsplatCustomize.js
  • Updated chunk collections to remove exports
  • Updated gsplatCommon.js to remove include
  • Updated gsplat.js and gsplatCorner.js to remove old API calls
  • Updated gsplatHelpers.js to remove covariance helpers
  • Updated system.js validation to report "removed"
  • Added to removedChunks in chunk-validation.js

@mvaligursky mvaligursky self-assigned this Jan 30, 2026
@mvaligursky mvaligursky added the area: graphics Graphics related issue label Jan 30, 2026
@mvaligursky mvaligursky requested a review from a team January 30, 2026 13:29
@mvaligursky mvaligursky merged commit d5963f2 into main Jan 30, 2026
7 checks passed
@mvaligursky mvaligursky deleted the mv-remove-gsplat-customize branch January 30, 2026 14:40
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.

2 participants