Skip to content

feat(go): add custom configs for all primitives #2883

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 15 commits into from
Jul 8, 2025

Conversation

sahdev77
Copy link
Contributor

@sahdev77 sahdev77 commented May 7, 2025

Addition and handling of custom config for Embedder and Retriever for all the plugins
Bug: #2444
Checklist (if applicable):

Page not found · GitHub · GitHub
Skip to content
404 “This is not the web page you are looking for”
Copy link
Collaborator

@apascal07 apascal07 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you show usage here? I want to see how you would call Embed() with custom options for example.

@sahdev77 sahdev77 requested a review from apascal07 June 4, 2025 19:40
@sahdev77 sahdev77 marked this pull request as ready for review June 4, 2025 19:41
@apascal07
Copy link
Collaborator

Please re-request review when you are done making changes so this ends up in my inbox.

@sahdev77 sahdev77 dismissed apascal07’s stale review June 27, 2025 11:41

sample app have the usage of config of embedder

@sahdev77 sahdev77 requested a review from apascal07 June 27, 2025 12:21
@apascal07 apascal07 changed the title feat(go):Support custom configs for all primitives feat(go): add custom configs for all primitives Jul 8, 2025
@apascal07 apascal07 enabled auto-merge (squash) July 8, 2025 10:03
@apascal07 apascal07 merged commit 2b84da4 into main Jul 8, 2025
7 checks passed
@apascal07 apascal07 deleted the sahdev/support-custom-config branch July 8, 2025 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants