WebGPU: Supports to IBL + PBR #24038
Conversation
|
I think we can now remove these two lines from the WebGPUNodeBuilder? Also, can you please add new nodes and functions to the ShaderNode? |
I don't think it's the best add all nodes in |
|
I think it is best to export from the ShaderNode the same nodes as exported from Nodes.js, and if some nodes are "internal" (i.e. not for external user user), we can remove them from both... And about functions I think they all should be exported... |
In fact, this just make sense if Nodes.js no have additional imports that ShaderNode... but seeing the way of things, I think that is better to let users include the nodes and the lighting model for the specific project or include all if desired, others base nodes like MathNode, OperatorNode, TextureNode... must be persistent. But we have time to decide this together with the community. |
|
Merging to sync new updates... |
* add Node.getReference() used to for share a similar node * Remove SPECULAR for now and add EMISSIVE * improve Object3DNode.update() implementation * Nodes: move ./light/ to ./lighting/ * Move to LightingContextNode * fix viewMatrix and update names * update names * Support to RGBMLoader * Rename TextureBias to TextureLevel * NodeMaterial: Improve support to IBL + PBR * add webgpu_loader_gltf example * update examples * cleanup * Delete webgpu_lights_hemisphere.html * cleanup * cleanup imports
Description
This is the first implementation of IBL over PBR in WebGPU.
This implementation open some issues related with
ShaderNodethat I will fix in the next days and weeks.The implementation is not finished yet, but I think that is a good opportunity to merger as we will be working until
r141release.Live
https://raw.githack.com/sunag/three.js/dev-webgpu-pbr/examples/webgpu_loader_gltf.html
This contribution is funded by Google via Igalia