Fix CI: add matplotlib/seaborn to dev deps, disable fail-fast in test matrix

brycewang-stanford · claude · brycewang-stanford · commit 1e812360d8e4 · 2026-04-03T21:12:23.000-07:00
Tests importing matplotlib were failing with ModuleNotFoundError because
matplotlib was only in the 'plotting' optional extra, not in 'dev'.

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/.github/workflows/ci-cd.yml b/.github/workflows/ci-cd.yml
@@ -12,6 +12,7 @@ jobs:
   test:
     runs-on: ${{ matrix.os }}
     strategy:
+      fail-fast: false
       matrix:
         os: [ubuntu-latest, windows-latest, macos-latest]
         python-version: ['3.9', '3.10', '3.11', '3.12']
diff --git a/pyproject.toml b/pyproject.toml
@@ -58,6 +58,8 @@ dev = [
     "mypy",
     "sphinx",
     "sphinx-rtd-theme",
+    "matplotlib>=3.5.0",
+    "seaborn>=0.11.0",
 ]
 performance = [
     "jax[cpu]>=0.4.0",

Original file line number	Diff line number	Diff line change
`@@ -58,6 +58,8 @@ dev = [`
`58`	`58`	`"mypy",`
`59`	`59`	`"sphinx",`
`60`	`60`	`"sphinx-rtd-theme",`
	`61`	`+ "matplotlib>=3.5.0",`
	`62`	`+ "seaborn>=0.11.0",`
`61`	`63`	`]`
`62`	`64`	`performance = [`
`63`	`65`	`"jax[cpu]>=0.4.0",`