Skip to content

Commit 841d1ad

Browse files
authored
Merge pull request #1298 from tisnik/lcore-1438-use-removeprefix-method
LCORE-1438: Use removeprefix method
2 parents b1a6cf3 + 8e08ec1 commit 841d1ad

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

src/llama_stack_configuration.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -251,8 +251,7 @@ def construct_models_section(
251251

252252
# Strip sentence-transformers/ prefix if present
253253
provider_model_id = embedding_model
254-
if provider_model_id.startswith("sentence-transformers/"):
255-
provider_model_id = provider_model_id[len("sentence-transformers/") :]
254+
provider_model_id = provider_model_id.removeprefix("sentence-transformers/")
256255

257256
# Skip if embedding model already registered
258257
existing_model_ids = [m.get("provider_model_id") for m in output]
@@ -489,8 +488,7 @@ def enrich_solr(ls_config: dict[str, Any], solr_config: dict[str, Any]) -> None:
489488

490489
# Strip sentence-transformers/ prefix from constant for provider_model_id
491490
provider_model_id = constants.SOLR_DEFAULT_EMBEDDING_MODEL
492-
if provider_model_id.startswith("sentence-transformers/"):
493-
provider_model_id = provider_model_id[len("sentence-transformers/") :]
491+
provider_model_id = provider_model_id.removeprefix("sentence-transformers/")
494492

495493
# Check if already registered
496494
registered_models = ls_config["registered_resources"]["models"]

src/utils/suid.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ def check_suid(suid: str) -> bool:
4040
return False
4141

4242
# Strip 'conv_' prefix if present
43-
hex_part = suid[5:] if suid.startswith("conv_") else suid
43+
hex_part = suid.removeprefix("conv_")
4444

4545
# Check for 48-char hex string (llama-stack conversation ID format)
4646
if len(hex_part) == 48:

0 commit comments

Comments
 (0)