-
Notifications
You must be signed in to change notification settings - Fork 30
Set up CI for Python package with GitHub Actions #360
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
Set up CI for Python package with GitHub Actions #360
Conversation
@@ -85,7 +85,7 @@ | |||
"-DDPCPP_INSTALL_DIR=" + DPCPP_ROOT, | |||
"-DCMAKE_C_COMPILER:PATH=" + os.path.join(DPCPP_ROOT, "bin", "clang"), | |||
"-DCMAKE_CXX_COMPILER:PATH=" + os.path.join(DPCPP_ROOT, "bin", "dpcpp"), | |||
"-DDPCTL_ENABLE_LO_PROGRAM_CREATION=ON", | |||
# "-DDPCTL_ENABLE_LO_PROGRAM_CREATION=ON", |
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.
It is blocker for the PR and should be fixed before merging this PR.
LevelZero is not available.
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.
@@ -0,0 +1,13 @@ | |||
#!/bin/bash |
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.
This script from https://github.com/oneapi-src/oneapi-ci
@@ -0,0 +1,23 @@ | |||
#!/bin/bash |
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.
This script from https://github.com/oneapi-src/oneapi-ci
Cython | ||
numpy | ||
pytest | ||
pytest-cov |
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.
Created requirements.txt
for setup python environment.
- name: Exclude unused files from oneAPI cache | ||
if: steps.cache-oneapi.outputs.cache-hit != 'true' | ||
run: scripts/cache_exclude_linux.sh |
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.
This makes oneAPI cache folder from 800Mb to 330Mb.
with: | ||
fetch-depth: 0 |
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.
It fetches whole repository history. Is not necessary but with it we can see package version based on tag.
Convert to draft because problem with LevelZero (#361) is not fixed yet. |
😃👍🏼👍🏼 |
branches: [master] | ||
|
||
env: | ||
LINUX_BASEKIT_URL: https://registrationcenter-download.intel.com/akdlm/irc_nas/17769/l_BaseKit_p_2021.2.0.2883_offline.sh |
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.
@PokhodenkoSA Why use this hard coded url? Instead we should just use the APT oneAPI repo. Github actions allow using password-less sudo
.
Another option can be to use nightly builds of dpcpp from https://github.com/intel/llvm/releases. I am working on a set of clean ups to our build system that will make it very easy to build with the open source dpcpp.
Deprecated. See #515. |
This PR creates CI based on GitHub Actions.
python setup.py develop