ignore games that are marked as invalid in the catalog

Author: edwardchalstrey1

doc/tutorials/01_quickstart.ipynb

@@ -485,6 +485,27 @@
     "gbt.catalog.games(game_type=\"nfg\", num_players=2)"
    ]
   },
+  {
+   "cell_type": "code",
+   "execution_count": 14,
+   "id": "ff307b74-16bc-4889-b57c-fef3124ce5ca",
+   "metadata": {},
+   "outputs": [
+    {
+     "data": {
+      "text/plain": [
+       "''"
+      ]
+     },
+     "execution_count": 14,
+     "metadata": {},
+     "output_type": "execute_result"
+    }
+   ],
+   "source": [
+    "gbt.catalog.PrisonersDilemma.description"
+   ]
+  },
   {
    "cell_type": "markdown",
    "id": "a919ddf7",
@@ -495,7 +516,7 @@
   },
   {
    "cell_type": "code",
-   "execution_count": 14,
+   "execution_count": 15,
    "id": "6db7a29a",
    "metadata": {},
    "outputs": [
@@ -509,7 +530,7 @@
        "Game(title='Two person Prisoner's Dilemma game')"
       ]
      },
-     "execution_count": 14,
+     "execution_count": 15,
      "metadata": {},
      "output_type": "execute_result"
     }
@@ -535,7 +556,7 @@
   },
   {
    "cell_type": "code",
-   "execution_count": 15,
+   "execution_count": 16,
    "id": "f58eaa77",
    "metadata": {},
    "outputs": [],
@@ -553,7 +574,7 @@
   },
   {
    "cell_type": "code",
-   "execution_count": 16,
+   "execution_count": 17,
    "id": "4119a2ac",
    "metadata": {},
    "outputs": [],

doc/tutorials/02_extensive_form.ipynb

@@ -1238,6 +1238,30 @@
   {
    "cell_type": "code",
    "execution_count": 16,
+   "id": "bd8b291b-6cc6-4f9a-b5c3-0d1f4c05252b",
+   "metadata": {},
+   "outputs": [
+    {
+     "data": {
+      "text/html": [
+       "Game(title='One-shot trust game, after Kreps (1990)')"
+      ],
+      "text/plain": [
+       "Game(title='One-shot trust game, after Kreps (1990)')"
+      ]
+     },
+     "execution_count": 16,
+     "metadata": {},
+     "output_type": "execute_result"
+    }
+   ],
+   "source": [
+    "gbt.catalog.OneShotTrust(unique_NE_variant=True)"
+   ]
+  },
+  {
+   "cell_type": "code",
+   "execution_count": 17,
    "id": "4556c0bf",
    "metadata": {},
    "outputs": [
@@ -1416,7 +1440,7 @@
        "<IPython.core.display.SVG object>"
       ]
      },
-     "execution_count": 16,
+     "execution_count": 17,
      "metadata": {},
      "output_type": "execute_result"
     }
@@ -1442,7 +1466,7 @@
   },
   {
    "cell_type": "code",
-   "execution_count": 17,
+   "execution_count": 18,
    "id": "37c51152",
    "metadata": {},
    "outputs": [],
@@ -1460,7 +1484,7 @@
   },
   {
    "cell_type": "code",
-   "execution_count": 18,
+   "execution_count": 19,
    "id": "0d86a750",
    "metadata": {},
    "outputs": [],
@@ -1479,7 +1503,7 @@
   },
   {
    "cell_type": "code",
-   "execution_count": 19,
+   "execution_count": 20,
    "id": "1bab777f-8a0b-4f1e-9c0c-270690288243",
    "metadata": {},
    "outputs": [],
@@ -1491,7 +1515,7 @@
   },
   {
    "cell_type": "code",
-   "execution_count": 20,
+   "execution_count": 21,
    "id": "2b715221-e427-4092-ad2f-9f4f2b548fa4",
    "metadata": {},
    "outputs": [],

src/pygambit/catalog/catalog.py

@@ -171,17 +171,29 @@ def _generate_contrib_game_classes(catalog: dict[str, dict]) -> None:
             raise ValueError(f"Missing 'file' for catalog entry '{class_name}'")
 
         game_file = entry["file"]
+        metadata = entry.get("metadata", {})
 
-        cls = type(
-            class_name,
-            (CatalogGameFromContrib,),
-            {
-                "game_file": game_file,
-                "__module__": __name__,
-            },
-        )
+        # Build class attributes dict
+        class_attrs = {
+            "game_file": game_file,
+            "__module__": __name__,
+        }
+
+        # Add metadata fields as class attributes
+        if metadata and "valid_game" in metadata and metadata["valid_game"] is False:
+            pass  # Marked as invalid game, do not create class
+        else:
+            if metadata:
+                for key, value in metadata.items():
+                    class_attrs[key] = value
+
+            cls = type(
+                class_name,
+                (CatalogGameFromContrib,),
+                class_attrs,
+            )
 
-        setattr(module, class_name, cls)
+            setattr(module, class_name, cls)
 
 
 # Generate classes at import time