-
Notifications
You must be signed in to change notification settings - Fork 126
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 Ollama Generator #146
Comments
Features to add to this integration in the future (taking notes here, but should open issues):
|
@AlistairLR112 we can continue working on this integration, using the checklist above. You can choose what you want:
Of course, if you don't have time, I'll take care of it... FYI @bilgeyucel @dfokina |
Am happy to take it on :) |
Example started here: #170 |
Tile added: deepset-ai/haystack-integrations#108 |
Docs are published: https://docs.haystack.deepset.ai/v2.0/docs/ollamagenerator |
Hello everyone, one thing I have noticed we need to add in the docs and the example is the mapping of the volumes in docker - this has already been done in the CI pipeline We need to add It won't affect any functionality Needs to be added to the following: |
@AlistairLR112 please correct me if I am wrong. |
Ah whoops, you are right, was thinking it was mapping the entrypoint! False alarm!!! |
Opened a PR in Ollama to mention the integration in their README: ollama/ollama#1808 |
Why Can you fix that? |
Summary and motivation
See deepset-ai/haystack#6514
Detailed design
See deepset-ai/haystack#6514
Checklist
If the request is accepted, ensure the following checklist is complete before closing this issue.
main
branchintegration:<your integration name>
has been added to this repoThe text was updated successfully, but these errors were encountered: