Skip to content

Comments

Model: add e5-nl models#3646

Merged
Samoed merged 2 commits intoembeddings-benchmark:mainfrom
nikolay-banar:e5-nl
Dec 5, 2025
Merged

Model: add e5-nl models#3646
Samoed merged 2 commits intoembeddings-benchmark:mainfrom
nikolay-banar:e5-nl

Conversation

@nikolay-banar
Copy link
Contributor

  • I have filled out the ModelMeta object to the extent possible
  • I have ensured that my model can be loaded using
    • mteb.get_model(model_name, revision) and
    • mteb.get_model_meta(model_name, revision)
  • I have tested the implementation works on a representative set of tasks.
  • The model is public, i.e. is available either as an API or the weight are publicly available to download

@nikolay-banar nikolay-banar changed the title Add e5-nl models Model: add e5-nl models Dec 3, 2025
Copy link
Contributor

@KennethEnevoldsen KennethEnevoldsen left a comment

Choose a reason for hiding this comment

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

A few minor comments - generally it looks good. Any reason why the dataset is not shared?

@nikolay-banar
Copy link
Contributor Author

nikolay-banar commented Dec 3, 2025

A few minor comments - generally it looks good. Any reason why the dataset is not shared?

We mostly used public data (some datasets from BEIR-NL). I included a link to this collection. The LLM generated dataset is not available yet, but we plan to release it soon.

@KennethEnevoldsen
Copy link
Contributor

Fair - you can always add the link in the future - otherwise this is all good. I have set it to auto-merge

@nikolay-banar
Copy link
Contributor Author

Fair - you can always add the link in the future - otherwise this is all good. I have set it to auto-merge

Is auto-merge set? Do I need to do anything else?

@Samoed Samoed merged commit f3e5763 into embeddings-benchmark:main Dec 5, 2025
10 checks passed
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