-
Notifications
You must be signed in to change notification settings - Fork 56
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
More tests #95
Comments
I am happy to take this issue if no one is working on it. It is my first issue, I might need some help. |
@aliabdelkader yess I left these as first issues for anyone interested in contributing and having these settings tested as part of the CI. You just need to add a configuration and make sure that it's tested in the CI (might require adding some packages). |
#take |
Hi @aliabdelkader and @IlyasMoutawwakil I benchmarked some GPTQ models in the past week and I'd be up to write a test for this quantization scheme if you're not already on it. Let me know |
thanks ! would love to review the quantization configs ! make sure u include |
@lopozz no i did not work on that scheme yet. So, I would not mind if you want to write the test for it. |
Hi @IlyasMoutawwakil I was trying the same configuration file of #125 and I got this error
I believe it is caused by the newest refactoring. The last commit I see is 97e0b9e of 4 days ago, so I believe I am working with the latest version. Should I open a new issue for this? |
@lopozz I think it's a transformers issue unslothai/unsloth#168 (comment), try downgrading ur transformers version |
) (#144) Co-authored-by: Ali Abdelkader <aliabdelkader@users.noreply.github.com>
Hi, I just want to mention that I wont be able to work on the point related to FAv2 in case someone else wants to pick it up. |
Most of these features are covered in
examples
but not in the testing suite.The text was updated successfully, but these errors were encountered: