Add Spark splat dynamic lighting and line-based point helper#39
Open
querielo wants to merge 1 commit into
Open
Add Spark splat dynamic lighting and line-based point helper#39querielo wants to merge 1 commit into
querielo wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Introduces a Spark dynamic lighting bridge that converts Three.js scene lights into Spark SDF-based lighting edits, wires it into the main renderer lifecycle, and updates helpers/loader behavior to support the new lighting workflow.
Changes:
- Added
SparkSceneLightingBridgeto scan scene lights and drive SparkSplatEdit/SplatEditSdfinstances. - Integrated lighting bridge creation/update/disposal into
EffectRenderer. - Updated
VolumePointLightHelperimplementation and made loaded splats editable.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| client/packages/editor-oss/src/render/SparkLightingBridge.ts | New bridge that maps Three lights to Spark lighting SDF edits and manages lifecycle. |
| client/packages/editor-oss/src/render/EffectRenderer.js | Creates/updates/disposes the lighting bridge during renderer init/render/cleanup. |
| client/packages/editor-oss/src/helper/light/VolumePointLightHelper.js | Replaces PointLightHelper inheritance with a custom LineSegments-based helper. |
| client/packages/editor-oss/src/assets/js/loaders/SparkGaussianSplatLoader.ts | Loads splats as editable to support downstream edits/lighting workflows. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add dynamic Spark lighting support for Gaussian splats and switch point light helper rendering to line geometry (no wireframe mesh).
Screen.Recording.2026-06-05.at.00.28.08.mp4
Why
Lighting Behavior
Performance
scanIntervalMs).Validation
Risk / Notes
scene.userData.rendering.splat.lighting.scanIntervalMs = 0for immediate updates.