-
Notifications
You must be signed in to change notification settings - Fork 639
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
Update installation instructions #1047
Conversation
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 ! I left one suggestion - wdyt?
Co-authored-by: Younes Belkada <[email protected]>
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
I verified the instructions on macOS (obviously swapping cuda -> mps) and Linux (well, the cuda container under Docker desktop). Can someone please double check the WIndows instructions as I have no way of testing it currently? |
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 again ! I left more open questions, what do you think?
cmake -B build -DBUILD_CUDA=ON -S . | ||
cmake --build build --config Release | ||
cmake -DCOMPUTE_BACKEND=cuda -S . | ||
cmake --build . --config Release | ||
python -m build --wheel |
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.
python -m build --wheel | |
pip install . |
For consistency with above
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 left it as it is since I can't verify this. Can someone with access to a windows VM double check these?
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.
Ah yes good point - cc @wkpark 🙏
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 works for me with pip install .
.
Alternatively with building the wheel, this also works:
python -m build --wheel
pip install dist/bitsandbytes-0.43.0.dev0-cp311-cp311-win_amd64.whl
Co-authored-by: Younes Belkada <[email protected]>
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 ! We can also merge this PR to at least unblock Linux users and we can always do a follow up PR to fix the windows instructions, what do you think @rickardp ?
I realize this section in this PR duplicates the information i the "compiing from source" document. Better close this and reference this one in PR #1048 instead? |
IMO it is fine to have both as we can see |
IMO good to merge as @matthewdouglas tested it, what do you think @rickardp ? |
(I used
|
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 so much everyone for testing ! 🎉
Fixes: #1042