Commit 1c00ccd
authored
Add specification proposal for maxcomponent and mincomponent nodes (#2876)
## Summary
Add specification proposal for `maxcomponent` and `mincomponent` nodes to the Math Nodes section of the Specification Proposals document.
Closes #2802
## Description
Per the discussion in #2802 with @jstone-lucasfilm :
- These nodes support **all vectorN and colorN types** (`vector2`, `vector3`, `vector4`, `color3`,
`color4`)
- Output type is **float**
The proposal format follows existing nodes such as `magnitude` and `dotproduct` in the Standard Nodes document.
### Use Cases
As mentioned in the issue, two places in the MaterialX data libraries currently use verbose 5-node patterns (3 `extract` + 2 `max`/`min`) that these nodes would simplify to a single node:
1. **Sheen intensity for glTF PBR** (`libraries/bxdf/gltf_pbr.mtlx`) — uses `maxcomponent` pattern
2. **Absorption coefficient for OpenPBR Surface** (`libraries/bxdf/open_pbr_surface.mtlx`) — uses `mincomponent` pattern
### Scope
This PR addresses the specification proposal only. I can work on the follow-up implementation via graph definitions as a
separate PR once the proposal is reviewed and merged.1 parent cb99ac9 commit 1c00ccd
1 file changed
Lines changed: 19 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
290 | 290 | | |
291 | 291 | | |
292 | 292 | | |
| 293 | + | |
| 294 | + | |
| 295 | + | |
| 296 | + | |
| 297 | + | |
| 298 | + | |
| 299 | + | |
| 300 | + | |
| 301 | + | |
| 302 | + | |
| 303 | + | |
| 304 | + | |
| 305 | + | |
| 306 | + | |
| 307 | + | |
| 308 | + | |
| 309 | + | |
| 310 | + | |
| 311 | + | |
293 | 312 | | |
294 | 313 | | |
295 | 314 | | |
| |||
0 commit comments