Unify transform spreadsheet source

Author: thomasmansencal

README.rst

@@ -44,10 +44,10 @@ The following features are available:
 
     -   Discovery of *aces-dev* *CTL* transforms.
     -   Generation of the *CTL* transforms graph.
-    -   `Spreadsheet <https://docs.google.com/spreadsheets/d/1z3xsy3sF0I-8AN_tkMOEjHlAs13ba7VAVhrE8v4WIyo>`__-driven generation.
+    -   `Unified spreadsheet <https://docs.google.com/spreadsheets/d/1V6tbYwPOK8fssOpO91LTUMV_5Lj1ULjadz65r4zxBHw>`__-driven generation.
 
--   Generators producing the *OCIO* **CG** and **Studio** configurations.
-    -   `Spreadsheet <https://docs.google.com/spreadsheets/d/1PXjTzBVYonVFIceGkLDaqcEJvKR6OI63DwZX0aajl3A>`__-driven generation.
+-   Generators producing the *OCIO* **CG** and **Studio** configurations from
+    the same unified spreadsheet.
 
 -   Included *CLF* transforms along with generator and discovery support.
 

docs/index.rst

@@ -26,10 +26,10 @@ The following features are available:
 
     -   Discovery of *aces-dev* *CTL* transforms.
     -   Generation of the *CTL* transforms graph.
-    -   `Spreadsheet <https://docs.google.com/spreadsheets/d/1z3xsy3sF0I-8AN_tkMOEjHlAs13ba7VAVhrE8v4WIyo>`__-driven generation.
+    -   `Unified spreadsheet <https://docs.google.com/spreadsheets/d/1V6tbYwPOK8fssOpO91LTUMV_5Lj1ULjadz65r4zxBHw>`__-driven generation.
 
--   Generators producing the *OCIO* **CG** and **Studio** configurations.
-    -   `Spreadsheet <https://docs.google.com/spreadsheets/d/1PXjTzBVYonVFIceGkLDaqcEJvKR6OI63DwZX0aajl3A>`__-driven generation.
+-   Generators producing the *OCIO* **CG** and **Studio** configurations from
+    the same unified spreadsheet.
 
 -   Included *CLF* transforms along with generator and discovery support.
 

docs/process.rst

@@ -33,7 +33,7 @@ Analytical & Reference Configs Generation
 The colour conversion graph is then used to create the *Analytical* config that maps 1-to-1 the **aces-dev** **CTL** transforms to theoretical **OpenColorIO** builtin transforms.
 This config does not work but it is useful to test basic generation capabilities whilst diagnosing issues in the mapping.
 
-The *Reference* config is driven by a *CSV* file generated from a `spreadsheet <https://docs.google.com/spreadsheets/d/1z3xsy3sF0I-8AN_tkMOEjHlAs13ba7VAVhrE8v4WIyo>`__ mapping the ``ACEStransformID`` to **OpenColorIO** builtin transforms.
+The *Reference* config is driven by a *CSV* file generated from a `unified spreadsheet <https://docs.google.com/spreadsheets/d/1V6tbYwPOK8fssOpO91LTUMV_5Lj1ULjadz65r4zxBHw/edit#gid=273921464>`__ mapping the ``ACEStransformID`` to **OpenColorIO** builtin transforms. The same spreadsheet drives the *CG* and *Studio* configs.
 
 -   :func:`opencolorio_config_aces.generate_config_aces`
 
@@ -52,7 +52,7 @@ The repository contains code to generate, discover and classify the additional *
 CG Config Generation
 ^^^^^^^^^^^^^^^^^^^^
 
-The *CG* config generator also uses a *CSV* file generated from a `spreadsheet <https://docs.google.com/spreadsheets/d/1PXjTzBVYonVFIceGkLDaqcEJvKR6OI63DwZX0aajl3A/edit#gid=365242296>`__ pivot table that expresses which ``ACEStransformID`` should be used from the ``Reference`` config and which ``CLFtransformID`` should be used from the shipped *CLF* transforms.
+The *CG* config generator also uses a *CSV* file generated from the unified spreadsheet's `CG Config mapping <https://docs.google.com/spreadsheets/d/1V6tbYwPOK8fssOpO91LTUMV_5Lj1ULjadz65r4zxBHw/edit#gid=863695708>`__ pivot table that expresses which ``ACEStransformID`` should be used from the ``Reference`` config and which ``CLFtransformID`` should be used from the shipped *CLF* transforms.
 
 -   :func:`opencolorio_config_aces.generate_config_cg`
 
@@ -63,7 +63,7 @@ The conversion process is as follows:
 Studio Config Generation
 ^^^^^^^^^^^^^^^^^^^^^^^^
 
-The *Studio* config generator follows the same approach but uses a different `pivot table <https://docs.google.com/spreadsheets/d/1PXjTzBVYonVFIceGkLDaqcEJvKR6OI63DwZX0aajl3A/edit#gid=1155125238>`__ of the spreadsheet.
+The *Studio* config generator follows the same approach but uses the unified spreadsheet's `Studio Config mapping <https://docs.google.com/spreadsheets/d/1V6tbYwPOK8fssOpO91LTUMV_5Lj1ULjadz65r4zxBHw/edit#gid=1613109614>`__ pivot table.
 
 -   :func:`opencolorio_config_aces.generate_config_studio`
 

opencolorio_config_aces/config/cg/generate/config.py

@@ -40,6 +40,9 @@
     generate_config,
     named_transform_factory,
 )
