aai-institute / continuiti

Learning function operators with neural networks.
GNU Lesser General Public License v3.0
19 stars 3 forks source link

Optimize: CI Pipeline #82

Closed JakobEliasWagner closed 6 months ago

JakobEliasWagner commented 6 months ago

Optimize: CI Pipeline

Description

This pr introduces caching into the setup of dependencies of the runners. This significantly speeds up testing. Reduces CI pipeline runtime by up to 5 minutes.

Which issue does this PR tackle?

How does it solve the problem?

How are the changes tested?

Checklist for Contributors

Checklist for Reviewers:

JakobEliasWagner commented 6 months ago

I will not be adding cached dependencies to the code quality workflow as it is very fast already and does not need this.

JakobEliasWagner commented 6 months ago

@samuelburbulla currently the pr is waiting for 2 expected checks run_tests_macos and run_tests_ubuntu. I combined these tests into a single matrix strategy called run_tests. Additionally, I split up the test into run_tests and run_tests_notebooks to be able to run them in parallel (as dependencies are cached and do not need to be built every time. I think that the main branch is currently protected with the "Require status checks to pass before merging" option with both of these (deprecated) names listed. Would you please have a look at that and if true adjust these names. Thank you :)