Implement support for amf_transform_ids.

Closes #154.

Signed-off-by: Thomas Mansencal <[email protected]>

Author: KelSolaar

docs/opencolorio_config_aces.config.rst

@@ -82,6 +82,7 @@ Reference Configuration
     version_aces_dev
     classify_aces_ctl_transforms
     discover_aces_ctl_transforms
+    filter_amf_components
     filter_ctl_transforms
     generate_amf_components
     print_aces_taxonomy

opencolorio_config_aces/__init__.py

@@ -49,6 +49,7 @@
     conversion_path,
     ctl_transform_to_node,
     discover_aces_ctl_transforms,
+    filter_amf_components,
     filter_ctl_transforms,
     filter_nodes,
     generate_amf_components,
@@ -108,6 +109,7 @@
     "conversion_path",
     "ctl_transform_to_node",
     "discover_aces_ctl_transforms",
+    "filter_amf_components",
     "filter_ctl_transforms",
     "filter_nodes",
     "generate_amf_components",

opencolorio_config_aces/config/__init__.py

@@ -30,6 +30,7 @@
     conversion_path,
     ctl_transform_to_node,
     discover_aces_ctl_transforms,
+    filter_amf_components,
     filter_ctl_transforms,
     filter_nodes,
     generate_amf_components,
@@ -75,6 +76,7 @@
     "conversion_path",
     "ctl_transform_to_node",
     "discover_aces_ctl_transforms",
+    "filter_amf_components",
     "filter_ctl_transforms",
     "filter_nodes",
     "generate_amf_components",

opencolorio_config_aces/config/cg/generate/config.py

@@ -42,6 +42,7 @@
 )
 from opencolorio_config_aces.config.reference import (
     DescriptionStyle,
+    filter_amf_components,
     generate_config_aces,
 )
 from opencolorio_config_aces.config.reference.generate.config import (
@@ -1202,10 +1203,25 @@ def remove_existing_named_transform(name: str) -> None:
         clf_transform_id = transform_data["clf_transform_id"]
 
         if style:
+            clf_transform = clf_transform_from_style(style)
+
+            filtered_amf_components = None
+            if (
+                aces_transform_id := clf_transform.information.get(  # pyright: ignore
+                    "ACEStransformID"
+                )
+            ) is not None:
+                filtered_amf_components = filter_amf_components(
+                    amf_components, aces_transform_id.aces_transform_id
+                )
+
             kwargs.update(
                 {
                     "style": style,
-                    "clf_transform": clf_transform_from_style(style),
+                    "clf_transform": clf_transform,
+                    "interchange_mapping": None
+                    if filtered_amf_components is None
+                    else {"amf_transform_ids": ", ".join(filtered_amf_components)},
                 }
             )
 
@@ -1250,7 +1266,24 @@ def remove_existing_named_transform(name: str) -> None:
                 f'"{clf_transform_id}" "CLF" transform does not exist!',
             )
 
-            kwargs["clf_transform"] = clf_transform
+            filtered_amf_components = None
+            if (
+                aces_transform_id := clf_transform.information.get(  # pyright: ignore
+                    "ACEStransformID"
+                )
+            ) is not None:
+                filtered_amf_components = filter_amf_components(
+                    amf_components, aces_transform_id.aces_transform_id
+                )
+
+            kwargs.update(
+                {
+                    "clf_transform": clf_transform,
+                    "interchange_mapping": None
+                    if filtered_amf_components is None
+                    else {"amf_transform_ids": ", ".join(filtered_amf_components)},
+                }
+            )
 
             if transform_data["interface"] == "NamedTransform":
                 LOGGER.info(

opencolorio_config_aces/config/generation/configuration.py

@@ -91,19 +91,19 @@ def extended_fields(self) -> dict[str, str]:
     BuildConfiguration(
         aces=Version(2, 0),
         colorspaces=Version(3, 0, 0),
-        ocio=Version(2, 4),
+        ocio=Version(2, 5),
         variant="",
     ),
     BuildConfiguration(
         aces=Version(2, 0),
         colorspaces=Version(3, 0, 0),
-        ocio=Version(2, 4),
+        ocio=Version(2, 5),
         variant="D60 Views",
     ),
     BuildConfiguration(
         aces=Version(2, 0),
         colorspaces=Version(3, 0, 0),
-        ocio=Version(2, 4),
+        ocio=Version(2, 5),
         variant="All Views",
     ),
 ]

opencolorio_config_aces/config/generation/factories.py

@@ -125,6 +125,7 @@ def colorspace_factory(
     is_data: bool | None = None,
     reference_space: str | int | None = None,
     interop_id: str | None = None,
+    interchange_mapping: dict[str, str] | None = None,
     base_colorspace: Mapping[str, Any] | ocio.ColorSpace | None = None,
     **kwargs: Any,
 ) -> ocio.ColorSpace:
