python3Packages.bitsandbytes: 0.47.0 -> 0.48.1, support rocm#443210
Conversation
Could copy what torch does to mark broken. Not sure it's really needed since this depends on torch and in most cases you'll see the error that torch can't build for CUDA and ROCm at the same time. |
a102cfe to
0edafbb
Compare
|
Added the suggestion + broken packages part. Also included the other passthru parts and the platform section. One other thing I haven't mentioned is I trimmed out the CUDA part of the GPU targets bit. Potentially we'd need that for cuda too but the CMake does some bits and pieces around capabilities and I'm not sure what I'm doing with that so just dropped it. I've also added myself to the maintainer list since if I'm adding all this extra nix I should at least help maintain it. |
|
0edafbb to
480f7a5
Compare
|
Updated to latest & adjusted for cmake 4. Also after adding a few more libraries to their cmake config so they're actually resolved properly I didn't need to set ROCM_HOME. Only remaining annoyance with their cmake config is some code is built with Just setting C and CXX is a simple fix |
|
|
|
||
| brokenConditions = lib.attrsets.filterAttrs (_: cond: cond) { | ||
| "CUDA and ROCm are mutually exclusive" = cudaSupport && rocmSupport; | ||
| "CUDA is not targeting Linux" = cudaSupport && !stdenv.hostPlatform.isLinux; |
There was a problem hiding this comment.
| "CUDA is not targeting Linux" = cudaSupport && !stdenv.hostPlatform.isLinux; | |
| "CUDA is only available for Linux" = cudaSupport && !stdenv.hostPlatform.isLinux; |
Added support for ROCm since the 0.47.0 update bitsandbytes-foundation/bitsandbytes#1683
Made some small refactors to make more consistent with other packages following off
torch.cuda*All the ROCm stuff is in one drv, I'm not sure I recognise the need for the split between cuda-common-redist, cuda-native-redist, and cuda-redist.
Maybe needs an assert that cudaSupport and rocmSupport are exclusive? 🤷
For testing:
Ran a package that wanted bitsandbytes with ROCm locally (invokeai, still testing other stuff) and at least doesn't complain on launch anymore.
cc: @bcdarwin
Things done
passthru.tests.nixpkgs-reviewon this PR. See nixpkgs-review usage../result/bin/.Add a 👍 reaction to pull requests you find important.