-
Notifications
You must be signed in to change notification settings - Fork 16.2k
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
community: fix yuan2 errors in LLMs #19004
Conversation
cauwulixuan
commented
Mar 13, 2024
- fix yuan2 errors while invoke Yuan2.
- update tests.
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
"top_p and top_k cannot be set simultaneously. " | ||
"set top_k to 0 instead..." | ||
) | ||
self.top_k = 0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should be checked during initialization
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, thanks for your advice. PTAL;)
It seems like the CI progress is failed with some network error.. Could you trigger it again? @hwchase17 |
Well, all checks have passed. @hwchase17 PTAL, thanks. |
1. fix yuan2 errors while invoke Yuan2. 2. update tests.
1. fix yuan2 errors while invoke Yuan2. 2. update tests.