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

Leaderboard Update, in sync with BFCL April 27th Release #391

Merged
merged 6 commits into from
Apr 27, 2024

Conversation

HuanzhiMao
Copy link
Collaborator

@HuanzhiMao HuanzhiMao commented Apr 27, 2024

As mentioned in #390, in this PR, we fix some inconsistency issues in the cost and latency calculation for open-source models, which are now all calculated when serving the model with vLLM using 8 V100 GPUs. $$\text{Cost} = \text{Latency per 1000 function call} * (\text{8xV100 azure-pay-as-you-go-price per hour / 3600})$$

We want to thank the community for pointing out this oversight. Thanks @abacaj and @Teknium1 for initially raising the issue, and thanks @natikgadzhi @HamelHusain @nicoritschel @winglian @olafgeibig and many others for joining the conversation. We are listening to community feedback and continuously improving our Berkeley Function Calling Leaderboard. Discussions like this serve as great examples. Let us know what you want us to include next!

This PR DOES change the leaderboard scores for costs and latency, but not accuracy.


Co-authored-by: Charlie Cheng-Jie Ji [email protected]
Co-authored-by: Fanjia Yan [email protected]

Copy link
Collaborator

@CharlieJCJ CharlieJCJ left a comment

Choose a reason for hiding this comment

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

Numbers checked, LGTM

ShishirPatil pushed a commit that referenced this pull request Apr 27, 2024
In this PR, we fix some inconsistency issues in the cost and latency
calculation for open-source models, which are now all calculated when
serving the model with [vLLM](https://github.com/vllm-project/vllm)
using 8 V100 GPUs. $$\text{Cost} = \text{Latency per 1000 function call}
* (\text{8xV100 azure-pay-as-you-go-price per hour / 3600})$$

This PR **DOES** change the leaderboard value in the `cost` and
`latency` columns; but it **DOES NOT** change the accuracy score. We
will update the leaderboard in a different PR #391.

We want to thank the community for pointing out this oversight. Thanks
[@abacaj](https://twitter.com/abacaj) and
[@teknium1](https://twitter.com/Teknium1) for initially raising the
issue, and thanks [@natikgadzhi](https://twitter.com/natikgadzhi)
[@HamelHusain](https://twitter.com/HamelHusain)
[@nicoritschel](https://twitter.com/nicoritschel)
[@winglian](https://twitter.com/winglian)
[@olafgeibig](https://twitter.com/olafgeibig) and many others for
joining the conversation. We are listening to community feedback and
continuously improving our Berkeley Function Calling Leaderboard.
Discussions like
[this](https://twitter.com/abacaj/status/1784003306508980250) serve as
great examples. Let us know what you want us to include next!

---------

Co-authored-by: Charlie Cheng-Jie Ji <[email protected]>
Co-authored-by: Fanjia Yan <[email protected]>
@ShishirPatil ShishirPatil merged commit 2c87d43 into ShishirPatil:gh-pages Apr 27, 2024
@HuanzhiMao HuanzhiMao deleted the llama-website branch April 27, 2024 08:48
devanshamin pushed a commit to devanshamin/gorilla that referenced this pull request Jul 9, 2024
…rPatil#390)

In this PR, we fix some inconsistency issues in the cost and latency
calculation for open-source models, which are now all calculated when
serving the model with [vLLM](https://github.com/vllm-project/vllm)
using 8 V100 GPUs. $$\text{Cost} = \text{Latency per 1000 function call}
* (\text{8xV100 azure-pay-as-you-go-price per hour / 3600})$$

This PR **DOES** change the leaderboard value in the `cost` and
`latency` columns; but it **DOES NOT** change the accuracy score. We
will update the leaderboard in a different PR ShishirPatil#391.

We want to thank the community for pointing out this oversight. Thanks
[@abacaj](https://twitter.com/abacaj) and
[@teknium1](https://twitter.com/Teknium1) for initially raising the
issue, and thanks [@natikgadzhi](https://twitter.com/natikgadzhi)
[@HamelHusain](https://twitter.com/HamelHusain)
[@nicoritschel](https://twitter.com/nicoritschel)
[@winglian](https://twitter.com/winglian)
[@olafgeibig](https://twitter.com/olafgeibig) and many others for
joining the conversation. We are listening to community feedback and
continuously improving our Berkeley Function Calling Leaderboard.
Discussions like
[this](https://twitter.com/abacaj/status/1784003306508980250) serve as
great examples. Let us know what you want us to include next!

---------

Co-authored-by: Charlie Cheng-Jie Ji <[email protected]>
Co-authored-by: Fanjia Yan <[email protected]>
aw632 pushed a commit to vinaybagade/gorilla that referenced this pull request Aug 22, 2024
…rPatil#390)

In this PR, we fix some inconsistency issues in the cost and latency
calculation for open-source models, which are now all calculated when
serving the model with [vLLM](https://github.com/vllm-project/vllm)
using 8 V100 GPUs. $$\text{Cost} = \text{Latency per 1000 function call}
* (\text{8xV100 azure-pay-as-you-go-price per hour / 3600})$$

This PR **DOES** change the leaderboard value in the `cost` and
`latency` columns; but it **DOES NOT** change the accuracy score. We
will update the leaderboard in a different PR ShishirPatil#391.

We want to thank the community for pointing out this oversight. Thanks
[@abacaj](https://twitter.com/abacaj) and
[@teknium1](https://twitter.com/Teknium1) for initially raising the
issue, and thanks [@natikgadzhi](https://twitter.com/natikgadzhi)
[@HamelHusain](https://twitter.com/HamelHusain)
[@nicoritschel](https://twitter.com/nicoritschel)
[@winglian](https://twitter.com/winglian)
[@olafgeibig](https://twitter.com/olafgeibig) and many others for
joining the conversation. We are listening to community feedback and
continuously improving our Berkeley Function Calling Leaderboard.
Discussions like
[this](https://twitter.com/abacaj/status/1784003306508980250) serve as
great examples. Let us know what you want us to include next!

---------

Co-authored-by: Charlie Cheng-Jie Ji <[email protected]>
Co-authored-by: Fanjia Yan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants