-
Notifications
You must be signed in to change notification settings - Fork 82
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
neo4j GraphRag implementation group 17 #45
Conversation
…e, can be accessed by the server running on google colab to store the graph
…ns that the crud operations can be written in the flask app and after the server hosted on google colab can be updated to use these database endpoints. Note: before running the Flask app, do not forget to start the neo4j server on your desktop app.
Merging updates from Erik
…minder vector docs toevoegd als er meer graph docs gevonden zijn
…ever te laten werken. 2 toevoegingen: een schema extractor en een query runner
…e document, amount of other documents is limited by the length of the results form graph_db, based on the env variable chunk_size
Begin van graph retrieval README deel
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.
Please fix the comments so I can merge your PR
neo4j/.env.template
Outdated
@@ -0,0 +1,4 @@ | |||
ngrok_token='Your_ngrok_token' |
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.
Could you remove this?
README.md
Outdated
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.
Can you remove this file from the PR? And/or make a section on just how GraphRAG works.
neo4j/requirements.txt
Outdated
@@ -0,0 +1,3 @@ | |||
neo4j | |||
flask | |||
pyngrok |
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.
Can you remove this?
server/.env.template
Outdated
@@ -76,8 +80,9 @@ number_of_chunks=None | |||
|
|||
use_openai=False | |||
openai_model_name='gpt-4o-mini' | |||
use_gemini=False | |||
use_gemini=True |
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.
Can you undo this?
server/RAGHelper.py
Outdated
from langchain_core.documents.base import Document | ||
from langchain_experimental.text_splitter import SemanticChunker | ||
from langchain_milvus.vectorstores import Milvus | ||
from langchain_postgres.vectorstores import PGVector | ||
#from langchain_postgres.vectorstores import PGVector |
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.
Can you undo this?
server/RAGHelper_cloud.py
Outdated
@@ -126,13 +133,45 @@ def handle_rewrite(self, user_query: str) -> str: | |||
""" | |||
if os.getenv("use_rewrite_loop") == "True": | |||
response = self.rewrite_ask_chain.invoke(user_query) | |||
self.logger.info(f"The response of the rewrite loop is - {response}") | |||
# self.logger.info(f"The response of the rewrite loop is - {response}") |
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.
Undo
server/data/example.txt
Outdated
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.
Can you remove this file?
We still have conflicts in: |
No description provided.