Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update rag_llamaindex_librarian.ipynb #60

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

javapapo
Copy link

@javapapo javapapo commented Mar 9, 2024

Add a small change - make sure that the reader has pulled llama2 prior to running the example.

What does this PR do?

Add a small change - make sure that the reader has pulled llama2 prior to running the example.

Who can review?

@jinnovation

Add a small change - make sure that the reader has pulled llama2 prior to running the example.
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@@ -242,7 +242,7 @@
"\n",
"First let's start up the Ollama server. Unfortunately, there is no support in the [Ollama Python client](https://github.com/ollama/ollama-python) for actually starting and stopping the server itself, so we'll have to pop out of Python land for this.\n",
"\n",
"In a separate terminal, run: `ollama serve`. Remember to terminate this after we're done here!"
"In a separate terminal, run: 'ollama pull llama2' and then`ollama serve`. Remember to terminate this after we're done here!"
Copy link
Contributor

Choose a reason for hiding this comment

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

Good catch. However, I think you actually need to have ollama serve running before running pull.

> ollama pull zephyr
Error: could not connect to ollama app, is it running?

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants