-
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
Make bitsandbytes portable (with express goal of supporting Apple Silicon/ARM64) #257
Conversation
Instead of 'ccs is not None', as ccs is always a list, so this if is currently always True
… people aware that a cast happens and the operation quantization is performed in fp16.
77d1cbf
to
a6b944f
Compare
https://github.com/rickardp/bitsandbytes/actions/runs/4719555280 The GH actions added in this PR now builds the library and wheels for the following platforms
i think it could be a good point limit this PR to add this form of portability, and then fork out with different efforts depending on interest
|
I just want to leave a comment here for the community. I am not sure why I closed this. For most issues that I closed I left a comment to why. There was significantly work invested in this and it is a great contribution! Sorry, that I close it like this — it must have been very frustrating. We will try to add Apple silicon support soon along with AMD and Intel support. We need to figure out next steps first though. It will likely take some weeks, but we will get there. This is one of the highest-priority features that we have and we start working on this soon. |
No worries! It is good to hear back from you that portability is a goal and priority. At the time there was a lot of active talk and PRs floating around, even some ideas on forking, in anticipation of an "official" answer from the maintainers. Good to avoid the fragmentation. In one of these issues there were quite a number of people expressing interest in helping out. If you get the basics running you might be able to make good use of the community here to progress faster. |
@rickardp The closing of this PR was an accident it seems. We're really sorry about that. I don't find any active Apple Silicon PRs. What's the status? We would be happy to help. |
Ah, now I see, this got replaced by #947. I also gotta gather info about forks. If you can point me to anything important, let me know. It would be good to avoid fragmentation. BNB now has an official maintenance team and is sponsored by Hugging Face. Therefore, we're quite certain that such things wont go unaddressed for so long in the future. |
Yes, I tried to save portability changes in #947, but I would say that #949 is the successor to this one. The idea was to make it possible to easily parallelize the porting effort by ensuring automated test and cross platform builds using GH actions. This way it would be easy to verify correctness of new kernels. It is good that this project now have a dedicated mantenance team. There were some talks about forking a while back due to the lack of response from maintainers earlier, but AFAIK those efforts are no longer active. I think the discussion were in the now closed PRs and issues (like #485) and were continued on the HF Discord. I had to attend to other stuff after a while so I might have missed out on some discussions. |
Now that this repo is more actively maintained, maybe better to keep the discussions here? @Titus-von-Koeller @TimDettmers have you considered enabling GitHub discussions on this repo? |
Yes, thanks for the hint! I agree, it would be better to keep these discussions here. I'll search the HF Discord to see what I find and look into GH Discussions. There's so much willingness to contribute in the community around this library: really, really cool! I'll work hard to enable people and change the circumstances to make it easier to enable + accept contributions. I'm on bitsandbytes full-time now and @younesbelkada will also contribute a significant fraction of his time. We also have HF as an official sponsor now, yay 🤗 Feel free to contact me (also through messaging, e.g. Slack or Discord) if you have any needs or ideas regarding BNB. Thanks again for your active contributions and sorry for any delays + frustrations you may have experienced! |
Ok, we enabled Discussions and I started one on the cross-platform topic. |
Thanks @rickardp for all your work on this ! Thanks @Titus-von-Koeller for your message 💪 |
See issue #252 for discussion
malloc
and using type-safe C++ primitives)