Skip to content

Set up CI for Python package with GitHub Actions#360

Closed
PokhodenkoSA wants to merge 29 commits intoIntelPython:masterfrom
PokhodenkoSA:github-actions-python-package
Closed

Set up CI for Python package with GitHub Actions#360
PokhodenkoSA wants to merge 29 commits intoIntelPython:masterfrom
PokhodenkoSA:github-actions-python-package

Conversation

@PokhodenkoSA
Copy link
Copy Markdown
Contributor

@PokhodenkoSA PokhodenkoSA commented Apr 2, 2021

This PR creates CI based on GitHub Actions.

Comment thread scripts/build_backend.py
"-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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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.

Comment thread scripts/install_linux.sh
@@ -0,0 +1,23 @@
#!/bin/bash
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Comment thread requirements.txt
Comment on lines +1 to +4
Cython
numpy
pytest
pytest-cov No newline at end of file
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

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

@diptorupd
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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