Skip to content

ci: install matplotlib/seaborn for graphguard tests#2

Merged
MeaFew merged 3 commits into
mainfrom
ci/test-deps-20260725
Jul 25, 2026
Merged

ci: install matplotlib/seaborn for graphguard tests#2
MeaFew merged 3 commits into
mainfrom
ci/test-deps-20260725

Conversation

@MeaFew

@MeaFew MeaFew commented Jul 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • install the plotting/training deps used by the test suite: matplotlib, seaborn, xgboost
  • run graph materialization under coverage so the coverage gate measures download_data / build_graph too

Why

tests/test_explain.py imports graphguard.explain_gnn, which imports matplotlib; the baseline training test imports xgboost. After those were fixed, the 30% coverage gate was still failing because the setup steps that execute download_data and build_graph ran outside coverage.

Test plan

  • CI lint/test/typecheck all pass on the PR branch.

@MeaFew
MeaFew merged commit afb27d8 into main Jul 25, 2026
3 checks passed
@MeaFew
MeaFew deleted the ci/test-deps-20260725 branch July 25, 2026 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant