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

Embedding Doesn't Work Without Manually Pulling Embedding Model #337

Open
anaseinea opened this issue Oct 2, 2024 · 2 comments
Open

Embedding Doesn't Work Without Manually Pulling Embedding Model #337

anaseinea opened this issue Oct 2, 2024 · 2 comments
Labels
documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers

Comments

@anaseinea
Copy link

Describe the bug
Embedding Doesn't Work Without Manually Pulling Embedding Model

To Reproduce
start fresh, there will be no embedding model, open the embedding options and click "Embed workspace documents", check ollama logs and you will see a bunch of 404

Expected behavior
To pull the default embedding model or at least warn the user to do so.

API Provider
Ollama

Chat or Auto Complete?
Chat

Desktop (please complete the following information):

  • OS: Windows
@rjmacarthy rjmacarthy added documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers labels Oct 3, 2024
@rjmacarthy
Copy link
Collaborator

It's a good point, but we cannot force a user to add an embedding model or pull models for them when we don't know what provider they are using. Perhaps a warning if it doesn't work would be better?

@anaseinea
Copy link
Author

Yes a warning is good.
There are other things not sure if I should open separate issues for:
1- The embedding model setting in the general settings is useless?
2- I couldn't see a button for toggling the embeddings (on/off button per message)
3- After clicking the embed button, chatting with the model didn't take into consideration the embeddings only when I add @general in the message and that would take a long time (possible re embedding everything?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants