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

Clear roughness from back of TransmissionRoughnessTest #65

Open
emackey opened this issue Jul 22, 2021 · 4 comments
Open

Clear roughness from back of TransmissionRoughnessTest #65

emackey opened this issue Jul 22, 2021 · 4 comments
Labels
enhancement New feature or request

Comments

@emackey
Copy link
Member

emackey commented Jul 22, 2021

It was noticed that path tracers and rasterizers see different amounts of roughness on a clear volume, given that the path tracer will take into account roughness on the back of a transmissive surface, but a rasterizer will not.

We could consider adjusting the TransmissionRoughnessTest model to remove the roughness from the backs of the sample tiles, so all the roughness is on the fronts only. This might allow a more direct comparison between the different rendering strategies.

@emackey emackey added the enhancement New feature or request label Jul 22, 2021
@abwood
Copy link

abwood commented Jul 22, 2021

Earlier conversation about this topic can be found here: KhronosGroup/glTF-Sample-Models#319 (comment)

@cx20
Copy link

cx20 commented Jul 29, 2021

For reference, I tried to display this model in some libraries.
I thought the change from Smooth to Rough in three.js was a bit steep.

TransmissionRoughnessTest.gltf result:
image
image
image
image
image

Edit 2021.08.20
I added the display result in Dassault path-tracer.

@DRx3D
Copy link
Contributor

DRx3D commented Nov 14, 2023

@emackey : Is there anything to do on this issue? It is not clear to me.

If there are no responses to this comment, the issue will be closed by 27 Nov.

@emackey
Copy link
Member Author

emackey commented Nov 15, 2023

@DRx3D I suppose this issue could be transferred to the new repo. Sadly this isn't high-priority given my current project list at the moment.

It is not clear to me.

This is an epic pun on an issue about a transmissive model with too much roughness.

@DRx3D DRx3D transferred this issue from KhronosGroup/glTF-Sample-Models Nov 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants