- Notifications
You must be signed in to change notification settings - Fork3.5k
WIP - Support vertex pulling#16826
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:master
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s). |
Snapshot stored with reference name: Test environment: To test a playground add it to the URL, for example: https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/16826/merge/index.html#WGZLGJ#4600 Links to test babylon tools with this snapshot: https://playground.babylonjs.com/?snapshot=refs/pull/16826/merge To test the snapshot in the playground with a playground ID add it after the snapshot query string: https://playground.babylonjs.com/?snapshot=refs/pull/16826/merge#BCU1XR#0 |
Visualization tests for WebGPU |
WebGL2 visualization test reporter: |
It would be a great feature to have! However, I think we should strive to improve integration into the engine so that we/the user don't have to manually manage storage buffers and so that it remains as transparent as possible.
I wonder if we could just create the vertex buffers with STORAGE flags (in addition to the VERTEX + WRITE flags we currently use)... This way, the buffers should be usable in both pulling and non-pulling modes (to be tested, however, and I don't know if adding unnecessary flags has an impact on performance - STORAGE is not necessary in non-pulling mode, and VERTEX is not necessary in pulling mode). In this case, we could add a I'm probably missing a few things, but I think this would be a better way to support this mode. cc@sebavan for the discussion. |
Also, in your PG example, I think you would want to calculate the normal yourself by performing the vector product of two edges of the triangle, because the normal to a vertex is generally the average of the normals of the faces to which that vertex belongs, so it is not the true normal to the face. |
I really like where it is going :-) and I ll chat with Mike after his break. |
a4c7ec9 to8e92eedCompareWebGL2 visualization test reporter: |
Visualization tests for WebGPU |
I've updated the PR with:
I'm not sure if the way I did this is okay but it seems to work. |
Visualization tests for WebGPU |
WebGL2 visualization test reporter: |
Ah, looks like the index buffer in my test is being created as a Uint16Array and WGSL only supports u32 as a type... |
Uh oh!
There was an error while loading.Please reload this page.
Vertex pulling is where you do your own reading of vertex data in the vertex shader instead of relying on the standard vertex attribute pipeline. I'm not sure of the best approach in Babylon but I made some minimal changes to make it work. I'd appreciate some feedback and ideas for a better way to set this up.
First, some motivation:
I want vertex pulling as a way of selecting vertex data from neighboring vertices in the IBL Shadows voxelization shader. By retrieving normal info for the provoking vertex of a triangle, I can selectively swizzle the axes of the position to maximize rasterization area and avoid missing voxels. This eliminates the need for 3-pass voxelization and opens the door for doing realtime voxelization of animated geometry (in WebGPU only due to the need for 3D storage textures).
https://playground.babylonjs.com/?snapshot=refs/pull/16826/merge#XSNYAU#128
For vertex pulling to work, we typically need:
@builtin(vertex_index)to be sequential which is critical for fetching info about neighboring vertices.No changes are needed to Babylon for the first two requirements. For the 3rd point though, we want to do a
engine.drawArraysTypeeven though the mesh has indices. To do this, we setmaterial.useVertexPulling.The shader must alsonot declare vertex attributes.
Thoughts?