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.

Sign up for GitHub

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

Jump to bottom

Continuous integration #31

Merged
merged 49 commits into from
Dec 10, 2020
Merged

Continuous integration #31

merged 49 commits into from
Dec 10, 2020

Conversation

balbasty
Copy link
Owner

@balbasty balbasty commented Dec 10, 2020 •
edited
Loading

This is a first working version of the continuous integration workflow:

  • compilation is tested on a matrix of Python x PyTorch x CUDA versions (only on ubuntu for the moment) and triggered when c++/cuda files change (or when setup.py changes)
  • tests are run on two versions of PyTorch (on cpu only since the servers do not have a gpu) and triggered when any of the library files change (nitorch/**)

I had to disable one test that failed (grid_grad -> see #29)

I also had to disable compilation on Python 3.7 which fails weirdly (see #30)

Compilation on mac and windows is not there yet (needs a lot more work)

@balbasty
Fix(setup): be a bit more robust with openmp
bcd3aa8
@balbasty
Merge branch 'master' into buildsystem
3eb5139
@balbasty
Merge branch 'master' into buildsystem
5df9b34
@balbasty
Merge branch 'master' into buildsystem
b144c2c
@balbasty
Enhancement(setup): find arch list in libtorch
ad3e85b
@balbasty
Merge branch 'master' into buildsystem
77de355
@balbasty
Merge branch 'master' into buildsystem
33116ac
@balbasty
Fix(setup): missing space
a6e7aa6
@balbasty
Fix(setup): oops... started counting at 1
34d2354
@balbasty
Fix(setup): always add '/usr/include' to include_dirs
91cdb1f
@balbasty
Refactor(ci)
49de03d
@balbasty
Fix(ci)
9b9230f
@balbasty
Fix(ci)
5530bf5
@balbasty
CI: activate build/test
d64c043
@balbasty
CI: do not lint for now (needs advanced config)
a1d94f9
@balbasty
CI(test
9c317aa
@balbasty
pytest needs test folders to be proper modules
717423f
@balbasty
Ci(test): build in devel
44dacc5
@balbasty
Fix: remove dependency to nibabel
25d7df8
@balbasty
CI(test): install optional dependencies
dd43139
@balbasty
Fix failing test (behaviour of neg2pos had changed
450d386
@balbasty
FixMe(grid_grad): disabled failing test -- backward pass of grid_grad…
a867183 
… is wrong
@balbasty
Fix(io.tests): test files must be prefixed 'test_'
637c81e
@balbasty
Fix(ci)
afe33ea
@balbasty
Fix(install_pytorch): bad cuda suffix
de584ea
@balbasty
Fix(install_pytorch): default cuda versions
8ffa78c
@balbasty
Fix(ci.build): include/exclude
1f1a6f3
@balbasty
Fix(ci.build): typo
72396b5
@balbasty
Fix(ci.build): ecclude
8e4d860
@balbasty
Fix(ci)
001e5f3
@balbasty balbasty added the build / ci Build system or continuous integration label Dec 10, 2020
@balbasty balbasty merged commit 7d66ff6 into master Dec 10, 2020
@balbasty balbasty deleted the buildsystem branch December 10, 2020 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
build / ci Build system or continuous integration
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@balbasty