Skip to content

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

Conversation

PokhodenkoSA
Copy link
Contributor

@PokhodenkoSA PokhodenkoSA commented Apr 2, 2021

This PR creates CI based on GitHub Actions.

@@ -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",
Copy link
Contributor Author

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.

Copy link
Contributor Author

@PokhodenkoSA PokhodenkoSA Apr 2, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ticket for fixing it #361 or #340 will make Level Zero always available by downloading headers from GitHub.

@@ -0,0 +1,13 @@
#!/bin/bash
Copy link
Contributor Author

@PokhodenkoSA PokhodenkoSA Apr 2, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -0,0 +1,23 @@
#!/bin/bash
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines +1 to +4
Cython
numpy
pytest
pytest-cov
Copy link
Contributor Author

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.

Comment on lines +44 to +49
- name: Exclude unused files from oneAPI cache
if: steps.cache-oneapi.outputs.cache-hit != 'true'
run: scripts/cache_exclude_linux.sh
Copy link
Contributor Author

@PokhodenkoSA PokhodenkoSA Apr 2, 2021

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.

Comment on lines +21 to +25
with:
fetch-depth: 0
Copy link
Contributor Author

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.

@PokhodenkoSA
Copy link
Contributor Author

Convert to draft because problem with LevelZero (#361) is not fixed yet.

@diptorupd
Copy link
Contributor

😃👍🏼👍🏼

branches: [master]

env:
LINUX_BASEKIT_URL: https://registrationcenter-download.intel.com/akdlm/irc_nas/17769/l_BaseKit_p_2021.2.0.2883_offline.sh
Copy link
Contributor

@diptorupd diptorupd May 6, 2021

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.

@PokhodenkoSA
Copy link
Contributor Author

Deprecated. See #515.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants