Merge remote-tracking branch origin/main into devdays-2025

Author: chellmuth

.github/workflows/main.yml

@@ -7,13 +7,18 @@ on:
   pull_request:
     paths-ignore:
       - '**.md'
+  schedule:
+    # Nightly build
+    - cron: '30 10 * * *'
   workflow_dispatch:
 
 jobs:
 
   build:
     name: Build
     runs-on: ${{ matrix.os }}
+    env:
+      IS_EXTENDED_BUILD: ${{ github.event_name == 'schedule' || github.event_name == 'workflow_dispatch' }}
     strategy:
       fail-fast: false
       matrix:
@@ -109,6 +114,7 @@ jobs:
           python: 3.13
           cmake_config: -G "Visual Studio 17 2022" -A "x64"
           test_shaders: ON
+          extended_build_oiio: ON
 
         - name: Windows_VS2022_x64_SharedLibs
           os: windows-2025
@@ -156,11 +162,10 @@ jobs:
         Add-Content $env:GITHUB_PATH "$PWD/build/installed/bin"
 
     - name: Install OpenImageIO
-      if: matrix.build_oiio == 'ON' && runner.os == 'Windows'
+      if: env.IS_EXTENDED_BUILD == 'true' && matrix.extended_build_oiio == 'ON' && runner.os == 'Windows'
       run: |
-        C:/vcpkg/vcpkg install openimageio --triplet=x64-windows-release
-        Add-Content $env:GITHUB_PATH "C:/vcpkg/installed/x64-windows-release/bin"
-        Add-Content $env:GITHUB_PATH "C:/vcpkg/installed/x64-windows-release/share/openimageio"
+        C:/vcpkg/vcpkg install openimageio --triplet=x64-windows
+        Add-Content $env:GITHUB_PATH "C:/vcpkg/installed/x64-windows/bin"
 
     - name: Install Python ${{ matrix.python }}
       if: matrix.python != 'None'
@@ -178,7 +183,12 @@ jobs:
       run: find source \( -name *.h -o -name *.cpp -o -name *.mm -o -name *.inl \) ! -path "*/External/*" ! -path "*/NanoGUI/*" | xargs clang-format -i --verbose
 
     - name: CMake Generate
-      run: cmake -S . -B build -DMATERIALX_BUILD_PYTHON=ON -DMATERIALX_BUILD_VIEWER=ON -DMATERIALX_BUILD_GRAPH_EDITOR=ON -DMATERIALX_BUILD_TESTS=ON -DMATERIALX_TEST_RENDER=OFF -DMATERIALX_WARNINGS_AS_ERRORS=ON ${{matrix.cmake_config}}
+      shell: bash
+      run: |
+        if [ "${{ env.IS_EXTENDED_BUILD }}" == "true" ] && [ "${{ matrix.extended_build_oiio }}" == "ON" ]; then
+          EXTENDED_BUILD_CONFIG="-DMATERIALX_BUILD_OIIO=ON -DCMAKE_TOOLCHAIN_FILE=C:/vcpkg/scripts/buildsystems/vcpkg.cmake -DVCPKG_TARGET_TRIPLET=x64-windows"
+        fi
+        cmake -S . -B build -DMATERIALX_BUILD_PYTHON=ON -DMATERIALX_BUILD_VIEWER=ON -DMATERIALX_BUILD_GRAPH_EDITOR=ON -DMATERIALX_BUILD_TESTS=ON -DMATERIALX_TEST_RENDER=OFF -DMATERIALX_WARNINGS_AS_ERRORS=ON $EXTENDED_BUILD_CONFIG ${{matrix.cmake_config}}
 
     - name: CMake Build
       run: cmake --build build --target install --config Release --parallel 2
@@ -309,15 +319,15 @@ jobs:
       run: |
         git clone https://github.com/emscripten-core/emsdk --recursive
         cd emsdk
-        ./emsdk install 2.0.20
-        ./emsdk activate 2.0.20
+        ./emsdk install 4.0.8
+        ./emsdk activate 4.0.8
         source ./emsdk_env.sh
         echo "EMSDK=$EMSDK" >> $GITHUB_ENV
 
     - name: Install Node
       uses: actions/setup-node@v4
       with:
-         node-version: '16'
+         node-version: '22.16.0'
 
     - name: JavaScript CMake Generate
       run: cmake -S . -B javascript/build -DMATERIALX_BUILD_JS=ON -DMATERIALX_EMSDK_PATH=${{ env.EMSDK }}

CMakeLists.txt

@@ -316,7 +316,12 @@ else()
         add_link_options(${DYNAMIC_ANALYSIS_OPTIONS})
     endif()
     if(MATERIALX_BUILD_JS)
-        add_compile_options(-fexceptions)
+        if (CMAKE_BUILD_TYPE MATCHES Debug)
+            add_compile_options(-fexceptions)
+        else()
+            add_compile_options(-fexceptions -Os -flto)
+            add_link_options(-flto)
+        endif()
     endif()
 endif()
 
@@ -465,6 +470,10 @@ add_subdirectory(source/MaterialXFormat)
 # Add shader generation subdirectories
 add_subdirectory(source/MaterialXGenShader)
 if(MATERIALX_BUILD_GEN_GLSL OR MATERIALX_BUILD_GEN_OSL OR MATERIALX_BUILD_GEN_OSL_NETWORK OR MATERIALX_BUILD_GEN_MDL OR MATERIALX_BUILD_GEN_MSL)
+    if(MATERIALX_BUILD_GEN_GLSL OR MATERIALX_BUILD_GEN_MSL)
+        add_subdirectory(source/MaterialXGenHw)
+    endif()
+
     if (MATERIALX_BUILD_GEN_GLSL)
         add_definitions(-DMATERIALX_BUILD_GEN_GLSL)
         add_subdirectory(source/MaterialXGenGlsl)

CONTRIBUTING.md

@@ -189,7 +189,96 @@ in the ASWF Slack or in TSC meetings before any PR is submitted, in order to
 solicit feedback, build consensus, and alert all stakeholders to be on the
 lookout for the eventual PR when it appears.
 
-### Coding Conventions
+### Developer Guidelines
+
+The following guidelines represent coding standards that we strive to follow
+in the MaterialX project. While not all existing code may adhere to these
+standards yet, we encourage all new contributions to follow these practices,
+and we welcome incremental improvements to bring existing code into alignment
+with these guidelines.
+
+#### Naming Conventions
+
+Class names should use PascalCase, as in `NodeGraph` or `ShaderGenerator`.
+Method names should use camelCase starting with a lowercase letter, for
+example `getNode` or `setName`. Protected and private member variables should
+use an underscore prefix, such as `_name` or `_parent`. Constants should be
+written in UPPER_CASE with underscores separating words, as in `EMPTY_STRING`
+or `CATEGORY`. Type aliases should append appropriate suffixes to indicate
+their purpose, using `Ptr` for pointers, `Vec` for vectors, `Map` for maps,
+and `Set` for sets.
+
+#### Static Constants and Class Organization
+
+Class members should be organized in order of decreasing visibility: public,
+protected, then private. Static constants should be placed at the end of their
+respective visibility section. String constants should be defined in implementation
+files rather than headers to avoid One Definition Rule violations. The
+`EMPTY_STRING` constant should be used instead of empty string literals (`""`)
+for clarity and consistency.
+
+#### Smart Pointer Conventions
+
+Heap-allocated objects in the public API should always use `shared_ptr` for
+memory management. Type aliases should be defined for all shared pointers,
+following the pattern of `ElementPtr` for `shared_ptr<Element>`. Both mutable
+and const versions of these type aliases should be provided, e.g. `ElementPtr`
+and `ConstElementPtr`. Raw pointers should be avoided except when representing
+non-owning references within implementation details.
+
+#### Const Correctness
+
+Methods that do not modify an object's state should be marked as `const`.
+Accessor methods should provide const versions to enable their use on const
+objects. Type aliases following the pattern `ConstElementPtr` should be used to
+indicate read-only access through shared pointers. Parameters that should not
+be modified within a function should be declared as const.
+
+#### Parameter Passing and Return Values
+
+Strings and complex objects should be passed by `const&` to avoid unnecessary
+copies. Shared pointers should be passed by value since they are designed to
+be cheap to copy. When returning shared pointers, they should be returned by
+value rather than by reference. Methods should be marked as `const` whenever
+they do not modify the object's state.
+
+#### Exception Handling
+
+Exceptions should be used for exceptional conditions rather than for normal
+control flow. Custom exception types should be defined by inheriting from
+`Exception` to represent specific error categories. Exception messages should
+be descriptive and include relevant context to aid in debugging. All exceptions
+that may be thrown by a method should be documented using the `@throws` tag in
+the method's documentation. When catching exceptions, specific exception types
+should be caught rather than generic exceptions whenever possible.
+
+#### Header Includes
+
+Header includes should be written with angle brackets, with paths relative to
+the root source folder (e.g. `#include <MaterialXCore/Element.h>`). This
+ensures consistent include paths across the entire codebase, regardless of the
+location of the referencing file.
+
+Each implementation file should include its corresponding header file first,
+so the first include in `Element.cpp` should be `Element.h`. This ensures that
+the header file is self-contained and doesn't accidentally depend on includes
+from other headers.
+
+After the corresponding header, include blocks should be ordered hierarchically,
+with high-level modules listed before low-level modules (e.g.
+`MaterialXGenShader`, followed by `MaterialXFormat`, followed by
+`MaterialXCore`). This maximizes opportunities to catch missing dependencies in
+the high-level modules, which might otherwise be hidden at build time. Within
+include blocks, individual includes should be ordered alphabetically, providing
+a simple canonical order that is straightforward for developers to check.
+
+In the interest of avoiding include cycles, developers are free to leverage
+forward declarations of classes that are trivially referenced within another
+header. In the interest of clarity and efficiency, developers are free to
+leverage transitive header includes, where low-level headers that have already
+been included by a high-level header do not need to be restated individually.
+
+#### Coding Style
 
 The coding style of the MaterialX project is defined by a
 [clang-format](.clang-format) file in the repository, which is supported by
@@ -200,7 +289,18 @@ file to automatically align the code to MaterialX conventions. When modifying
 existing code, follow the surrounding formatting conventions so that new or
 modified code blends in with the current code.
 
-### Unit Tests
+#### Documentation Standards
+
+All classes and methods in the public API should be documented with Doxygen
+comments. Classes should be documented with the `@class` tag, and structs with
+the `@struct` tag, followed by a brief description and any detailed
+documentation. Method documentation should include `@param`, `@return`, and
+`@throws` tags where applicable. Related methods should be grouped together
+using `/// @name GroupName` sections to improve readability. File-level
+documentation should be placed immediately after the copyright header using
+the `/// @file` directive.
+
+#### Unit Tests
 
 Each MaterialX module has a companion folder within the
 [MaterialXTest](source/MaterialXTest) module, containing a set of unit tests

javascript/MaterialXTest/browser/karma.conf.js

@@ -33,5 +33,15 @@ module.exports = function (config)
             'karma-mocha',
             'karma-mocha-reporter',
         ],
+        customLaunchers: {
+            ChromeHeadlessGL: {
+              base: 'Chrome',
+              flags: [
+                '--headless=new',
+                '--enable-gpu',
+                '--enable-webgl',
+              ],
+            },
+        },
     });
 };

javascript/MaterialXTest/browser/shaderGenerator.spec.js

@@ -12,6 +12,10 @@ function createStandardSurfaceMaterial(mx)
     shaderElement.setType('surfaceshader');
     shaderElement.setNodeName(ssName);
     expect(doc.validate()).to.be.true;
+    // Release local wrappers
+    shaderElement.delete();
+    smNode.delete();
+    ssNode.delete();
     return doc;
 }
 
@@ -94,13 +98,32 @@ describe('Generate Shaders', function ()
                         console.error("--- PIXEL SHADER END ---");
                     }
                     expect(gl.getShaderParameter(glPixelShader, gl.COMPILE_STATUS)).to.equal(true);
+                    // Cleanup GL shaders
+                    gl.deleteShader(glVertexShader);
+                    gl.deleteShader(glPixelShader);
                 }
+                // Cleanup shader wrapper
+                mxShader.delete();
             }
             catch (errPtr)
             {
                 console.error("-------- Failed code generation: ----------------");
-                console.error(mx.getExceptionMessage(errPtr));
+                if (typeof mx.getExceptionMessage === 'function')
+                {
+                    console.error(mx.getExceptionMessage(errPtr));
+                }
+                else
+                {
+                    console.error(errPtr);
+                }
             }
+            // Cleanup per-generator wrappers
+            stdlib.delete();
+            genContext.delete();
+            gen.delete();
         }
+        // Cleanup element and document
+        elem.delete();
+        doc.delete();
     });
 });

javascript/MaterialXTest/codeExamples.spec.js

@@ -76,6 +76,12 @@ describe('Code Examples', () =>
 
         // // Validate the value of roughness in the context of this material.
         // expect(roughness.getBoundValue(material).getValueString()).to.equal('0.5');
+        // Cleanup wrappers
+        nodeDefs.forEach(nd => nd.delete());
+        output.delete();
+        image.delete();
+        nodeGraph.delete();
+        doc.delete();
     });
 
     it('Traversing a Document Tree', async () =>
@@ -101,6 +107,7 @@ describe('Code Examples', () =>
             }
         }
         expect(imageCount).to.greaterThan(0);
+        doc.delete();
     });
 
     it('Building a MaterialX Document', async () =>
@@ -133,5 +140,6 @@ describe('Code Examples', () =>
 
         // expect(materialCount).to.equal(0);
         // expect(shaderInputCount).to.equal(0);
+        doc.delete();
     });
 });