Skip to content

Conversation

@sunag
Copy link
Collaborator

@sunag sunag commented Sep 15, 2025

@sunag sunag added this to the r181 milestone Sep 15, 2025
@github-actions
Copy link

github-actions bot commented Sep 15, 2025

📦 Bundle size

Full ESM build, minified and gzipped.

Before After Diff
WebGL 338.24
79.14
338.24
79.14
+0 B
+0 B
WebGPU 584.41
161.49
584.78
161.56
+367 B
+70 B
WebGPU Nodes 583.02
161.25
583.39
161.32
+367 B
+71 B

🌳 Bundle size after tree-shaking

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

Before After Diff
WebGL 469.7
113.74
469.7
113.74
+0 B
+0 B
WebGPU 653.77
176.99
654.13
177.06
+369 B
+72 B
WebGPU Nodes 607.88
166.16
608.25
166.24
+369 B
+75 B

@sunag sunag marked this pull request as ready for review September 15, 2025 19:01
@sunag
Copy link
Collaborator Author

sunag commented Sep 15, 2025

WebGLBackend is not fully compatible with inspector and is crashing the examples in E2E tests, I will disable it for now.

@sunag sunag merged commit 7ac88e4 into mrdoob:dev Sep 15, 2025
9 checks passed
@sunag sunag deleted the dev-inspector-1 branch September 15, 2025 19:16
@Mugen87
Copy link
Collaborator

Mugen87 commented Sep 15, 2025

It seems the FPS does not update after this PR.

@sunag
Copy link
Collaborator Author

sunag commented Sep 15, 2025

It seems the FPS does not update after this PR.

I disabled it for WebGLBackend, I'm still going to create a version that doesn't crash on E2E.

@Mugen87
Copy link
Collaborator

Mugen87 commented Sep 15, 2025

Understood! When you think the inspector is robust enough, I can help with adding it to the WebGPU examples so they all benefit from the new beautiful UI.

@sunag
Copy link
Collaborator Author

sunag commented Sep 18, 2025

Understood! When you think the inspector is robust enough, I can help with adding it to the WebGPU examples so they all benefit from the new beautiful UI.

I think we can start migrating, when I enable the WebGLBackend version there shouldn't be any changes in the examples, this could even help with testing.

I didn't add the button in the GUI yet, but I think some examples that use it can use booleans instead like webgpu_layers (Edit: Hmm, now that I see there's an enable all, I'll added the button if you find it necessary)

If you find any bugs, you can tag me and I will do my best to fix them as quickly as possible.

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