Skip to content

fix: Correcting the cohere lstrip bug in cohere#3610

Merged
Samoed merged 2 commits intoembeddings-benchmark:mainfrom
fzoll:cohere_correction
Nov 24, 2025
Merged

fix: Correcting the cohere lstrip bug in cohere#3610
Samoed merged 2 commits intoembeddings-benchmark:mainfrom
fzoll:cohere_correction

Conversation

@fzoll
Copy link
Contributor

@fzoll fzoll commented Nov 24, 2025

Correcting the cohere lstrip bug

@Samoed
Copy link
Member

Samoed commented Nov 24, 2025

Hm, yes.

"Cohere/Cohere-embed-multilingual-v3.0"[14:]
# 'embed-multilingual-v3.0'
"Cohere/Cohere-embed-multilingual-v3.0".lstrip("Cohere/Cohere-")
# 'mbed-multilingual-v3.0'

Co-authored-by: Roman Solomatin <samoed.roman@gmail.com>
@Samoed Samoed enabled auto-merge (squash) November 24, 2025 14:18
@Samoed Samoed disabled auto-merge November 24, 2025 14:18
@Samoed Samoed changed the title Correcting the cohere lstrip bug fix: Correcting the cohere lstrip bug in cohere Nov 24, 2025
@Samoed Samoed enabled auto-merge (squash) November 24, 2025 14:19
@Samoed Samoed merged commit 398b31b into embeddings-benchmark:main Nov 24, 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.

2 participants