-
Notifications
You must be signed in to change notification settings - Fork 985
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
fix: update getting started guide to use ollama pull
#1855
fix: update getting started guide to use ollama pull
#1855
Conversation
@raghotham @leseb ptal |
ollama pull
ollama pull
@ashwinb @raghotham afaict the docs aren't versioned. i recommend only publishing this change after a release w/ #1854 is cut. |
@mattf please rebase and resolve the conflicts :) Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Holding since #1855 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about we add this in #1854?
This PR made the model names inconsistent - llama3.2:3b-instruct-fp16 vs llama3.2:3b. This broke the tutorial. Also, we should stop pushing docs automatically to readthedocs. This change is now live. |
Good catch, here is a fix #1926 |
What does this PR do?
download the getting started w/ ollama model instead of downloading and running it.
directly running it was necessary before #1854
Test Plan
run the code on the page