-
Notifications
You must be signed in to change notification settings - Fork 23
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
Improve CI #22
Improve CI #22
Conversation
Codecov Report
@@ Coverage Diff @@
## main #22 +/- ##
==========================================
- Coverage 82.25% 80.44% -1.81%
==========================================
Files 39 36 -3
Lines 3330 2890 -440
==========================================
- Hits 2739 2325 -414
+ Misses 591 565 -26
|
5405a9c
to
0519e73
Compare
When jax is installed, any numpy array will use the jax_fns xnp backend. When jax is not installed, now all numpy arrays will use the np_fns xnp backend.
0519e73
to
c4de357
Compare
|
||
on: | ||
pull_request: | ||
branches: [ main ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We used to run CI on all pull requests. For now I think it makes sense to just run it on main, but I could be convinced otherwise.
|
||
on: | ||
push: | ||
branches: [ main ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here, I feel more strongly that CI should only be run on pushes to the main branch. We were running CI twice for PRs (once on branch push, once for the PR).
These markings make it possible to run the test suite when only one of the backends are installed. It also skips big operator tests on on CI. NOTE: for all future testing, neither torch, jax, cola.torch_fns, nor cola.jax_fns can be imported in a testing file. - With the 'not jax' mark, all non-jax tests run. - With the 'not torch' mark, all non-torch tests run. - With the 'not big' mark, all non-big LO tests run.
The unit tests are now run on 4 configurations: - jax installed only - torch (stable) installed only - torch (latest/unstable) installed only - both jax and torch are installed Codecov report is updated only for the jax + torch backend configuration. Other updates: - Set python-version to 3.10 to match minimum requirements - Only run CI on PRs/pushes to main branch
c4de357
to
561f4b8
Compare
I will leave the "run example notebook smoke tests" task for a separate PR. |
Fixes #19
CI improvements
Run example notebook smoke testsNecessary fixes discovered by CI improvements