Additional updates for OCIO (AcademySoftwareFoundation#2323)

- Add documentation for the new `MATERIALX_BUILD_OCIO` flag in the Developer Guide.
- Remove the legacy Python OCIO functionality in `colorspace.py`, which we believe is no longer used in any application or studio pipeline.

Author: jstone-lucasfilm

CMakeLists.txt

@@ -116,8 +116,6 @@ set(MATERIALX_PYTHON_VERSION "" CACHE STRING
     "Python version to be used in building the MaterialX Python package (e.g. '3.9').")
 set(MATERIALX_PYTHON_EXECUTABLE "" CACHE FILEPATH
     "Python executable to be used in building the MaterialX Python package (e.g. 'C:/Python39/python.exe').")
-set(MATERIALX_PYTHON_OCIO_DIR "" CACHE PATH
-    "Path to a folder containing the default OCIO configuration to be packaged with MaterialX Python (e.g. 'D:/Projects/OpenColorIO-Configs/aces_1.0.3').")
 set(MATERIALX_PYTHON_PYBIND11_DIR "" CACHE PATH
     "Path to a folder containing the PyBind11 source to be used in building MaterialX Python.")
 
@@ -187,7 +185,6 @@ mark_as_advanced(MATERIALX_COVERAGE_ANALYSIS)
 mark_as_advanced(MATERIALX_DYNAMIC_ANALYSIS)
 mark_as_advanced(MATERIALX_PYTHON_VERSION)
 mark_as_advanced(MATERIALX_PYTHON_EXECUTABLE)
-mark_as_advanced(MATERIALX_PYTHON_OCIO_DIR)
 mark_as_advanced(MATERIALX_PYTHON_PYBIND11_DIR)
 mark_as_advanced(MATERIALX_OIIO_DIR)
 mark_as_advanced(MATERIALX_OSL_BINARY_OSLC)

documents/DeveloperGuide/MainPage.md

@@ -26,10 +26,11 @@ The Python bindings for MaterialX are based on [PyBind11](https://github.com/pyb
 
 The MaterialX C++ libraries are automatically included when building MaterialX through CMake.
 
-To enable OpenImageIO support in MaterialX builds, the following additional options may be used:
+To enable OpenImageIO and OpenColorIO support in MaterialX builds, the following additional options may be used:
 
 - `MATERIALX_BUILD_OIIO`: Requests that MaterialXRender be built with OpenImageIO in addition to stb_image, extending the set of supported image formats.
 - `MATERIALX_OIIO_DIR`: Path to the root folder of an OpenImageIO installation.  If MATERIALX_BUILD_OIIO has been enabled, then this option may be used to select which installation is used.
+- `MATERIALX_BUILD_OCIO`: Requests that MaterialXGenShader be built with support for custom OpenColorIO color spaces and transforms.  The minimum supported version of OpenColorIO is 2.4.
 
 See the [MaterialX Unit Tests](https://github.com/AcademySoftwareFoundation/MaterialX/tree/main/source/MaterialXTest) page for documentation on shader generation and render testing in GLSL, OSL, and MDL.
 
@@ -42,7 +43,6 @@ By default, the `MATERIALX_BUILD_PYTHON` option will use the active version of P
 
 Additional options for the generation of MaterialX Python include the following:
 
-- `MATERIALX_PYTHON_OCIO_DIR`: Path to a folder containing the default OCIO configuration to be packaged with MaterialX Python. The recommended OpenColorIO configuration for MaterialX is [ACES 1.2](https://github.com/colour-science/OpenColorIO-Configs/tree/feature/aces-1.2-config/aces_1.2).
 - `MATERIALX_PYTHON_PYBIND11_DIR`: Path to a folder containing the PyBind11 source to be used in building MaterialX Python. Defaults to the included PyBind11 source.
 
 ### Building The MaterialX Viewer

python/CMakeLists.txt

@@ -11,13 +11,6 @@ if(SKBUILD)
   )
 endif()
 
-if(MATERIALX_PYTHON_OCIO_DIR)
-  if(NOT EXISTS "${MATERIALX_PYTHON_OCIO_DIR}/config.ocio")
-    message(WARNING "No file named config.ocio was found in the given OCIO directory.")
-  endif()
-  install(DIRECTORY "${MATERIALX_PYTHON_OCIO_DIR}/" DESTINATION "${MATERIALX_PYTHON_FOLDER_NAME}/config/" MESSAGE_NEVER)
-endif()
-
 if(MATERIALX_INSTALL_PYTHON AND PYTHON_EXECUTABLE AND NOT SKBUILD)
   set(SETUP_PY "${CMAKE_INSTALL_PREFIX}/python/setup.py")
   configure_file("${CMAKE_CURRENT_SOURCE_DIR}/setup.py.in" "${SETUP_PY}")

python/MaterialX/__init__.py

@@ -20,11 +20,5 @@
             os.add_dll_directory(bindir)
 
 from .main import *
-from .colorspace import *
-
-try:
-    from .legacy import *
-except ImportError:
-    pass
 
 __version__ = getVersionString()