Skip to content
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

Merged
merged 43 commits into from
Dec 23, 2024
Merged

Conversation

Tjielke
Copy link
Contributor

@Tjielke Tjielke commented Dec 6, 2024

No description provided.

TNabuurs and others added 30 commits November 21, 2024 20:58
…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
Copy link
Contributor

@ErikTromp ErikTromp left a 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

@@ -0,0 +1,4 @@
ngrok_token='Your_ngrok_token'
Copy link
Contributor

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
Copy link
Contributor

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.

@@ -0,0 +1,3 @@
neo4j
flask
pyngrok
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you remove this?

@@ -76,8 +80,9 @@ number_of_chunks=None

use_openai=False
openai_model_name='gpt-4o-mini'
use_gemini=False
use_gemini=True
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you undo this?

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
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you undo this?

@@ -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}")
Copy link
Contributor

Choose a reason for hiding this comment

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

Undo

Copy link
Contributor

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?

@ErikTromp
Copy link
Contributor

We still have conflicts in:
server/.env.template
server/RAGHelper.py

@ErikTromp ErikTromp merged commit 18dbf27 into AI-Commandos:main Dec 23, 2024
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.

5 participants