Merge pull request #496 from neuromatch/jupyter-book-2

fix: add skip-execution tag to NotImplementedError stub cells for JB2…

Author: iamzoltan

generate_book_v2.py

@@ -235,10 +235,31 @@ def pre_process_notebook(file_path):
     content = open_in_colab_new_tab(content)
     content = change_video_widths(content)
     content = link_hidden_cells(content)
+    content = tag_stub_cells(content)
     with open(file_path, "w", encoding="utf-8") as fh:
         json.dump(content, fh, indent=1, ensure_ascii=False)
 
 
+def tag_stub_cells(content):
+    """Add skip-execution tag to cells containing raise NotImplementedError.
+
+    JB1 used allow_errors:true to silently swallow errors from these stub cells.
+    JB2 has no global equivalent, so we skip execution instead — the source is
+    still rendered (students see the stub), but no error traceback is produced.
+    """
+    for cell in content["cells"]:
+        if cell["cell_type"] != "code":
+            continue
+        if any("NotImplementedError" in s for s in cell.get("source", [])):
+            if "metadata" not in cell:
+                cell["metadata"] = {}
+            if "tags" not in cell["metadata"]:
+                cell["metadata"]["tags"] = []
+            if "skip-execution" not in cell["metadata"]["tags"]:
+                cell["metadata"]["tags"].append("skip-execution")
+    return content
+
+
 def open_in_colab_new_tab(content):
     cells = content["cells"]
     if not cells or not cells[0].get("source"):