@@ -166,6 +167,9 @@ def colorspace_factory(
         See https://github.com/AcademySoftwareFoundation/ColorInterop/blob/\
 5aebc3f37ac192c86694a47bb92fa65cc95e4e67/Recommendations/\
 01_TextureAssetColorSpaces/TextureAssetColorSpaces.md
+    interchange_mapping : dict, optional
+        Mapping of key and value pairs for interchange, e.g.,
+        `amf_transform_ids` or `icc_profile_name`.
     base_colorspace : dict or ColorSpace, optional
         *OpenColorIO* base `Colorspace` inherited for initial attribute values.
 
@@ -261,6 +265,10 @@ def colorspace_factory(
     if interop_id is not None:
         colorspace.setInteropID(interop_id)  # pyright: ignore
 
+    if interchange_mapping is not None:
+        for key, value in interchange_mapping.items():
+            colorspace.setInterchangeAttribute(key, value)  # pyright: ignore
+
     return colorspace
 
 

opencolorio_config_aces/config/reference/__init__.py

@@ -6,6 +6,7 @@
     discover_aces_ctl_transforms,
     classify_aces_ctl_transforms,
     unclassify_ctl_transforms,
+    filter_amf_components,
     filter_ctl_transforms,
     generate_amf_components,
     print_aces_taxonomy,
@@ -26,6 +27,7 @@
     "discover_aces_ctl_transforms",
     "classify_aces_ctl_transforms",
     "unclassify_ctl_transforms",
+    "filter_amf_components",
     "filter_ctl_transforms",
     "generate_amf_components",
     "print_aces_taxonomy",

opencolorio_config_aces/config/reference/discover/__init__.py

@@ -6,6 +6,7 @@
     discover_aces_ctl_transforms,
     classify_aces_ctl_transforms,
     unclassify_ctl_transforms,
+    filter_amf_components,
     filter_ctl_transforms,
     generate_amf_components,
     print_aces_taxonomy,
@@ -24,6 +25,7 @@
     "discover_aces_ctl_transforms",
     "classify_aces_ctl_transforms",
     "unclassify_ctl_transforms",
+    "filter_amf_components",
     "filter_ctl_transforms",
     "generate_amf_components",
     "print_aces_taxonomy",

opencolorio_config_aces/config/reference/discover/classify.py

@@ -70,6 +70,7 @@
     "filter_ctl_transforms",
     "print_aces_taxonomy",
     "generate_amf_components",
+    "filter_amf_components",
 ]
 
 LOGGER: logging.Logger = logging.getLogger(__name__)
@@ -1553,9 +1554,31 @@ def generate_amf_components(
                     - {relation}
                 )
 
-    result = {key: sorted(set(value)) for key, value in amf_components.items() if value}
+    return {key: sorted(set(value)) for key, value in amf_components.items() if value}
 
-    return result
+
+def filter_amf_components(
+    amf_components: dict[str, list[str]], aces_transform_id: str
+) -> list[str] | None:
+    """
+    Filter the *ACES* *AMF* components for specified *ACEStransformID*.
+
+    Parameters
+    ----------
+    amf_components : dict
+        *ACES* *AMF* components to filter.
+    aces_transform_id : str
+        *ACEStransformID* to filter the *ACES* *AMF* components with.
+
+    Returns
+    -------
+    :class:`list`
+        Filtered *ACES* *AMF* components.
+    """
+
+    filtered_amf_components = list(amf_components.get(aces_transform_id, []))
+
+    return filtered_amf_components if len(filtered_amf_components) else None
 
 
 if __name__ == "__main__":

opencolorio_config_aces/config/reference/generate/config.py

@@ -44,6 +44,7 @@
 from opencolorio_config_aces.config.reference import (
     classify_aces_ctl_transforms,
     discover_aces_ctl_transforms,
+    filter_amf_components,
     filter_ctl_transforms,
     generate_amf_components,
     unclassify_ctl_transforms,
@@ -1196,6 +1197,10 @@ def generate_config_aces(
 
             display_style = transform_data["linked_display_colorspace_style"]
 
+            filtered_amf_components = filter_amf_components(
+                amf_components, display_style
+            )
+
             display = style_to_display_colorspace(
                 display_style,
                 describe,
@@ -1206,6 +1211,11 @@ def generate_config_aces(
                 categories=transform_data.get("categories"),
                 aliases=transform_data_aliases(transform_data),
                 interop_id=transform_data.get("interop_id"),
+                interchange_mapping=(
+                    None
+                    if filtered_amf_components is None
+                    else {"amf_transform_ids": ", ".join(filtered_amf_components)}
+                ),
             )
             display["transforms_data"] = [transform_data]
             display_name = display["name"]
@@ -1255,6 +1265,10 @@ def generate_config_aces(
                 style,
             )
 
+            filtered_amf_components = filter_amf_components(
+                amf_components, ctl_transform.aces_transform_id.aces_transform_id
+            )
+
             colorspace = ctl_transform_to_colorspace(
                 ctl_transform,
                 describe,
@@ -1270,6 +1284,11 @@ def generate_config_aces(
                 categories=transform_data.get("categories"),
                 aliases=transform_data_aliases(transform_data),
                 interop_id=transform_data.get("interop_id"),
+                interchange_mapping=(
+                    None
+                    if filtered_amf_components is None
+                    else {"amf_transform_ids": ", ".join(filtered_amf_components)}
+                ),
             )
             colorspace["transforms_data"] = [transform_data]
             if colorspace not in colorspaces: