Skip to content
This repository was archived by the owner on Feb 2, 2024. It is now read-only.

Improve CI builds #229

Merged
merged 40 commits into from
Oct 23, 2019
Merged

Improve CI builds #229

merged 40 commits into from
Oct 23, 2019

Conversation

Vyacheslav-Smirnov
Copy link
Contributor

@Vyacheslav-Smirnov Vyacheslav-Smirnov commented Oct 15, 2019 •
edited
Loading

  • Decrease Azure build matrix (stay Win and OSX num_pes 0, 2)
  • Build only with numpy 1.16 and Python 3.7 (should decrease single build for twice)
  • Move Linux num_pes 0, 1, 2, 3 (3 - allow_failure) to Travis
  • Set time limit for all builds for 2 hours
  • Add coverage
  • Add flake8
  • Add HPAT wheels build and test on Linux
  • Skip test_intraday for numpes > 1 due to it hangs on nump_pes 2 and 3

Vyacheslav-Smirnov and others added 15 commits October 10, 2019 19:10
@Vyacheslav-Smirnov
Merge pull request #11 from IntelPython/master
26e3cd5 
Merge changes from origin repo
@Vyacheslav-Smirnov
Merge pull request #12 from IntelPython/master
7e59ea7 
Merge changes from origin repo
@Vyacheslav-Smirnov
Improve CI builds
5e4539b 
 - Remove NNUM_PES=3 build from OSX
 - Split Windows build to NUM_PES=0 and NUM_PES=3
 - Add python ./setup.py style
 - Add coveralls
@Vyacheslav-Smirnov
Correct name for Windows NUM_PES=3 build
9a92ceb
@Vyacheslav-Smirnov
Merge pull request #13 from IntelPython/master
62cff87 
Merge changes from origin repo
@Vyacheslav-Smirnov
Fix syntax error in Travis
30fb183
@Vyacheslav-Smirnov
Install numpy for coveralls as required from setup.py
ec1d6eb
@Vyacheslav-Smirnov
Numpy should be installed for check style
93381af
@Vyacheslav-Smirnov
Add HPAT_CONFIG_MPI=False build; add source conda for check style
0264fb3
@Vyacheslav-Smirnov
Fix syntax in Travis config
a913e01
@Vyacheslav-Smirnov
Another change of syntax for Travis
26a933f
@Vyacheslav-Smirnov
Add env to split Travis builds
7001b82
@Vyacheslav-Smirnov
Another try to correctly specify allow_failure for Travis
b59d90e
@Vyacheslav-Smirnov
Try to move allow_failures to upper level
c4e4d24
@Vyacheslav-Smirnov
Install components in travis in one env
f368e79
@shssf
Copy link
Contributor

shssf commented Oct 15, 2019

@Vyacheslav-Smirnov As far as I understand this PR is "work in progress" stage.
Please let me know (by removing label WIP) if it ready to review/merge.
Thank you for PR

@shssf
Copy link
Contributor

shssf commented Oct 15, 2019

@Vyacheslav-Smirnov And please, squash commits

Vyacheslav-Smirnov and others added 12 commits October 16, 2019 19:29
Vyacheslav-Smirnov and others added 11 commits October 21, 2019 17:40
@Vyacheslav-Smirnov Vyacheslav-Smirnov changed the title {WIP} Improve CI builds Improve CI builds Oct 22, 2019
@Vyacheslav-Smirnov Vyacheslav-Smirnov requested review from shssf and Alexander-Makaryev and removed request for shssf October 22, 2019 15:40
@Vyacheslav-Smirnov
Copy link
Contributor Author

Hi @shssf , @Alexander-Makaryev ,
The PR is ready for review - Azure checks are passed, Linux num_pes 3 takes too mach time on Travis thus it fails (it is allowed to fail)
Regarding the coverage - there is a link for this job (https://coveralls.io/jobs/54710087) https://travis-ci.com/IntelPython/hpat/jobs/248300955#L5911
For some reason it is not reported in the PR - I will investigate later

I have not done squash because I had to test each commit in CI

@Vyacheslav-Smirnov
Copy link
Contributor Author

Vyacheslav-Smirnov commented Oct 23, 2019 •
edited
Loading

Hi,
Have done changes to skip test_intraday only for numpes > 1
@Hardcode84, could you please review and if all is good merge the changes?

@Hardcode84 Hardcode84 merged commit 902ea10 into IntelPython:master Oct 23, 2019
@Vyacheslav-Smirnov Vyacheslav-Smirnov deleted the ci/improve_builds branch October 23, 2019 13:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Alexander-Makaryev Alexander-Makaryev Awaiting requested review from Alexander-Makaryev

@shssf shssf Awaiting requested review from shssf

1 more reviewer

@Hardcode84 Hardcode84 Hardcode84 approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Vyacheslav-Smirnov @shssf @Hardcode84 @Alexander-Makaryev