-
-
Notifications
You must be signed in to change notification settings - Fork 632
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
ollama: use official client #1036
base: main
Are you sure you want to change the base?
Conversation
de38692
to
c61a01c
Compare
c61a01c
to
727a7aa
Compare
727a7aa
to
41fb545
Compare
I look forward to it, and I can use it together, so when can I use it? |
There's a problem with this PR with null option values (0, "", etc.). For now I see two ways to fix this in refactoring the generic llms package:
To move forward and quickly propose a solution I go for 2. but I'm open to discuss and modify this as I'm not fully confident with this change. @pdxrlj you can try this PR code if you want, feel free to give feedback on it |
Close #1035
Conflict #1022
Need to see if it fix #961
PR Checklist
memory: add interfaces for X, Y
orutil: add whizzbang helpers
).Fixes #123
).golangci-lint
checks.