-
Notifications
You must be signed in to change notification settings - Fork 464
fix: Added docs for mteb.evaluate
#2831
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
Conversation
- renamed `mteb.run_tasks` to `mteb.evaluate`. Reverting this is fairly easy but I think the rename makes a lot of sense - Added docs to most places - some aren't changed yet as they haven't been tested (#2830) - I didn't change the datasheet to avoid confusion with uploaded datasets partly fixes: #2793
…into docs-for-run-tasks
Renaming is fine, but for when we revisit this months down the road, would you mind noting why renaming is needed? Also what is left to do in #2793? |
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.
Great! I like new naming
The new name was primarily chosen because ChatGPT and Claude kept suggesting It also makes the order of the tasks and models clearer, I think. Evaluate model on task. Where, and run_task doesn't provide the same intuition.
I found a few cases where |
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, sounds good! Just 2 small suggestions.
Co-authored-by: Isaac Chung <chungisaac1217@gmail.com>
Co-authored-by: Isaac Chung <chungisaac1217@gmail.com>
mteb.run_tasks
tomteb.evaluate
. Reverting this is fairly easy but I think the rename makes a lot of sensemteb.run_tasks
is fully MTEB compatible #2830)partly fixes: #2793
If you add a model or a dataset, please add the corresponding checklist: