-
Notifications
You must be signed in to change notification settings - Fork 33
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
Switch workflows to not use intel channel. #1483
Conversation
0549fa1
to
40d13b6
Compare
95c8f7b
to
5cc320d
Compare
Documentation preview: show. |
5cc320d
to
2020c88
Compare
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.
Give it a once over. All changes look good. Thanks @diptorupd !
.github/workflows/conda-package.yml
Outdated
@@ -170,7 +170,7 @@ jobs: | |||
# We want to make sure that all dependecies install automatically. | |||
# intel::intel-opencl-rt is needed for set-intel-ocl-icd-registry.ps1 | |||
- name: Install built package | |||
run: conda install ${{ env.PACKAGE_NAME }}=${{ env.PACKAGE_VERSION }} intel::intel-opencl-rt -c ${{ env.CHANNEL_PATH }} | |||
run: conda install ${{ env.PACKAGE_NAME }}=${{ env.PACKAGE_VERSION }} https://software.repos.intel.com/python/conda::intel-opencl-rt -c ${{ env.CHANNEL_PATH }} --override-channels |
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.
run: conda install ${{ env.PACKAGE_NAME }}=${{ env.PACKAGE_VERSION }} https://software.repos.intel.com/python/conda::intel-opencl-rt -c ${{ env.CHANNEL_PATH }} --override-channels | |
run: conda install ${{ env.PACKAGE_NAME }}=${{ env.PACKAGE_VERSION }} 'https://software.repos.intel.com/python/conda::intel-opencl-rt' -c ${{ env.CHANNEL_PATH }} |
Try this. I guess bash may split arguments...
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.
No, it did not work
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.
run: conda install ${{ env.PACKAGE_NAME }}=${{ env.PACKAGE_VERSION }} https://software.repos.intel.com/python/conda::intel-opencl-rt -c ${{ env.CHANNEL_PATH }} --override-channels | |
run: conda install ${{ env.PACKAGE_NAME }}=${{ env.PACKAGE_VERSION }} intel-opencl-rt -c ${{ env.CHANNEL_PATH }} |
This may work
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.
There is a cf repack, so it must work now: https://anaconda.org/conda-forge/intel-opencl-rt
@oleksandr-pavlyk @diptorupd if we decide it is good to go with just |
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.
LGTM!
Co-authored-by: Yevhenii Havrylko <[email protected]>
f924da6
to
623a204
Compare
Documentation preview removed. |
Switch workflows to not use intel channel. dc94b7e
Changes existing github workflows based on Error during installation with conda: FORBIDDEN for channel intel <https://conda.anaconda.org/intel> #1480 (comment)
Fixes #1481