+from opencolorio_config_aces.config.generation.spreadsheet import (
+    url_export_transforms_mapping_file,
+)
 from opencolorio_config_aces.config.reference import (
     DescriptionStyle,
     filter_amf_components,
@@ -89,10 +92,8 @@
 
 LOGGER: logging.Logger = logging.getLogger(__name__)
 
-URL_EXPORT_TRANSFORMS_MAPPING_FILE_CG: str = (
-    "https://docs.google.com/spreadsheets/d/"
-    "1PXjTzBVYonVFIceGkLDaqcEJvKR6OI63DwZX0aajl3A/"
-    "export?format=csv&gid=365242296"
+URL_EXPORT_TRANSFORMS_MAPPING_FILE_CG: str = url_export_transforms_mapping_file(
+    863695708
 )
 """
 URL to the *ACES* *CTL* transforms to *OpenColorIO* colorspaces mapping file.
@@ -1314,7 +1315,7 @@ def ordering(element: dict[str, Any]) -> int:
     # Virtual Display Shared Views
     # ============================
     data.virtual_display_shared_views = list(
-        {
+        dict.fromkeys(
             shared_view["view"]
             for shared_view in data.shared_views
             if shared_view["display"]
@@ -1323,7 +1324,7 @@ def ordering(element: dict[str, Any]) -> int:
                 for a in data.colorspaces
                 if a.get("family") == "Display" and a.get("encoding") == "sdr-video"
             ]
-        }
+        )
     )
 
     # Virtual Display Views

opencolorio_config_aces/config/generation/spreadsheet.py

@@ -0,0 +1,25 @@
+# SPDX-License-Identifier: BSD-3-Clause
+# Copyright Contributors to the OpenColorIO Project.
+"""
+OpenColorIO Config Generation Spreadsheet
+=========================================
+"""
+
+from __future__ import annotations
+
+__all__ = [
+    "URL_SPREADSHEET_CONFIG_TRANSFORMS",
+    "url_export_transforms_mapping_file",
+]
+
+URL_SPREADSHEET_CONFIG_TRANSFORMS: str = (
+    "https://docs.google.com/spreadsheets/d/"
+    "1V6tbYwPOK8fssOpO91LTUMV_5Lj1ULjadz65r4zxBHw"
+)
+"""URL to the unified *OpenColorIO-Config-ACES* transforms spreadsheet."""
+
+
+def url_export_transforms_mapping_file(gid: int) -> str:
+    """Return the *CSV* export URL for the given mapping sheet identifier."""
+
+    return f"{URL_SPREADSHEET_CONFIG_TRANSFORMS}/export?format=csv&gid={gid}"

opencolorio_config_aces/config/reference/generate/config.py

@@ -41,6 +41,9 @@
     produce_transform,
     view_transform_factory,
 )
+from opencolorio_config_aces.config.generation.spreadsheet import (
+    url_export_transforms_mapping_file,
+)
 from opencolorio_config_aces.config.reference import (
     classify_aces_ctl_transforms,
     discover_aces_ctl_transforms,
@@ -93,10 +96,8 @@
 
 LOGGER: logging.Logger = logging.getLogger(__name__)
 
-URL_EXPORT_TRANSFORMS_MAPPING_FILE_REFERENCE: str = (
-    "https://docs.google.com/spreadsheets/d/"
-    "1z3xsy3sF0I-8AN_tkMOEjHlAs13ba7VAVhrE8v4WIyo/"
-    "export?format=csv&gid=273921464"
+URL_EXPORT_TRANSFORMS_MAPPING_FILE_REFERENCE: str = url_export_transforms_mapping_file(
+    273921464
 )
 """
 URL to the *ACES* *CTL* transforms to *OpenColorIO* colorspaces mapping file.

opencolorio_config_aces/config/studio/generate/config.py

@@ -27,6 +27,9 @@
     generate_config,
 )
 from opencolorio_config_aces.config.generation.common import ConfigData
+from opencolorio_config_aces.config.generation.spreadsheet import (
+    url_export_transforms_mapping_file,
+)
 from opencolorio_config_aces.config.reference import (
     DescriptionStyle,
 )
@@ -54,10 +57,8 @@
 
 LOGGER: logging.Logger = logging.getLogger(__name__)
 
-URL_EXPORT_TRANSFORMS_MAPPING_FILE_STUDIO: str = (
-    "https://docs.google.com/spreadsheets/d/"
-    "1PXjTzBVYonVFIceGkLDaqcEJvKR6OI63DwZX0aajl3A/"
-    "export?format=csv&gid=1155125238"
+URL_EXPORT_TRANSFORMS_MAPPING_FILE_STUDIO: str = url_export_transforms_mapping_file(
+    1613109614
 )
 """
 URL to the *ACES* *CTL* transforms to *OpenColorIO* colorspaces mapping file.

opencolorio_config_aces/utilities/common.py

@@ -653,11 +653,11 @@ def google_sheet_title(url: str) -> str:
     --------
     >>> url = (
     ...     "https://docs.google.com/spreadsheets/d/"
-    ...     "1z3xsy3sF0I-8AN_tkMOEjHlAs13ba7VAVhrE8v4WIyo/"
+    ...     "1V6tbYwPOK8fssOpO91LTUMV_5Lj1ULjadz65r4zxBHw/"
     ...     "export?format=csv&gid=273921464"
     ... )
     >>> google_sheet_title(url)  # doctest: +SKIP
-    'OpenColorIO-Config-ACES "Reference" Transforms - v...'
+    'OpenColorIO-Config-ACES - Transforms - v4'
     """
 
     class Parser(HTMLParser):