Skip to content

Replace bobFacts with menuSuggestion in rag docs#113

Merged
tagboola merged 5 commits into
mainfrom
ta/rag-docs
May 13, 2024
Merged

Replace bobFacts with menuSuggestion in rag docs#113
tagboola merged 5 commits into
mainfrom
ta/rag-docs

Conversation

@tagboola
Copy link
Copy Markdown
Contributor

I punted on completely revamping the docs and just replaced the instances of bob facts with menu examples

Comment thread docs/rag.md
Comment on lines -127 to -130
It uses the local file-based vector similarity retriever
that Genkit provides out-of-the box for simple testing and prototyping (_do not
use in production_)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is duplicated below on line 169.

Copy link
Copy Markdown
Contributor

@MaesterChestnut MaesterChestnut left a comment

Choose a reason for hiding this comment

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

Perfect!

Comment thread docs/rag.md Outdated
@@ -124,10 +124,6 @@ Embedding model support is provided through the following plugins:
The following examples show how you could ingest a collection of PDF documents
into a vector database and retrieve them for use in a flow.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Consider adding a blurb about what the sample(s) do (read in restaurant menus and answer questions about them).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done. PTAL.

@tagboola tagboola merged commit 38c3d6d into main May 13, 2024
@tagboola tagboola deleted the ta/rag-docs branch May 13, 2024 19:28
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.

4 participants