Skip to content

Conversation

KennethEnevoldsen
Copy link
Contributor

  • 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

partly fixes: #2793

If you add a model or a dataset, please add the corresponding checklist:

- 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
Base automatically changed from main2v2 to v2.0.0 June 16, 2025 20:47
@KennethEnevoldsen KennethEnevoldsen self-assigned this Jun 17, 2025
@KennethEnevoldsen KennethEnevoldsen added the v2 Issues and PRs related to `v2` branch label Jun 17, 2025
@isaac-chung
Copy link
Collaborator

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?

Copy link
Member

@Samoed Samoed left a 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

@KennethEnevoldsen
Copy link
Contributor Author

KennethEnevoldsen commented Jun 22, 2025

Renaming is fine, but for when we revisit this months down the road, would you mind noting why renaming is needed?

The new name was primarily chosen because ChatGPT and Claude kept suggesting mteb.evalaute, and I also thought it was a nicer word.

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.

Also what is left to do in #2793?

I found a few cases where mteb.evaluate couldn't replace mteb.MTEB (e.g. cross-encoders)

Copy link
Collaborator

@isaac-chung isaac-chung left a 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.

KennethEnevoldsen and others added 2 commits June 24, 2025 07:55
Co-authored-by: Isaac Chung <chungisaac1217@gmail.com>
Co-authored-by: Isaac Chung <chungisaac1217@gmail.com>
@KennethEnevoldsen KennethEnevoldsen merged commit 8c79609 into v2.0.0 Jun 24, 2025
4 of 8 checks passed
@KennethEnevoldsen KennethEnevoldsen deleted the docs-for-run-tasks branch June 24, 2025 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v2 Issues and PRs related to `v2` branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants