[ImageEngine/cortex] 143e05: DataAlgo : Allow conversion of VtValue without Sdf...

0 views
Skip to first unread message

John Haddon

unread,
Oct 24, 2024, 7:06:39 AM10/24/24
to cortex...@googlegroups.com
Branch: refs/heads/RB-10.5
Home: https://github.com/ImageEngine/cortex
Commit: 143e05b208cd6de13dd5d8645eaa3006c09e4e7f
https://github.com/ImageEngine/cortex/commit/143e05b208cd6de13dd5d8645eaa3006c09e4e7f
Author: John Haddon <thehad...@gmail.com>
Date: 2024-10-24 (Thu, 24 Oct 2024)

Changed paths:
M Changes
M contrib/IECoreUSD/include/IECoreUSD/DataAlgo.h
M contrib/IECoreUSD/src/IECoreUSD/DataAlgo.cpp

Log Message:
-----------
DataAlgo : Allow conversion of VtValue without SdfValueTypeName

This is needed when values aren't sourced from attributes - for instance, from metadata instead.


Commit: 2f375d2bf3efbc9fbb3f61d37b779b184f8a8ee1
https://github.com/ImageEngine/cortex/commit/2f375d2bf3efbc9fbb3f61d37b779b184f8a8ee1
Author: John Haddon <thehad...@gmail.com>
Date: 2024-10-24 (Thu, 24 Oct 2024)

Changed paths:
M Changes
M contrib/IECoreUSD/src/IECoreUSD/DataAlgo.cpp
M contrib/IECoreUSD/test/IECoreUSD/DataAlgoTest.py

Log Message:
-----------
DataAlgo : Add conversions between `VtDictionary` and `CompoundData`


Commit: d4f8106ad76d9a138a27b10e4119931dc7691d3a
https://github.com/ImageEngine/cortex/commit/d4f8106ad76d9a138a27b10e4119931dc7691d3a
Author: John Haddon <thehad...@gmail.com>
Date: 2024-10-24 (Thu, 24 Oct 2024)

Changed paths:
M Changes
M contrib/IECoreUSD/resources/plugInfo.json
M contrib/IECoreUSD/src/IECoreUSD/ShaderAlgo.cpp
M contrib/IECoreUSD/test/IECoreUSD/USDSceneTest.py
A contrib/IECoreUSD/test/IECoreUSD/data/legacyComponentConnections.usda

Log Message:
-----------
USD ShaderAlgo : Replace fixed metadata with generic blind data support

I think we should be moving away from pre-registered metadata entirely, since the definitions will not be available anywhere outside of Gaffer.


Commit: 2ba41a7d2ea3218bb5bf51fac02b605210bd0e6a
https://github.com/ImageEngine/cortex/commit/2ba41a7d2ea3218bb5bf51fac02b605210bd0e6a
Author: John Haddon <thehad...@gmail.com>
Date: 2024-10-24 (Thu, 24 Oct 2024)

Changed paths:
M src/IECoreScene/ShaderNetworkAlgo.cpp

Log Message:
-----------
ShaderNetworkAlgo : Shuffle into related sections

Nothing in the code is changing here - we're just putting related stuff in the same part of the file.


Commit: 38c7fdb6fa6db6b496f6d4b217a11b72c567be0e
https://github.com/ImageEngine/cortex/commit/38c7fdb6fa6db6b496f6d4b217a11b72c567be0e
Author: John Haddon <thehad...@gmail.com>
Date: 2024-10-24 (Thu, 24 Oct 2024)

Changed paths:
M Changes
M include/IECoreScene/ShaderNetworkAlgo.h
M src/IECoreScene/ShaderNetworkAlgo.cpp
M src/IECoreScene/bindings/ShaderNetworkAlgoBinding.cpp
M test/IECoreScene/ShaderNetworkAlgoTest.py
A test/IECoreScene/data/legacyComponentConnectionAdaptors.cob

Log Message:
-----------
ShaderNetworkAlgo : Allow customisation of component adaptors

The main motivation here is that the MaterialX shaders we were using as adaptors don't really make sense - they're not actually available as OSL shaders any more, so they have zero chance of working in other DCCs when we export shading to USD from Gaffer. Instead we can now register renderer-specific adaptors that are available everywhere the renderer is.

This also somewhat changes our approach to removing adaptors. Instead of just tagging them as "an adaptor" and figuring the rest out based on the shader name, we now register all the necessary information as blind data. This will allow us to reliably remove an adaptor even if adaptor registrations change in the future. It should also be useful at a certain studio which rewrites the `info:id` attribute on export to USD, in such a way that it broke our old adaptor removal code.


Commit: bc72de8198baa42fb685c4feea14530b349f3fc0
https://github.com/ImageEngine/cortex/commit/bc72de8198baa42fb685c4feea14530b349f3fc0
Author: John Haddon <thehad...@gmail.com>
Date: 2024-10-24 (Thu, 24 Oct 2024)

Changed paths:
M Changes
M contrib/IECoreUSD/include/IECoreUSD/DataAlgo.h
M contrib/IECoreUSD/resources/plugInfo.json
M contrib/IECoreUSD/src/IECoreUSD/DataAlgo.cpp
M contrib/IECoreUSD/src/IECoreUSD/ShaderAlgo.cpp
M contrib/IECoreUSD/test/IECoreUSD/DataAlgoTest.py
M contrib/IECoreUSD/test/IECoreUSD/USDSceneTest.py
A contrib/IECoreUSD/test/IECoreUSD/data/legacyComponentConnections.usda
M include/IECoreScene/ShaderNetworkAlgo.h
M src/IECoreScene/ShaderNetworkAlgo.cpp
M src/IECoreScene/bindings/ShaderNetworkAlgoBinding.cpp
M test/IECoreScene/ShaderNetworkAlgoTest.py
A test/IECoreScene/data/legacyComponentConnectionAdaptors.cob

Log Message:
-----------
Merge pull request #1438 from johnhaddon/shaderAlgoImprovements

ShaderNetworkAlgo improvements


Compare: https://github.com/ImageEngine/cortex/compare/53bd04357d62...bc72de8198ba

To unsubscribe from these emails, change your notification settings at https://github.com/ImageEngine/cortex/settings/notifications
Reply all
Reply to author
Forward
0 new messages