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 support for LLaVA through Ollama #152

Merged
merged 5 commits into from
Feb 9, 2024

Conversation

michaelhhogue
Copy link
Collaborator

@michaelhhogue michaelhhogue commented Feb 7, 2024

What does this PR do?

This PR uses the new Ollama Python module to add support for locally hosted vision models. llava will be the first supported local model through Ollama.

Instructions were also added to the README for how to get up and running with LLaVA through Ollama.

Type of change

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

Important

Accuracy when using LLaVA is extremely low. However, this PR serves as a starting point for developers to build off of as local vision models improve.

@michaelhhogue michaelhhogue marked this pull request as ready for review February 7, 2024 01:24
@joshbickett
Copy link
Contributor

joshbickett commented Feb 8, 2024

Just did initial pass. I can see it working, but see what you mention about higher error rates. Not a problem though, this is great progress. I need to get back to some other tasks, but I'll take a final look later and merge it in. @michaelhhogue

@joshbickett joshbickett merged commit 33af25c into OthersideAI:main Feb 9, 2024
@joshbickett
Copy link
Contributor

@michaelhhogue merged, great job! I'm going to play around with improving the prompts and see if I can get LLaVa to perform better. Either way I'll do a Twitter post about it this week or next and mention you, thanks again!

@michaelhhogue
Copy link
Collaborator Author

@joshbickett Sounds good. Let me know if you notice anything that can be improved with the llava support.

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.

None yet

2 participants