-
Notifications
You must be signed in to change notification settings - Fork 44
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
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: