-
Notifications
You must be signed in to change notification settings - Fork 8
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
CI Pipeline + test framework for Jupyter Notebooks #19
Comments
The test framework is now available here: https://github.com/ReviewNB/treon |
Hi, any prediction for when this will be a part of ReviewNB? I am trying to decide if I set up a different CI system to run |
@nickponvert I would recommend setting up your CI to use
I've considered making it premium only feature or BYOC (Bring Your Own Compute) but not particularly happy with either of these. One viable path is to restrict free open source repositories to only one parallel build at any time with timeout and cool-off period. If you or anyone from the community have inputs on this, I'm all ears! |
Saturncloud is running into the exact same problem here: https://discourse.jupyter.org/t/bitcoin-mining-abuse-security/1367 There are some solid recommendations for best practises in that thread. |
Problem
Jupyter Notebooks neither have a dedicated test framework nor a CI pipeline. Which results in,
Users tend to jump around Notebook cells while developing, resulting in an unpredictable kernel state. Given Notebooks are prone to muddled state, importance of testing & continuous integration increases multifold.
Solution
Here are the bare minimum things we need,
All of the above is possible today by tinkering with separate tools such as Doctest, unittest, papermill etc. We need to combine it all, fill the gaps and make an open source testing framework dedicated to testing Jupyter Notebooks.
As a last step, this testing framework can then be integrated into ReviewNB interface to run tests after each commit is pushed for users who wish to enable CI pipeline on their repos.
Feel free to upvote/downvote the issue indicating whether you think this is useful feature or not. I also welcome additional questions/comments/discussion on the issue.
The text was updated successfully, but these errors were encountered: