MaterialX reference implementation errors

AcademySoftwareFoundation / OpenPBR

Specification and reference implementation for the OpenPBR Surface shading model

Apache License 2.0

407 stars 18 forks source link

Closed JenusL closed 5 months ago

JenusL commented 7 months ago

While trying out OpenPBR I noticed that there's some errors in the MaterialX reference implementation.

transmission_dispersion should be removed.
transmission_dispersion_abbe_number is missing.
transmission_dispersion_scale is missing.
transmission_scatter_anisotropy has uimin set to 0, this should be -1 so that it can allow for back scattering.

portsmouth commented 6 months ago

jstone-lucasfilm commented 5 months ago

Thanks for posting this issue, @JenusL, and thanks to @portsmouth for the fix!