Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test notebooks with papermill #86

Open
TTitcombe opened this issue Feb 10, 2021 · 1 comment
Open

Test notebooks with papermill #86

TTitcombe opened this issue Feb 10, 2021 · 1 comment
Labels
Good first issue 🎓 Perfect for beginners, welcome to OpenMined! Status: Available 👋 Available for assignment, who wants it? Type: Testing 🧪 Add testing or improving existing testing of a file, feature, or codebase

Comments

@TTitcombe
Copy link
Member

Description

Use papermill in CI pipeline to test that each notebook example can execute

Type of Test

  • Integration test (e.g. checking if a certain group or set of functionality is working as intended)

Expected Behavior

  • Papermill added as developer dependency
  • Add a step in the CI process for branches and master that:
    • Executes each notebook using papermill
    • Tests fail if a single notebook fails
    • Process for slow-to-run notebooks TBD

Additional Context

See PySyft for examples

@TTitcombe TTitcombe added Good first issue 🎓 Perfect for beginners, welcome to OpenMined! Status: Available 👋 Available for assignment, who wants it? Type: Testing 🧪 Add testing or improving existing testing of a file, feature, or codebase labels Feb 10, 2021
@alex-treebeard
Copy link

hey folks, just stumbled across this issue. You may find nbmake useful here.

Feel free to leave any issues/feedback

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good first issue 🎓 Perfect for beginners, welcome to OpenMined! Status: Available 👋 Available for assignment, who wants it? Type: Testing 🧪 Add testing or improving existing testing of a file, feature, or codebase
Projects
None yet
Development

No branches or pull requests

2 participants