Skip to content
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

WebGPURenderer: WebGL State improvements #29132

Merged

Conversation

RenaudRohlinger
Copy link
Collaborator

@RenaudRohlinger RenaudRohlinger commented Aug 14, 2024

Related issue: #27388

Next is about moving VAO into a new WebGLBindingStates class and handle the binding of buffers to a state comparator too.

This contribution is funded by Utsubo

Copy link

github-actions bot commented Aug 14, 2024

📦 Bundle size

Full ESM build, minified and gzipped.

Filesize dev Filesize PR Diff
685.1 kB (169.6 kB) 685.1 kB (169.6 kB) +0 B

🌳 Bundle size after tree-shaking

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

Filesize dev Filesize PR Diff
462 kB (111.4 kB) 462 kB (111.4 kB) +0 B


} else {

console.warn( 'bindBufferRange is only applicable for uniform buffers' );
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be good to always add the component to the message that logs it. E.g.:

console.warn( 'THREE.WebGLBackend: bindBufferRange() is only applicable for uniform buffers.' );

Copy link
Collaborator Author

@RenaudRohlinger RenaudRohlinger Aug 14, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, actually _bindBufferRange was a method I forgot to cleanup that I will work on later on, also I don't think internal warning is useful. Thanks for the heads up!

@RenaudRohlinger RenaudRohlinger marked this pull request as ready for review August 14, 2024 08:13
@RenaudRohlinger RenaudRohlinger added this to the r168 milestone Aug 14, 2024
@RenaudRohlinger RenaudRohlinger merged commit a92e8bc into mrdoob:dev Aug 14, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants