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

ENH: Add Python Jupyter Notebook testing procedure #50

Merged
merged 2 commits into from
Jan 5, 2023

Conversation

tbirdso
Copy link
Collaborator

@tbirdso tbirdso commented Jan 5, 2023

Adds job to build-test-package-python workflow that retrieves a built Linux Python wheel artifact previously output by the workflow and uses it to test Jupyter Notebooks in the examples/ folder of the external module.

For initial implementation only Linux wheels are tested with the Python version fixed at 3.10. To coincide with the Github Actions Linux runner image and machine architecture, only the _2_28-x64 wheel is used.

Depends on #17. Tested in https://github.com/tbirdso/ITKSplitComponents/actions/runs/3843342651. Closes #49 .

EDIT: Note that the ITKSplitComponents test job fails because the ITKSplitComponents example notebook needs to be updated (current kernel is python2). What is important is that the newly built package is correctly installed and the notebook test runs, which is confirmed by looking back through the logs.

@dzenanz
Copy link
Member

dzenanz commented Jan 5, 2023

Looks really cool!

@tbirdso tbirdso force-pushed the manylinux-options branch from bdc5501 to 68f3ead Compare January 5, 2023 17:08
@tbirdso tbirdso linked an issue Jan 5, 2023 that may be closed by this pull request
Adds job to `build-test-package-python` workflow that retrieves a built
Linux Python wheel artifact previously output by the workflow and uses
it to test Jupyter Notebooks in the `examples/` folder of the external
module.

For initial implementation only Linux wheels are tested with the Python
version fixed at 3.10. To coincide with the Github Actions Linux runner
image and machine architecture, only the `_2_28-x64` wheel is used.

Closes
InsightSoftwareConsortium#49
.
@tbirdso tbirdso force-pushed the add-notebook-check branch from af158b6 to a79a8ed Compare January 5, 2023 17:12
@tbirdso tbirdso changed the base branch from manylinux-options to main January 5, 2023 17:13
@tbirdso tbirdso marked this pull request as ready for review January 5, 2023 17:14
@tbirdso tbirdso merged commit e63df7a into InsightSoftwareConsortium:main Jan 5, 2023
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.

Add Jupyter Notebook Testing Workflow
3 participants