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.

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

Graph rag #1007

Merged
merged 29 commits into from
Oct 30, 2024
Merged

Graph rag #1007

merged 29 commits into from
Oct 30, 2024
+5,171 −0

Conversation

rbrugaro
Copy link
Contributor

Description

GrapgRAG implementation with llama-index

Depends on:
opea-project/GenAIComps#793 for graph extraction and neo4j retriever microservices to get merged
Waiting @Spycsh to merge orchestrator patch supporting ChatCompletionRequest input. Changed validated locally.

Will update Dockerfile.without_rerank to clone from OPEA instead of my fork once all components are merged

Tests

./test_compose_without_rerank_on_gaudi.sh.

rbrugaro and others added 3 commits October 22, 2024 06:46
@rbrugaro
graphrag backend and compose
9744977 
Signed-off-by: Rita Brugarolas <rita.brugarolas.brufau@intel.com>
@rbrugaro
graphrag backend wrking and add test code
c9b7028 
Signed-off-by: Rita Brugarolas <rita.brugarolas.brufau@intel.com>
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
045550c 
for more information, see https://pre-commit.ci
@Spycsh Spycsh mentioned this pull request Oct 23, 2024
Merged
4 tasks
rbrugaro and others added 7 commits October 22, 2024 21:18
@rbrugaro
Merge branch 'opea-project:main' into graphRAG
31acdb3
@rbrugaro
code cleanup and test_compose validated
99197ea 
Signed-off-by: Rita Brugarolas <rita.brugarolas.brufau@intel.com>
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
50c594d 
for more information, see https://pre-commit.ci
@rbrugaro
add README
d8f791c 
Signed-off-by: Rita Brugarolas <rita.brugarolas.brufau@intel.com>
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
ca3b0a0 
for more information, see https://pre-commit.ci
@rbrugaro
README fixes
4f07a4c 
Signed-off-by: Rita Brugarolas <rita.brugarolas.brufau@intel.com>
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
3c2b11f 
for more information, see https://pre-commit.ci
@rbrugaro rbrugaro marked this pull request as ready for review October 23, 2024 21:31
rbrugaro and others added 10 commits October 23, 2024 22:16
@rbrugaro
readme relative path fix
b894a84 
Signed-off-by: Rita Brugarolas <rita.brugarolas.brufau@intel.com>
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
623693c 
for more information, see https://pre-commit.ci
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
dfb89f7 
for more information, see https://pre-commit.ci
@rbrugaro
marmaid update
4021fb4 
Signed-off-by: Rita Brugarolas <rita.brugarolas.brufau@intel.com>
@rbrugaro
marmaid update
986b74b 
Signed-off-by: Rita Brugarolas <rita.brugarolas.brufau@intel.com>
@rbrugaro
Merge branch 'graphRAG' of https://github.com/rbrugaro/GenAIExamples …
6ec03a9 
…into graphRAG
@rbrugaro
marmaid update
43d6adc 
Signed-off-by: Rita Brugarolas <rita.brugarolas.brufau@intel.com>
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
66eff31 
for more information, see https://pre-commit.ci
@rbrugaro
fix readme link
493e681 
Signed-off-by: Rita Brugarolas <rita.brugarolas.brufau@intel.com>
@rbrugaro
fix readme cicd error
b0c774b 
Signed-off-by: Rita Brugarolas <rita.brugarolas.brufau@intel.com>
@rbrugaro rbrugaro marked this pull request as draft October 24, 2024 05:49
@rbrugaro rbrugaro marked this pull request as ready for review October 24, 2024 06:35
@rbrugaro
update comp image names
8cbf08c 
Signed-off-by: Rita Brugarolas <rita.brugarolas.brufau@intel.com>
@rbrugaro
Copy link
Contributor Author

  • Need to update git clone in dockerfile once the GenAIComps are merged

@XuhuiRen XuhuiRen mentioned this pull request Oct 25, 2024
Closed
@rbrugaro
add UI folder, install docker script and renabled frontend from test …
110bb6d 
…script

Signed-off-by: Rita Brugarolas <rita.brugarolas.brufau@intel.com>
@rbrugaro
added missing assets for UI
3cd7179 
Signed-off-by: Rita Brugarolas <rita.brugarolas.brufau@intel.com>
@rbrugaro rbrugaro mentioned this pull request Oct 28, 2024
Open
4 tasks
ashahba
ashahba requested changes Oct 29, 2024
View reviewed changes
Copy link
Collaborator

@ashahba ashahba left a comment

Choose a reason for hiding this comment

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

Thanks @rbrugaro for this PR!
My change requests are mostly general BKMs and in some cases for consistency purpose.

Thanks.

GraphRAG/Dockerfile Outdated Show resolved Hide resolved
GraphRAG/Dockerfile Outdated Show resolved Hide resolved
GraphRAG/Dockerfile Outdated Show resolved Hide resolved
GraphRAG/README.md Show resolved Hide resolved
GraphRAG/README.md Show resolved Hide resolved
GraphRAG/ui/svelte/src/routes/+layout.svelte Outdated Show resolved Hide resolved
GraphRAG/ui/svelte/src/routes/+page.svelte Outdated Show resolved Hide resolved
GraphRAG/ui/svelte/src/routes/+page.ts Outdated Show resolved Hide resolved
GraphRAG/ui/svelte/tailwind.config.cjs Outdated Show resolved Hide resolved
GraphRAG/ui/svelte/vite.config.ts Outdated Show resolved Hide resolved
rbrugaro and others added 5 commits October 29, 2024 21:59
@rbrugaro
PR bkm feedback minor fixes
f74e4be 
Signed-off-by: Rita Brugarolas <rita.brugarolas.brufau@intel.com>
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
3f256b9 
for more information, see https://pre-commit.ci
@rbrugaro
change component references to main branch instead of fork and minor …
2b78b04 
…readability fixes

Signed-off-by: Rita Brugarolas <rita.brugarolas.brufau@intel.com>
@rbrugaro
fix repo link
52b7b80 
Signed-off-by: Rita Brugarolas <rita.brugarolas.brufau@intel.com>
@rbrugaro
Merge branch 'main' into graphRAG
edb3076
@rbrugaro
updated license in svelte UI
d36b280 
Signed-off-by: Rita Brugarolas <rita.brugarolas.brufau@intel.com>
ashahba
ashahba approved these changes Oct 30, 2024
View reviewed changes
Copy link
Collaborator

@ashahba ashahba left a comment

Choose a reason for hiding this comment

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

LGTM!

lkk12014402
lkk12014402 approved these changes Oct 30, 2024
View reviewed changes
Copy link
Collaborator

@lkk12014402 lkk12014402 left a comment

Choose a reason for hiding this comment

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

LGTM

@ashahba ashahba merged commit a65640b into opea-project:main Oct 30, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ichbinblau ichbinblau ichbinblau left review comments

@ashahba ashahba ashahba approved these changes

@lkk12014402 lkk12014402 lkk12014402 approved these changes

@XuhuiRen XuhuiRen XuhuiRen approved these changes

@lvliang-intel lvliang-intel Awaiting requested review from lvliang-intel lvliang-intel is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@rbrugaro @ichbinblau @ashahba @lkk12014402 @XuhuiRen