8000 Sim 2873 ollama conversations and get actions working with simpler context and json mode by frankcarey · Pull Request #23 · fablestudio/thistle-gulch · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Sim 2873 ollama conversations and get actions working with simpler context and json mode #23

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 8 commits into from
Mar 29, 2024

Conversation

frankcarey
Copy link
Contributor

Screenshot 2024-03-29 004305

* Mainly, we are overriding endpoints now.
…alid back from it.

* Main trick seems to be using the  flag, which works similar to openAI. This isn't
  a LangChain thing, this is part of the Ollama API. I'm not sure what they are doing
  under the hood, but it works a hell of a lot better.
* Conversations started working out of the box pretty much, but Actions generation
  still struggled to fill the skills without hallucinating. I ended up using the
  context_obj in a custom endpoint that allowed me to simplify the context heavily and
  that seems to be enough for it to basically work.
@frankcarey frankcarey changed the title Sim 2873 ollama conversations Sim 2873 ollama conversations and get actions working with simpler context and json mode Mar 29, 2024
Copy link
Contributor
@fablechris fablechris left a comment

Choose a reason for hiding this comment

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

LGTM! All the demos are working as expected

@frankcarey frankcarey merged commit 573eef4 into main Mar 29, 2024
1 check passed
@frankcarey frankcarey deleted the SIM-2873-ollama-conversations branch March 29, 2024 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0