Skip to content

fix: rename log-depth shader define and bump three to 0.183.2#290

Open
JodokGBS wants to merge 3 commits intosparkjsdev:mainfrom
GEOBILITY-SYSTEMS:fix/logarithmic-depth-buffer-three-0.183.2
Open

fix: rename log-depth shader define and bump three to 0.183.2#290
JodokGBS wants to merge 3 commits intosparkjsdev:mainfrom
GEOBILITY-SYSTEMS:fix/logarithmic-depth-buffer-three-0.183.2

Conversation

@JodokGBS
Copy link
Copy Markdown

Summary

  • Rename deprecated Three.js shader define USE_LOGDEPTHBUF to USE_LOGARITHMIC_DEPTH_BUFFER in src/shaders/splatVertex.glsl.
  • Update three from ^0.178.0 to ^0.183.2 in package.json (devDependencies and peerDependencies).
  • Refresh package-lock.json for the dependency bump.

Why

USE_LOGDEPTHBUF was renamed upstream in Three.js. This keeps Spark compatible with newer Three.js versions.

@dmarcos
Copy link
Copy Markdown
Contributor

dmarcos commented Apr 2, 2026

Thanks. Is this applicable to https://github.com/sparkjsdev/spark/tree/v2.0.0-preview branch?

@JodokGBS
Copy link
Copy Markdown
Author

JodokGBS commented Apr 2, 2026

Thanks. Is this applicable to https://github.com/sparkjsdev/spark/tree/v2.0.0-preview branch?

No as far as i can see, this PR is not needed for v2 as it has no #ifdef USE_LOGDEPTHBUF.
It seems in v2 Spark lets Three handle log-depth internally through its shader chunks, so there’s nothing to rename in the Spark code.

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