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: Increase performance #26673

Open
3 of 5 tasks
sunag opened this issue Aug 31, 2023 · 6 comments
Open
3 of 5 tasks

WebGPURenderer: Increase performance #26673

sunag opened this issue Aug 31, 2023 · 6 comments

Comments

@sunag
Copy link
Collaborator

sunag commented Aug 31, 2023

Description

I started a work to increase the performance in WebGPURenderer and it will work for both Backends, it's about a better management of Bindings and Cache.

Solution

  • Check Material Cache Key only after the version difference -- material.needsUpdate=true -- Reuse NodeBuilder #26729
  • Reuse NodeBuilder based from Cache Key, currently the cache uses Material/Geometry as reference. For example, this means that two Materials of the same type with only a different color will be compiled twice instead of reuse the NodeBuilder, this sharing already happens for the GPU Program, but as the code is dynamically generated this optimization process should be API oriented for the Nodes. -- Reuse NodeBuilder #26729
  • Separate bindings groups for FRAME, RENDER, OBJECT groups. Node System already offers this support, we would have to take it to the level of optimization for the Bindings.
  • Node System allows Auto-Instancing with a certain flexibility, this has already been adopted in other engines like Unreal and we may be implementing it soon as well. -- Renderer: Auto-Instancing [WIP] #26640
@sunag sunag added this to the r157 milestone Aug 31, 2023
@danrossi
Copy link

I was about to comment. I just made new builds to figure out the iife bundle issue. And super low resources with video textures.

https://danrossi.github.io/three-webgpu-renderer/tests/webgpu_video_panorama_equirectangular.html

and using video frame callback

https://danrossi.github.io/three-webgpu-renderer/tests/webgpu_video_panorama_equirectangular_30fps.html

@PhilCrowther
Copy link

There is apparently still an issue with WebGPUrenderer and shadows. On a program with a single DirectionalLight that has shadow definitions, and that uses non-emissive materials (e.g. Lambert and Phong), switching to WebGPUrenderer will cause a massive slowdown. Oddly, if I remove the light, WebGPUrender will cause all the materials to become emissive - with the result that shadows are no longer an issue and the program operates at normal speed. (This has nothing to do with the use of NodeMaterials as I first thought since the same thing happens with regular Materials.) If it helps, I also had to add a bias definition to fix the appearance of some materials.

@sunag
Copy link
Collaborator Author

sunag commented Oct 30, 2023

@PhilCrowther Could it be related to this ?

@PhilCrowther
Copy link

@sunag Not sure. I have created a small demo on CodePen.
To show the difference in performance and results, disable the DirectionalLight (line 82: scene.add(DirLight);).
But, let me know if this is just a matter of adding a few lines to my program.

@sunag
Copy link
Collaborator Author

sunag commented Oct 30, 2023

Hmm... thanks for the example. It looks like there is a conflict in frameId.
Try to remove any requestAnimationFrame(rendAll); and add a single renderer.setAnimationLoop(rendAll);

Related: #24755

Anyway, I'll be checking for a fix for this.

@PhilCrowther
Copy link

Yes, that appears to have fixed the problems I was having with various versions of the program. I had to disable a smoke emitter, but I plan to update that anyways using the WebGPU Particles example. Thanks for the quick response!

This was referenced Nov 2, 2023
@mrdoob mrdoob modified the milestones: r159, r160 Nov 30, 2023
@mrdoob mrdoob modified the milestones: r160, r161 Dec 22, 2023
@mrdoob mrdoob modified the milestones: r161, r162 Jan 31, 2024
@mrdoob mrdoob modified the milestones: r162, r163 Feb 29, 2024
@mrdoob mrdoob modified the milestones: r163, r164 Mar 29, 2024
@mrdoob mrdoob modified the milestones: r164, r165 Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants