Skip to content

Support llm embedding prefix and suffix#2

Merged
sukhbinder merged 2 commits intosukhbinder:mainfrom
saksham-jain177:main
Jan 20, 2026
Merged

Support llm embedding prefix and suffix#2
sukhbinder merged 2 commits intosukhbinder:mainfrom
saksham-jain177:main

Conversation

@saksham-jain177
Copy link
Contributor

@saksham-jain177 saksham-jain177 commented Jan 19, 2026

Adds support for optional embedding prefix and suffix provided by llm, applies these values when present while preserving existing behavior for older llm versions

fixes #1

@sukhbinder
Copy link
Owner

@saksham-jain177 Thanks for this. Looks good, can you please add a test for this and also update the readme.md with an example to use this.

@saksham-jain177
Copy link
Contributor Author

Thanks for reviewing and for the feedback
Sure, I’ll add a test for the prefix support and update the README with an example showing how it’s used

@saksham-jain177
Copy link
Contributor Author

Hey, could you please review the latest changes when you get a chance.
I’ve added an integration test covering prefix support and updated the README with a Python usage example.

@sukhbinder sukhbinder merged commit cb1b9e9 into sukhbinder:main Jan 20, 2026
4 checks passed
@sukhbinder
Copy link
Owner

Thanks looks good!

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.

Support llm prefix for embedding models.

2 participants