Skip to content

test #31145

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

test #31145

wants to merge 1 commit into from

Conversation

cabanier
Copy link
Contributor

just a test. not for submit

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 550.56
152.65
550.53
152.65
-33 B
+1 B
WebGPU Nodes 549.91
152.5
549.88
152.5
-33 B
+1 B

🌳 Bundle size after tree-shaking

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

Before After Diff
WebGL 468.12
113.14
468.12
113.14
+0 B
+0 B
WebGPU 625.81
169.41
625.78
169.41
-33 B
+2 B
WebGPU Nodes 580.66
158.73
580.63
158.73
-33 B
+2 B

@mrdoob
Copy link
Owner

mrdoob commented May 22, 2025

What's the test for?

@cabanier
Copy link
Contributor Author

PR #31134 flagged that one of the tests had a pixel mismatch.
I wanted to test if my changes had introduced it but it was already there.

@Mugen87
Copy link
Collaborator

Mugen87 commented May 22, 2025

Some examples like webgl_buffergeometry_attributes_integer or webgl_buffergeometry hardly produce a deterministic output so they occasionaly fail. A rerun of the CI task solves the issues. That was also true for #31134 🎉.

@Mugen87 Mugen87 closed this May 22, 2025
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.

3 participants