-
Notifications
You must be signed in to change notification settings - Fork 24.7k
[Doc] Update CMAKE_PREFIX_PATH for XPU windows README #148863
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
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/148863
Note: Links to docs will display an error until the docs builds have been completed. ⏳ 1 Pending, 6 Unrelated FailuresAs of commit 4668109 with merge base f649ee7 ( BROKEN TRUNK - The following jobs failed but were present on the merge base:👉 Rebase onto the `viable/strict` branch to avoid these failures
UNSTABLE - The following job is marked as unstable, possibly due to flakiness on trunk:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
@pytorchbot label "module: xpu" "module: docs" |
README.md
Outdated
:: Set the CMAKE_PREFIX_PATH to help find corresponding packages | ||
:: %CONDA_PREFIX% only works after `conda activate custom_env` | ||
:: Use `$env:CMAKE_PREFIX_PATH = "${env:CONDA_PREFIX};${env:CMAKE_PREFIX_PATH}"` for powershell | ||
set CMAKE_PREFIX_PATH=%CONDA_PREFIX%;%CMAKE_PREFIX_PATH% |
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.
In CMD, assuming CMAKE_PREFIX_PATH
is undefined at the beginning, the value of CMAKE_PREFIX_PATH
will be:
/path/to/your_conda_env;%CMAKE_PREFIX_PATH%
.
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.
Thanks for the suggestion. Updated.
README.md
Outdated
@@ -362,6 +362,16 @@ Please make sure [the common prerequisites](#prerequisites) as well as [the prer | |||
Then PyTorch can be built with the command: | |||
|
|||
```cmd | |||
:: Set the CMAKE_PREFIX_PATH to help find corresponding packages | |||
:: %CONDA_PREFIX% only works after `conda activate custom_env` | |||
:: Use `$env:CMAKE_PREFIX_PATH = "${env:CONDA_PREFIX};${env:CMAKE_PREFIX_PATH}"` for powershell |
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.
\Library
is missing in powershell setting. Can we recommend powershell?
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.
I think we need to align with every others. So I delete the powershell and use cmd commands only.
Co-authored-by: Cui, Yifeng <yifeng.cui@intel.com>
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.
Sure, though it would be nice not to introduce new anaconda mentions in the doc
Sure. Thanks. |
@pytorchbot merge |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
@pytorchbot cherry-pick --onto release/2.7 -c docs |
We found that the `pip install cmake` and `conda install cmake` has different behavior. The reason is that the pip installed one doesn't find the corresponding libs under conda env. So we need to set the `CMAKE_PREFIX_PATH` for alignment. Pull Request resolved: #148863 Approved by: https://github.com/CuiYifeng, https://github.com/malfet Co-authored-by: Cui, Yifeng <yifeng.cui@intel.com> (cherry picked from commit ce52674)
Cherry picking #148863The cherry pick PR is at #150395 The following tracker issues are updated: Details for Dev Infra teamRaised by workflow job |
[Doc] Update CMAKE_PREFIX_PATH for XPU windows README (#148863) We found that the `pip install cmake` and `conda install cmake` has different behavior. The reason is that the pip installed one doesn't find the corresponding libs under conda env. So we need to set the `CMAKE_PREFIX_PATH` for alignment. Pull Request resolved: #148863 Approved by: https://github.com/CuiYifeng, https://github.com/malfet Co-authored-by: Cui, Yifeng <yifeng.cui@intel.com> (cherry picked from commit ce52674) Co-authored-by: Stonepia <tong.su@intel.com>
@pytorchbot --onto release/2.7 -c docs |
❌ 🤖 pytorchbot command failed:
Try |
@pytorchbot cherry-pick --onto release/2.7 -c docs |
Cherry picking #148863Command
Details for Dev Infra teamRaised by workflow job |
We found that the `pip install cmake` and `conda install cmake` has different behavior. The reason is that the pip installed one doesn't find the corresponding libs under conda env. So we need to set the `CMAKE_PREFIX_PATH` for alignment. Pull Request resolved: pytorch#148863 Approved by: https://github.com/CuiYifeng, https://github.com/malfet Co-authored-by: Cui, Yifeng <yifeng.cui@intel.com>
We found that the
pip install cmake
andconda install cmake
has different behavior.The reason is that the pip installed one doesn't find the corresponding libs under conda env. So we need to set the
CMAKE_PREFIX_PATH
for alignment.cc @svekars @sekyondaMeta @AlannaBurke @gujinghui @EikanWang @fengyuan14 @guangyey