-
Notifications
You must be signed in to change notification settings - Fork 7
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
check notebooks #17
base: main
Are you sure you want to change the base?
check notebooks #17
Conversation
7c40e30
to
bddaf40
Compare
16ee497
to
e8ee6c0
Compare
python -m nbcollection execute .
|
The CI doesn't appear to be running on github. |
yes, it's running in the fork but it seems not be running in this repository (perhaps because it's not in the main branch). I'm not sure why that is, but for this PR at least we can see the runs in the fork |
Trying close/reopen |
Could you add a simple |
Co-authored-by: William Jamieson <[email protected]>
Am I reading this right that the CI job to check notebooks is not yet running in the main branch? Also, there are errors in the 2 notebooks: Your_first_ASDF_converter.ipynb and Your_second_ASDF_converter.ipynb Both of these notebooks have cells that write files to disk (in a folder created in the home directory). Is this sort of pattern supported by nbcollection? So far I've been unable to replicate those errors locally and am instead running into other errors. |
The problem with these tutorials is when relevant schemas are updated, so do the example files need to be. It's almost like there needs to be a script to reconstruct the example files without manual intervention. |
workflows are currently run here: https://github.com/zacharyburnett/tutorials/actions/workflows/test.yml
I disabled the Python 3.10 workflow for MacOS because it had errors with installation