search

AcademySoftwareFoundation / OpenPBR

Specification and reference implementation for the OpenPBR Surface shading model
Apache License 2.0
425 stars 18 forks source link

Adding a section introducing and linking to the MaterialX reference #119

Closed portsmouth closed 11 months ago

portsmouth commented 11 months ago

As requested, adding a warning box with a list of the functionality which is currently TODO:

image

jstone-lucasfilm commented 11 months ago

I like this general approach, @portsmouth, though we should clarify that our MaterialX implementation is as faithful to the OpenPBR specification as existing shading languages (e.g. OSL, MDL) support. Additional improvements to the functionality in your list will require industry-wide adoption of these features, not merely new MaterialX features.

portsmouth commented 11 months ago

Additional improvements to the functionality in your list will require industry-wide adoption of these features, not merely new MaterialX features.

Note that seems true for some, but not all, of the listed features. The normal map blending and dispersion scaling should be easy to do, we just didn't implement them yet. So I can say "in some cases" perhaps. (Also setting the volumetric parameters of the internal medium seems like it must be supported in the industry in some fashion..).

portsmouth commented 11 months ago

New wording in d7d21c9d686c0d551f08f2acedece11c4a29ab28

image