Principled BSDF handles roughness incorrectly #96527

Closed
opened 2022-03-16 17:41:52 +01:00 by tempdevnova · 8 comments

System Information
Operating system: Windows-10-10.0.19043-SP0 64 Bits
Graphics card: GeForce GTX 1080/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 431.07

Blender Version
Broken: version: 3.0.0, branch: master, commit date: 2021-12-02 18:35, hash: blender/blender@f1cca30557
Worked: (newest version of Blender that worked as expected)

Short description of error
The Blender docs states for the Principled BSDF roughness: When converting from the older Glossy BSDF node, use the square root of the original value.
https://docs.blender.org/manual/en/latest/render/shader_nodes/shader/principled.html

However when doing testing I have found, that to get equal results the Principled BSDF needs exactly the same roughness values as the glossy shader.
Same is also applicable with transmission roughnes and the refraction/glass shader.

Exact steps for others to reproduce the error
This test file clearly demonstrates this in the case of glossy roughness:
Test file.blend

Is maybe just the documentation outdated?

Thank you and have a great day!

**System Information** Operating system: Windows-10-10.0.19043-SP0 64 Bits Graphics card: GeForce GTX 1080/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 431.07 **Blender Version** Broken: version: 3.0.0, branch: master, commit date: 2021-12-02 18:35, hash: `blender/blender@f1cca30557` Worked: (newest version of Blender that worked as expected) **Short description of error** The Blender docs states for the Principled BSDF roughness: `When converting from the older Glossy BSDF node, use the square root of the original value.` https://docs.blender.org/manual/en/latest/render/shader_nodes/shader/principled.html However when doing testing I have found, that to get equal results the Principled BSDF needs exactly the same roughness values as the glossy shader. Same is also applicable with transmission roughnes and the refraction/glass shader. **Exact steps for others to reproduce the error** This test file clearly demonstrates this in the case of glossy roughness: [Test file.blend](https://archive.blender.org/developer/F12930381/Test_file.blend) Is maybe just the documentation outdated? Thank you and have a great day!
Author

Added subscriber: @tempdevnova

Added subscriber: @tempdevnova

Added subscribers: @angavrilov, @mano-wii

Added subscribers: @angavrilov, @mano-wii

Changed status from 'Needs Triage' to: 'Needs Developer To Reproduce'

Changed status from 'Needs Triage' to: 'Needs Developer To Reproduce'

This is the result seen in Blender 2.79
image.png

The Hint was added in {rBM3542} by @angavrilov
Does the "older" in "older Glossy BSDF node" mean the node from older versions of Blender?
If so, this needs to be made clearer to avoid confusion. Otherwise it seems best to just remove that Hint.

This is the result seen in Blender 2.79 ![image.png](https://archive.blender.org/developer/F12931949/image.png) The `Hint` was added in {rBM3542} by @angavrilov Does the "older" in "older Glossy BSDF node" mean the node from older versions of Blender? If so, this needs to be made clearer to avoid confusion. Otherwise it seems best to just remove that `Hint`.

This issue was referenced by 9130

This issue was referenced by 9130

Added subscriber: @brecht

Added subscriber: @brecht

Changed status from 'Needs Developer To Reproduce' to: 'Resolved'

Changed status from 'Needs Developer To Reproduce' to: 'Resolved'
Brecht Van Lommel self-assigned this 2022-03-17 15:59:13 +01:00

This changed in blender/blender@7613ffc, I'll update the docs.

This changed in blender/blender@7613ffc, I'll update the docs.
Sign in to join this conversation.
No Milestone
No project
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender-manual#96527
No description provided.