Skip to content
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

Add function to Embedding model to get 'default' model #37

Open
tiadams opened this issue Oct 21, 2024 · 0 comments
Open

Add function to Embedding model to get 'default' model #37

tiadams opened this issue Oct 21, 2024 · 0 comments
Assignees
Labels
enhancement New feature or request refactoring Code that needs to be rewritten, without changing functionalities
Milestone

Comments

@tiadams
Copy link
Member

tiadams commented Oct 21, 2024

We need a function to get a default mordel (as for now: MPNet) from the general Embedding model adapter.

why:

  • Currently, we use MPNet as default in several functions - if we want to exchange it we need to adapt all these functions
  • If we have the 'default' model function, we only need to replace it in one class (EmbeddingModel) without having to touch any other implementation
@tiadams tiadams added enhancement New feature or request refactoring Code that needs to be rewritten, without changing functionalities labels Oct 21, 2024
@tiadams tiadams self-assigned this Oct 21, 2024
@tiadams tiadams added this to the v1.0.0 milestone Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request refactoring Code that needs to be rewritten, without changing functionalities
Projects
None yet
Development

No branches or pull requests

1 participant