Skip to content
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

eval script for llama #374

Merged
merged 2 commits into from
Jun 18, 2024
Merged

eval script for llama #374

merged 2 commits into from
Jun 18, 2024

Conversation

HDCharles
Copy link
Contributor

@HDCharles HDCharles commented Jun 15, 2024

Summary: previously we were only doing this in the tests but now we have an eval script to along with generate.py

Test Plan:

python eval.py -q "int4wo-64-gptq"

expected results:

(using meta-llama/Llama-2-7b-chat-hf weights)

Base (bfloat16) | 12.212 |
int8dq | 12.262 |
int8wo | 12.204 |
int4wo-64 | 12.843 |
int4wo-64-GPTQ | 12.489

Reviewers:

Subscribers:

Tasks:

Tags:

Summary: previously we were only doing this in the tests but now we have
an eval script to along with generate.py

Test Plan:

python eval.py -q "int4wo-64-gptq"

Reviewers:

Subscribers:

Tasks:

Tags:
Copy link

pytorch-bot bot commented Jun 15, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/ao/374

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit efb18ce with merge base bc2f8b7 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@HDCharles HDCharles requested a review from msaroufim June 15, 2024 03:40
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 15, 2024
Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:
@@ -0,0 +1,134 @@
# Copyright (c) Meta Platforms, Inc. and affiliates.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! since this is not tested in CI mind just printing the expected output and putting it in the github issue description

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i added the values to the PR description

@HDCharles HDCharles requested a review from msaroufim June 18, 2024 04:30
@msaroufim msaroufim merged commit 69ea59c into main Jun 18, 2024
13 checks passed
@msaroufim msaroufim deleted the 072_script_for_eval branch June 18, 2024 04:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants