Skip to content

Fix/1098 explainer python smoke tests - #1141

Merged
yakew7 merged 4 commits into
yakew7:mainfrom
Aaqibhafeezkhan:fix/1098-explainer-python-smoke-tests
Oct 23, 2026
Merged

yakew7 merged 4 commits into
yakew7:mainfrom
Aaqibhafeezkhan:fix/1098-explainer-python-smoke-tests

Conversation

@Aaqibhafeezkhan

Copy link
Copy Markdown
Contributor

What changed

  • Add a pytest smoke-test harness that discovers Python code blocks in explainers/*.md marked with # runnable and executes them in isolated subprocesses from the repository root.
  • Include the explainer filename and code-block number in failure messages, with execution timeouts and support for the existing slow pytest marker through an optional # slow directive.
  • Mark the predictive-parity example as runnable and add a self-contained pandas sample to exercise predictive_parity_gap and base_rate_gap.
  • Document how to mark runnable examples, configure slow examples, and keep snippets self-contained in CONTRIBUTING.md.

Closes #1098

@vercel

vercel Bot commented Oct 23, 2026

Copy link
Copy Markdown
Contributor

@Aaqibhafeezkhan is attempting to deploy a commit to the yashkewlani2020-gmailcom's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown

@yakew7 @Shreyash0712 @ahmdkaml - new PR touching a path you own, please review.

@yakew7
yakew7 merged commit a947e96 into yakew7:main Oct 23, 2026
17 of 18 checks passed
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.

Run the Python code blocks in the explainers as a smoke test

2 participants