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

notebook testing with github actions #20

Merged
merged 5 commits into from
Aug 30, 2022

Conversation

wd15
Copy link
Contributor

@wd15 wd15 commented Aug 30, 2022

This adds a single notebook (AIMS2022_Python_Basics.ipynb) to CI testing and linting using GitHub Actions. The environment is built with Nix, but Conda would be just the same. The notebook has been cleaned up somewhat to account for various issues. Tests run in under 4 minutes.

This issue, usnistgov/jarvis#251, was noticed in the notebook.

Need to switch on the github actions for this repository and then I'll reopen this PR to trigger them

wd15 added 5 commits August 29, 2022 14:18
Comment out pip install statements for consistent environment. Don't
install temp files in locations witout permissions (issue with
Jarvis-tools). Add NBVal statements to ignore stochastic outputs.
@wd15 wd15 force-pushed the ci-notebook-testing branch from a6eecbb to 65eb5e1 Compare August 30, 2022 17:00
@knc6 knc6 merged commit c19b0cb into JARVIS-Materials-Design:master Aug 30, 2022
@knc6
Copy link
Contributor

knc6 commented Aug 30, 2022

Thanks @wd15

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