fix(titles): name a PQ video track without HDR10 metadata HDR

docs/reference/configuration/output.md

@@ -134,8 +134,9 @@ Put the more specific rules first, because unshackle stops at the first match.
     Matching is exact, and `hdr` is a composite label for Dolby Vision titles: a DV title with
     an HDR10 base layer gives `DV.HDR`, and with an HDR10+ base layer `DV.HDR10P`. Plain `DV`
     does **not** match those. Name every variant you want, as the example above does. Note
-    also that HDR10 gives `HDR`, not `HDR10`. The full set of values is `HDR`, `HDR10P`, `DV`,
-    `DV.HDR`, `DV.HDR10P`, and `HLG`, as listed in the
+    also that HDR10 gives `HDR`, not `HDR10`, and so does a PQ video track with no HDR10
+    metadata. The full set of values is `HDR`, `HDR10P`, `DV`, `DV.HDR`, `DV.HDR10P`, and
+    `HLG`, as listed in the
     [variable table](../../guide/output-and-naming.md#template-variables).
 
 ### Operators

tests/core/test_hdr10p_naming.py

@@ -1,4 +1,4 @@
-"""Tests for the HDR tag in the filename context when MediaInfo misses HDR10+."""
+"""Tests for the HDR tag in the filename context when MediaInfo misses HDR10+ or the HDR format."""
 
 from __future__ import annotations
 
@@ -22,7 +22,7 @@ def clean_config(monkeypatch: pytest.MonkeyPatch) -> None:
     monkeypatch.setattr(config, "codec_names", {})
 
 
-def hdr_tag(commercial: str, range_: Video.Range) -> str:
+def hdr_tag(commercial: str, range_: Video.Range, transfer: str = "PQ") -> str:
     movie = Movie(id_="movie-0001", service=DummyService, name="The Film", year=2024, language="en")
     movie.tracks = Tracks(Video(url="https://example.test/v.mpd", language="en", codec=Video.Codec.AV1, range_=range_))
     video = types.SimpleNamespace(
@@ -33,7 +33,7 @@ def hdr_tag(commercial: str, range_: Video.Range) -> str:
         other_display_aspect_ratio=["16:9"],
         hdr_format_commercial=commercial,
         hdr_format="SMPTE ST 2086",
-        transfer_characteristics="PQ",
+        transfer_characteristics=transfer,
         transfer_characteristics_original=None,
         frame_rate="23.976",
     )
@@ -51,3 +51,15 @@ def test_an_hdr10_track_keeps_the_hdr_tag() -> None:
 
 def test_mediainfo_hdr10p_is_unchanged() -> None:
     assert hdr_tag("HDR10 / HDR10+", Video.Range.HDR10) == "HDR10P"
+
+
+def test_pq_without_an_hdr_format_gets_the_hdr_tag() -> None:
+    assert hdr_tag("", Video.Range.HDR10) == "HDR"
+
+
+def test_hlg_without_an_hdr_format_keeps_the_hlg_tag() -> None:
+    assert hdr_tag("", Video.Range.HLG, transfer="HLG") == "HLG"
+
+
+def test_sdr_gets_no_hdr_tag() -> None:
+    assert hdr_tag("", Video.Range.SDR, transfer="BT.709") == ""

unshackle/core/titles/title.py

@@ -173,6 +173,8 @@ class Title:
                         context["hdr"] = DYNAMIC_RANGE_MAP["HDR10+"]
             elif trc and "HLG" in trc:
                 context["hdr"] = "HLG"
+            elif trc and "PQ" in trc:
+                context["hdr"] = "HDR"
             else:
                 context["hdr"] = ""
             context["hdr"] = ".".join(filter(None, map(rename, context["hdr"].split("."))))