Skip to content

Commit 2158bf8

Browse files
Update space type (#574)
Signed-off-by: Nathalie Jonathan <[email protected]>
1 parent 1c0498b commit 2158bf8

File tree

3 files changed

+7
-6
lines changed

3 files changed

+7
-6
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
3535
- Update pretrained_models_all_versions.json by @nathaliellenaa ([#560](https://github.com/opensearch-project/opensearch-py-ml/pull/560))
3636
- Upgrade torch version to fix security issue ([#561](https://github.com/opensearch-project/opensearch-py-ml/pull/561))
3737
- Upgrade numpy and python version ([#562](https://github.com/opensearch-project/opensearch-py-ml/pull/562))
38+
- Update space type mapping for sentence transformer models ([#574](https://github.com/opensearch-project/opensearch-py-ml/pull/574))
3839

3940
### Fixed
4041
- Fix for uploading models with function_name instead of model_task ([#553](https://github.com/opensearch-project/opensearch-py-ml/pull/553))

opensearch_py_ml/ml_models/sentencetransformermodel.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -61,10 +61,10 @@ class SentenceTransformerModel(BaseUploadModel):
6161
"msmarco-distilbert-base-tas-b": "innerproduct",
6262
"multi-qa-MiniLM-L6-cos-v1": "l2",
6363
"multi-qa-mpnet-base-dot-v1": "innerproduct",
64-
"paraphrase-MiniLM-L3-v2": "cosine",
65-
"paraphrase-multilingual-MiniLM-L12-v2": "cosine",
66-
"paraphrase-mpnet-base-v2": "cosine",
67-
"distiluse-base-multilingual-cased-v1": "cosine",
64+
"paraphrase-MiniLM-L3-v2": "cosinesimil",
65+
"paraphrase-multilingual-MiniLM-L12-v2": "cosinesimil",
66+
"paraphrase-mpnet-base-v2": "cosinesimil",
67+
"distiluse-base-multilingual-cased-v1": "cosinesimil",
6868
}
6969

7070
def __init__(

tests/ml_models/test_sentencetransformermodel_pytest.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -276,7 +276,7 @@ def test_make_model_config_json_for_onnx():
276276
"embedding_dimension": 384,
277277
"pooling_mode": "MEAN",
278278
"normalize_result": False,
279-
"additional_config": {"space_type": "cosine"},
279+
"additional_config": {"space_type": "cosinesimil"},
280280
}
281281

282282
clean_test_folder(TEST_FOLDER)
@@ -313,7 +313,7 @@ def test_overwrite_fields_in_model_config():
313313
"embedding_dimension": 128,
314314
"pooling_mode": "MAX",
315315
"normalize_result": False,
316-
"additional_config": {"space_type": "cosine"},
316+
"additional_config": {"space_type": "cosinesimil"},
317317
}
318318

319319
clean_test_folder(TEST_FOLDER)

0 commit comments

Comments
 (0)