Skip to content

Ray: Handle empty spheres in intersectsSphere().#31200

Merged
Mugen87 merged 1 commit intomrdoob:devfrom
Mugen87:dev1
May 30, 2025
Merged

Ray: Handle empty spheres in intersectsSphere().#31200
Mugen87 merged 1 commit intomrdoob:devfrom
Mugen87:dev1

Conversation

@Mugen87
Copy link
Collaborator

@Mugen87 Mugen87 commented May 30, 2025

Fixed #31187.

Description

The PR makes sure empty spheres are properly handled in Ray.intersectsSphere().

@Mugen87 Mugen87 added this to the r177 milestone May 30, 2025
@github-actions
Copy link

📦 Bundle size

Full ESM build, minified and gzipped.

Before After Diff
WebGL 337.26
78.64
337.26
78.64
+0 B
+0 B
WebGPU 553.64
153.44
553.64
153.44
+0 B
+0 B
WebGPU Nodes 552.99
153.28
552.99
153.28
+0 B
+0 B

🌳 Bundle size after tree-shaking

Minimal build including a renderer, camera, empty scene, and dependencies.

Before After Diff
WebGL 468.44
113.28
468.45
113.29
+14 B
+6 B
WebGPU 628.6
170.13
628.61
170.14
+14 B
+9 B
WebGPU Nodes 583.45
159.47
583.46
159.48
+14 B
+8 B

@Mugen87 Mugen87 merged commit ea4ecb7 into mrdoob:dev May 30, 2025
12 checks passed
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.

Ray.intersectsSphere(sphere) is incorrect for empty Spheres

1 participant