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

Support text2gql search for GraphRAG #2227

Merged
merged 31 commits into from
Jan 22, 2025

Conversation

SonglinLyu
Copy link
Contributor

@SonglinLyu SonglinLyu commented Dec 20, 2024

Fix #2195

Description

Support translating natural language questions directly to a cypher query with a prompt based tex2gql translator on GraphRAG.

How to use

set TEXT2GQL_SEARCH_ENABLED=True in .env
False for default

@csunny
Copy link
Collaborator

csunny commented Dec 23, 2024

Cool~

@csunny
Copy link
Collaborator

csunny commented Dec 26, 2024

@SonglinLyu Please fix the format error and add TEXT2GQL_SEARCH_ENABLED=True to env.template

…/DB-GPT into graphrag_text2gql_dev

Conflicts:
	dbgpt/storage/knowledge_graph/community/tugraph_store_adapter.py
	dbgpt/storage/knowledge_graph/community_summary.py
@SonglinLyu SonglinLyu closed this Jan 14, 2025
@SonglinLyu SonglinLyu reopened this Jan 14, 2025
Copy link
Contributor

@Appointat Appointat left a comment

Choose a reason for hiding this comment

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

I left some comments

dbgpt/rag/transformer/text2cypher.py Outdated Show resolved Hide resolved
dbgpt/rag/transformer/awel_intent_interpreter.py Outdated Show resolved Hide resolved
dbgpt/rag/transformer/base.py Outdated Show resolved Hide resolved
dbgpt/rag/transformer/intent_interpreter.py Outdated Show resolved Hide resolved
dbgpt/rag/transformer/intent_interpreter.py Outdated Show resolved Hide resolved
dbgpt/rag/transformer/text2cypher.py Outdated Show resolved Hide resolved
dbgpt/rag/transformer/text2cypher.py Outdated Show resolved Hide resolved
dbgpt/rag/transformer/text2cypher.py Outdated Show resolved Hide resolved
Copy link
Contributor Author

@SonglinLyu SonglinLyu left a comment

Choose a reason for hiding this comment

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

reply for some of the comments

dbgpt/rag/transformer/awel_intent_interpreter.py Outdated Show resolved Hide resolved
dbgpt/rag/transformer/intent_interpreter.py Outdated Show resolved Hide resolved
Appointat

This comment was marked as duplicate.

Appointat

This comment was marked as duplicate.

Copy link
Contributor

@Appointat Appointat left a comment

Choose a reason for hiding this comment

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

Left some comments

dbgpt/storage/knowledge_graph/community_summary.py Outdated Show resolved Hide resolved
dbgpt/storage/knowledge_graph/community_summary.py Outdated Show resolved Hide resolved
dbgpt/storage/knowledge_graph/community_summary.py Outdated Show resolved Hide resolved
Copy link
Contributor

@Appointat Appointat left a comment

Choose a reason for hiding this comment

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

left some comments

dbgpt/storage/knowledge_graph/community_summary.py Outdated Show resolved Hide resolved
dbgpt/storage/knowledge_graph/community_summary.py Outdated Show resolved Hide resolved
dbgpt/storage/knowledge_graph/community_summary.py Outdated Show resolved Hide resolved
dbgpt/storage/knowledge_graph/community_summary.py Outdated Show resolved Hide resolved
dbgpt/storage/graph_store/tugraph_store.py Outdated Show resolved Hide resolved
dbgpt/rag/transformer/intent_interpreter.py Outdated Show resolved Hide resolved
.env.template Outdated Show resolved Hide resolved
@Aries-ckt
Copy link
Collaborator

try graph_store_apdater -> graph_store_adapter ?

Copy link
Collaborator

@Aries-ckt Aries-ckt left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@fanzhidongyzby fanzhidongyzby left a comment

Choose a reason for hiding this comment

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

r+

@Aries-ckt Aries-ckt merged commit e0081e6 into eosphoros-ai:main Jan 22, 2025
7 checks passed
@fanzhidongyzby fanzhidongyzby added the GraphRAG Module: GraphRAG label Jan 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GraphRAG Module: GraphRAG
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature][GraphRAG] Enhance GraphRAG search by Text2GQL model
5 participants