-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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 Copilot Arena leaderboard #3618
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.
I would move this to your own file to make it easier to change in the future.
Undraft it! @infwinston |
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 overall looks great @adityamittal13 @waynchi ! left one comment
fastchat/serve/monitor/monitor.py
Outdated
with gr.Tab("Copilot Arena Leaderboard", id=5): | ||
from fastchat.serve.monitor.copilot_arena import build_copilot_arena_tab | ||
|
||
build_copilot_arena_tab() |
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.
can we make this tab optional and default not showing? Otherwise people who do not have access to COPILOT_ARENA_LEADERBOARD_URL
will error out.
We can add an argument to the build_leaderboard_tab
function
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.
What if for simplicity we just try-catch and log it if it doesn't get built?
32cc477
to
59b3058
Compare
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 @adityamittal13 @waynchi !
Why are these changes needed?
Adding Copilot Arena leaderboard data to the Chatbot Arena leaderboard as a separate tab. Specifically adding a model ranking leaderboard displaying ELO results.
Related issue number (if applicable)
Checks
format.sh
to lint the changes in this PR.