Skip to content

Comments

fix: set use_instructions to True in models using prompts#79

Merged
KennethEnevoldsen merged 1 commit intoembeddings-benchmark:mainfrom
clinia:fix/set-use-instructions
Dec 29, 2024
Merged

fix: set use_instructions to True in models using prompts#79
KennethEnevoldsen merged 1 commit intoembeddings-benchmark:mainfrom
clinia:fix/set-use-instructions

Conversation

@dbuades
Copy link
Contributor

@dbuades dbuades commented Dec 20, 2024

As a follow up to embeddings-benchmark/mteb#1616, this PR updates the corresponding model_meta.json values for those specific models and revisions.

Note that I did not modify the values for certain Voyage models, as they show up as “no revision available” and lack a model_meta.json file altogether.

Checklist

  • Run tests locally to make sure nothing is broken using make test.
  • Run the results files checker make pre-push.

@Samoed Samoed enabled auto-merge (squash) December 28, 2024 11:17
@Samoed Samoed disabled auto-merge December 29, 2024 09:49
@Samoed Samoed enabled auto-merge (squash) December 29, 2024 09:49
@Samoed
Copy link
Member

Samoed commented Dec 29, 2024

It seems that you need to pull changes from main, but I can't push them to your PR

@KennethEnevoldsen KennethEnevoldsen enabled auto-merge (squash) December 29, 2024 15:36
@KennethEnevoldsen KennethEnevoldsen merged commit 9675eaa into embeddings-benchmark:main Dec 29, 2024
2 checks passed
@dbuades
Copy link
Contributor Author

dbuades commented Jan 3, 2025

Thanks guys, I was out of office for the past few days.

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