Don't read stacked game/boost clock cells as base == boost

RDNA and Polaris rows put game and boost clocks in one "Core / Clock"
cell, and some list only the game clock. Leave base unknown when a cell
stacks two clocks, and only copy base into boost for pre-2012 boards.

Author: Seungpyo1007

app/ingest/sources/wikipedia_gpu.py

@@ -52,6 +52,8 @@
 # The dataset keeps ATI-branded boards under ``ati``; AMD retired the ATI brand
 # in August 2010 and the curated records switch to ``amd`` from October 2010.
 _ATI_BRAND_END = date(2010, 10, 1)
+# GPU Boost / PowerTune Boost arrived in 2012; before that one core clock is both.
+_BOOST_ERA = date(2012, 1, 1)
 
 MEMORY_TYPES = (
     "GDDR7", "GDDR6X", "GDDR6", "GDDR5X", "GDDR5", "GDDR4", "GDDR3", "GDDR2",
@@ -301,9 +303,20 @@ def unit(key: str) -> str:
         return None
 
     base_clock, range_boost = _clock_mhz(text("base_clock"), unit("base_clock"))
+    unparenthesized = re.sub(r"\([^)]*\)", "", text("base_clock"))
+    if range_boost is None and len(re.findall(r"\d+", unparenthesized)) > 1:
+        # RDNA/Polaris "Core / Clock" cells stack game + boost ("1855 2495"): no base.
+        base_clock = None
     boost_clock = _clock_mhz(text("boost_clock"), unit("boost_clock"))[0] or range_boost
-    if boost_clock is None and base_clock is not None and "boost_clock" not in cells:
+    if (
+        boost_clock is None
+        and base_clock is not None
+        and "boost_clock" not in cells
+        and release_date is not None
+        and release_date < _BOOST_ERA
+    ):
         # Pre-boost boards: the dataset stores boost == base (586/586 pre-2010).
+        # Later single-clock cells are often a game clock, so they stay unknown.
         boost_clock = base_clock
 
     type_width = text("memory_type_width")

tests/unit/test_ingest_wikipedia_gpu.py

@@ -93,3 +93,28 @@ def test_codename_and_bus_helpers() -> None:
     assert normalize_bus_interface("AGP 8×") == "AGP 8x"
     assert normalize_bus_interface("AGP 4× PCI") == "AGP 4x"
     assert normalize_bus_interface("IGP") is None
+
+
+_RDNA_HTML = """
+<table class="wikitable">
+  <tr><th rowspan="2">Model</th><th rowspan="2">Launch</th><th rowspan="2">Architecture</th>
+      <th rowspan="2">Bus interface</th><th colspan="2">Core</th><th colspan="3">Memory</th>
+      <th rowspan="2">TDP</th></tr>
+  <tr><th>Clock (MHz)</th><th>Config</th><th>Size (GB)</th><th>Bus type</th>
+      <th>Bus width (bit)</th></tr>
+  <tr><th>Radeon Pro W7900 Dual Slot</th><td>June 19, 2024</td><td>RDNA 3</td>
+      <td>PCIe 4.0 ×16</td><td>1855 2495</td><td>6144:384:192</td><td>48</td><td>GDDR6</td>
+      <td>384</td><td>295 W</td></tr>
+  <tr><th>Radeon RX 6300M</th><td>January 4, 2022</td><td>RDNA 2</td>
+      <td>PCIe 4.0 ×4</td><td>1512</td><td>768:48:32</td><td>2</td><td>GDDR6</td>
+      <td>32</td><td>25 W</td></tr>
+</table>
+"""
+
+
+def test_game_clock_cells_do_not_become_base_and_boost() -> None:
+    page = "List_of_AMD_graphics_processing_units"
+    cards = {c.slug: c for c in WikipediaGpuIngest._extract(_RDNA_HTML, "amd", page, "")}
+    assert cards["radeon-pro-w7900-dual-slot"].record["base_clock_mhz"] is None
+    assert cards["radeon-rx-6300m"].record["boost_clock_mhz"] is None
+    assert not cards["radeon-rx-6300m"].is_complete