-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Add Yi Handler #543
Add Yi Handler #543
Conversation
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.
LGTM
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.
Hey @fantasist,
Thank you for the PR.
In the decode_ast
function, these lines should be removed. This will result in double-casting issue, and introduce false positives in evaluation score. For a detailed explanation, please check out #516. We addressed this issue for all model handlers there.
The rest of the PR looks great. Excited to see the yi-large-fc
model on the leaderboard!
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.
LGTM.
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.
Thanks for the PR. Can you please fill in the PR description as well! Post which we are happy to merge :)
Done! |
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.
Thanks for the PR @fantasist and the reviews @luojy95! Appreciate your contributions to BFCL and the open-source community!
Yi series models are authored by 01.ai. `yi-large` is known to be one of the SOTA LLMs. This PR adds a model handler for evaluating the public available endpoint of `yi-large-fc`, which is the version of `yi-large` model specialized in function calling capabilities. --------- Co-authored-by: Yunfan Zhong <alanwake@in.ai> Co-authored-by: Hector Lopez <hectorlopeze@in.ai> Co-authored-by: Huanzhi Mao <huanzhimao@gmail.com>
Yi series models are authored by 01.ai.
yi-large
is known to be one of the SOTA LLMs. This PR adds a model handler for evaluating the public available endpoint ofyi-large-fc
, which is the version ofyi-large
model specialized in function calling capabilities.