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.

Sign up for GitHub

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

Jump to bottom

feat: KnowledgeGraph builder #34

Merged

kmehant merged 23 commits into c2siorg:main from debrupf2946:dev

Jul 5, 2024

+3,920 −0

Collaborator

debrupf2946 commented Jul 2, 2024 •

edited

Loading

The GraphRag folder contains codes that allow users to build a Knowledge Graph from documents.


          added GraphRag folder that contain codes that allows users to build K…

b94f185

…nowledge Graph from Documents

debrupf2946 changed the title ~~added GraphRag~~ feat: KnowledgeGraph builder

Member

kmehant commented Jul 2, 2024

@debrupf2946 The README is poorly written with scant attention to grammar and formatting please update it.

kmehant requested changes

View reviewed changes

Member

kmehant left a comment

Rename the folder GraphRag to snake case and give it informative naming such graph_rag_using_llamaindex etc

kmehant reviewed

View reviewed changes

GraphRag/README.MD Outdated Show resolved Hide resolved

kmehant reviewed

View reviewed changes

GraphRag/knowledgeGraph.py Outdated Show resolved Hide resolved

kmehant reviewed

View reviewed changes

GraphRag/knowledgeGraph.py Outdated Show resolved Hide resolved

kmehant reviewed

View reviewed changes

GraphRag/knowledgeGraph.py Outdated Show resolved Hide resolved

kmehant reviewed

View reviewed changes

GraphRag/requirements.txt Outdated Show resolved Hide resolved


          Udated README.MD

d52b69c

kmehant reviewed

View reviewed changes

GraphRag/tools.py Outdated Show resolved Hide resolved

kmehant requested changes

View reviewed changes

Member

kmehant left a comment

Require more changes.

Collaborator Author

debrupf2946 commented Jul 2, 2024

@debrupf2946 The README is poorly written with scant attention to grammar and formatting please update it.
Thank you for the review
I have updated the README.MD

kmehant requested changes

View reviewed changes

Member

kmehant left a comment

Update your PR with more information description on the tooling and approach you have taken in this PR. Add screenshots of examples that you have tried with this contribution.

kmehant requested changes

View reviewed changes

Member

kmehant left a comment

update on review

debrupf2946 added 4 commits

July 4, 2024 00:32


          New colab file that shows implimentation of GraphRag on keras-nlp rea…

7c89fa4

…dme file


          formatted code with black

8ea8a6d


          updated save_index function in KnowledgeGraph.py by taking output_dir…

bb16353

…_path as a parameter,also formatted collab using black


          added docstrings to each file to describe its content

790b70f

kmehant reviewed

View reviewed changes

GraphRag/README.MD Outdated Show resolved Hide resolved

kmehant requested changes

View reviewed changes

Member

kmehant left a comment

move all this to graph_rag/graph_builder folder instead of GraphRag look at my other comments that I have added now. Thanks

GraphRag/requirements.txt Outdated Show resolved Hide resolved


          changed and renamed GraphRag to graph_rag and moved all files to grap…

d4b105a

…h_rag/graph_builder

Collaborator Author

debrupf2946 commented Jul 4, 2024

graph_builder

Refactored codes to graph_rag/graph_builder

debrupf2946 added 4 commits

July 5, 2024 00:04


          added exception handling to the codes

e0e5331


          added reffernce of example note-book in README.MD

e9790e3


          fixed reffernce bug of example note-book in README.MD

e3b14ac


          updated new path reference in example note-book

b2e76fa

kmehant self-requested a review

July 5, 2024 04:29

kmehant approved these changes

View reviewed changes

Member

kmehant left a comment

LGTM

Member

kmehant commented Jul 5, 2024

@debrupf2946 update the PR with signed commit and we are good to go.

debrupf2946 added 12 commits

July 5, 2024 17:10


          signed and added GraphRag folder that contain codes that allows users…

28df289

… to build Knowledge Graph from Documents


          signed dated README.MD

aa64569


          signed colab file that shows implimentation of GraphRag on keras-nlp …

cc60870

…readme file


          signed formatted code with black

79fff3a


          signed updated save_index function in KnowledgeGraph.py by taking out…

…put_dir_path as a parameter,also formatted collab using black


          signed added docstrings to each file to describe its content

4acd239


          signed changed and renamed GraphRag to graph_rag and moved all files …

4fa89ae

…to graph_rag/graph_builder


          signed added exception handling to the codes

12d2e04


          signed added reffernce of example note-book in README.MD

80e9a40


          signed fixed reffernce bug of example note-book in README.MD

59ebcc9


          signed updated new path reference in example note-book

458e036


          Merge branch 'test' into dev

d4763ef

kmehant merged commit daa3a95 into c2siorg:main

1 check passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet