-
Notifications
You must be signed in to change notification settings - Fork 65
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
Continuous Integration Testing framework using GitHub actions #315
Comments
Thanks Romain All recipes in
These two recipes need some review / rework and are currently not expected to pass
Some recipes have cells which are expected to fail / show errors for the purposes of the recipes. These cells are marked with a 'skip-execution' cell tag which skips that cell when run as the Jenkins job, but runs the rest of the notebook below the skipped cell. (It would be even better if they weren't skipped, but it was checked that they generated the expected error and the rest of the notebook was run!) |
@flicj191 is working on a CMOR compatible version for ESMValTool. |
@rbeucher where are we with this? in the |
Our repo file structure changed a lot btw. Do we want the testing to be in a different repo as you have started it @rbeucher or is this just a prototype to see whether the testing works. Seems good and it's great that you get a green or red badge! I'm very interested in incorporating in this repo so people know straight up whether the notebooks are working. |
Hi @navidcy , I think the testing should go in the COSIMA repo. |
Great! Exciting! |
I have started a repo on the ACCESS-NRI organisation to automate testing of the recipes:
https://github.com/ACCESS-NRI/COSIMA-recipes-workflow
This is to eventually replace the Jenkins server on access-dev which will be shut down in Q1 2024.
This is based on what I have set up for the ESMValTool recipes:
https://github.com/ACCESS-NRI/ESMValTool-workflow?tab=readme-ov-file#recipes-current-status
Will keep you posted.
The text was updated successfully, but these errors were encountered: