Skip to content

Conversation

gastoner
Copy link
Contributor

@gastoner gastoner commented Mar 6, 2025

What does this PR do?

Adds Graph Rag chat application to AI Lab recipes

Screenshot / video of UI

image

What issues does this PR fix or reference?

Closes #2629

How to test this PR?

@gastoner
Copy link
Contributor Author

gastoner commented Mar 7, 2025

@jeffmaury Should I wait with this PR until is containers/ai-lab-recipes#801 merged? Also I need to update the readme (I've just copied one from different RAG), but is that something that we extract from recipes/natural_language_processing/graph_rag/README.md?

@jeffmaury
Copy link
Collaborator

@jeffmaury Should I wait with this PR until is containers/ai-lab-recipes#801 merged? Also I need to update the readme (I've just copied one from different RAG), but is that something that we extract from recipes/natural_language_processing/graph_rag/README.md?

Yes recipe should be merged. And yes this is copy of the recipe README.md

@gastoner gastoner force-pushed the add_graph_rag_recipe_to_catalog branch 2 times, most recently from 59938df to 8b21a0a Compare March 17, 2025 12:13
@gastoner gastoner marked this pull request as ready for review March 17, 2025 12:21
@gastoner gastoner requested review from benoitf, jeffmaury and a team as code owners March 17, 2025 12:21
@gastoner gastoner requested review from dgolovin and axel7083 March 17, 2025 12:21
@gastoner gastoner force-pushed the add_graph_rag_recipe_to_catalog branch from 8b21a0a to 2d124ec Compare March 17, 2025 12:22
"description": "This demo provides a recipe to build out a custom Graph RAG (Graph Retrieval Augmented Generation) application using the repo LightRag which abstracts Microsoft's GraphRag implementation. It consists of two main components; the Model Service, and the AI Application with a built in Database.",
"name": "Graph RAG Chat Application",
"repository": "https://github.com/containers/ai-lab-recipes",
"ref": "v1.5.0",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should be main or a commit id as iy is not in the 1.5.0 release

Copy link
Contributor Author

@gastoner gastoner Mar 17, 2025

Choose a reason for hiding this comment

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

Using latest commit id 53def1030a84ad3dc07deee6e02d905efd5e6d59 now

@gastoner gastoner requested a review from jeffmaury March 17, 2025 12:41
Copy link
Collaborator

@jeffmaury jeffmaury left a comment

Choose a reason for hiding this comment

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

LGTM with the proposed change

"recommended": ["hf.instructlab.granite-7b-lab-GGUF"],
"backend": "llama-cpp",
"languages": ["python"],
"frameworks": ["streamlit", "langchain"]
Copy link
Collaborator

@jeffmaury jeffmaury Mar 17, 2025

Choose a reason for hiding this comment

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

Suggested change
"frameworks": ["streamlit", "langchain"]
"frameworks": ["streamlit", "lightrag"]

@gastoner gastoner force-pushed the add_graph_rag_recipe_to_catalog branch from c3fa58d to 168113d Compare March 17, 2025 14:58
"recommended": ["hf.instructlab.granite-7b-lab-GGUF"],
"backend": "llama-cpp",
"languages": ["python"],
"frameworks": ["streamlit", "lighrag"]
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"frameworks": ["streamlit", "lighrag"]
"frameworks": ["streamlit", "lightrag"]

Signed-off-by: Evzen Gasta <[email protected]>
@gastoner gastoner force-pushed the add_graph_rag_recipe_to_catalog branch from 168113d to f668134 Compare March 18, 2025 10:52
@gastoner gastoner requested a review from feloy March 18, 2025 10:52
Copy link
Contributor

@feloy feloy left a comment

Choose a reason for hiding this comment

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

app works correctly, tested on mac (applehv)

@jeffmaury jeffmaury merged commit 0be8694 into containers:main Mar 18, 2025
8 checks passed
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.

Add GraphRag recipe to the catalog
3 participants