Skip to content

model: add kalm_models ModelMeta (new PR)#2850

Closed
YanshekWoo wants to merge 6 commits intoembeddings-benchmark:mainfrom
KaLM-Team:main
Closed

model: add kalm_models ModelMeta (new PR)#2850
YanshekWoo wants to merge 6 commits intoembeddings-benchmark:mainfrom
KaLM-Team:main

Conversation

@YanshekWoo
Copy link
Contributor

Checklist

  • I did not add a dataset, or if I did, I added the dataset checklist to the PR and completed it.
  • I did not add a model, or if I did, I added the model checklist to the PR and completed it.

Model Checklist

  • 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 wieght are publicly avaiable to download

@YanshekWoo YanshekWoo mentioned this pull request Jun 25, 2025
8 tasks
}

HIT_TMG__KaLM_embedding_multilingual_mini_instruct_v1 = ModelMeta(
name="HIT-TMG/KaLM-embedding-multilingual-mini-instruct-v1",
Copy link
Member

Choose a reason for hiding this comment

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

I think these models are duplicated from kalm_models

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think these models are duplicated from kalm_models

Oh, there might be some mistakes when I merge the code.

The only changes are in kalm_models.py and overview.py.
Should I submit a new PR?

Copy link
Member

Choose a reason for hiding this comment

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

You can change in this PR

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have resubmitted the PR to make the commits clearer and more accurate: #2853

@KaLM-Team KaLM-Team closed this by deleting the head repository Jun 25, 2025
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