Skip to content

Conversation

emilyfertig
Copy link
Collaborator

No description provided.

@emilyfertig
Copy link
Collaborator Author

This is a proposal for how we could add notebook testing and set test-specific parameters behind the JAS_TEST_MODE flag.

@emilyfertig emilyfertig force-pushed the emilyaf-notebook-tests branch from f2707ba to a51943f Compare September 29, 2025 22:30
Copy link
Collaborator

@jakevdp jakevdp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Another option here that would avoid the search and replace would be to use an environment variable, and in the notebook do

import os
test_mode = os.environ.get('AI_STACK_TEST_MODE', False)

pip freeze
- name: Turn on test mode
run: |
find docs/source -type f \( -name "*.md" -o -name "*.ipynb" \) -exec sed -i "s/JAS_TEST_MODE = False/JAS_TEST_MODE = True/g" {} +
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should avoid JAS as an abbreviation. Maybe AI_STACK_TEST_MODE?

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.

2 participants