Skip to content

Fix: Update GraphRAG to be compatible with latest component changes #1384

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

Closed
wants to merge 7 commits into from

Conversation

rbrugaro
Copy link
Collaborator

  • Updated ENV VARS to align with recent changes in neo4j dataprep and retriever.
  • upgraded tgi-gaudi image version

Related to GenAIComps repo issue #1025 (opea-project/GenAIComps#1025)

- Updated ENV VARS to align with recent changes in neo4j dataprep and retriever.
- upgraded tgi-gaudi image version

Related to GenAIComps repo issue opea-project#1025 (opea-project/GenAIComps#1025)

Signed-off-by: rbrygaro <[email protected]>
@rbrugaro rbrugaro requested a review from ashahba as a code owner January 11, 2025 00:32
Copy link

github-actions bot commented Jan 11, 2025

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

@rbrugaro
Copy link
Collaborator Author

requires PR #924 to be merged first

@ashahba ashahba added this to the v1.2 milestone Jan 11, 2025
@ashahba ashahba added the r1.2 OPEA 1.2 RELEASE TAG label Jan 11, 2025
@@ -13,7 +13,7 @@ RUN useradd -m -s /bin/bash user && \
chown -R user /home/user/

WORKDIR /home/user/
RUN git clone https://github.com/opea-project/GenAIComps.git
RUN git clone --branch GRAG_1.2 https://github.com/rbrugaro/GenAIComps.git
Copy link
Collaborator

Choose a reason for hiding this comment

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

This change is for test only, right?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yes, just for test. as soon PR#924 gets merge I will update here.

Copy link
Collaborator

Choose a reason for hiding this comment

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

This PR has been merged, please revert test code.

@letonghan
Copy link
Collaborator

Please resolve the conflicts.

XinyaoWa added a commit to XinyaoWa/GenAIExamples that referenced this pull request Jan 20, 2025
Updated ENV VARS to align with recent changes in neo4j dataprep and retriever.
upgraded tgi-gaudi image version
Related to GenAIComps repo issue opea-project#1025
A re-writed PR for opea-project#1384

Signed-off-by: Xinyao Wang <[email protected]>
XinyaoWa added a commit that referenced this pull request Jan 20, 2025
Update to align with recent changes in neo4j dataprep and retriever.
upgraded tgi-gaudi image version
Related to GenAIComps issue #1025
Related to GenAIComps PR #1408
Based on PR #1384

Signed-off-by: Xinyao Wang <[email protected]>
@XinyaoWa
Copy link
Collaborator

Due to tight schedule for v1.2 release, we move to a new PR for quick merging: #1427

chensuyue pushed a commit that referenced this pull request Jan 20, 2025
- Updated ENV VARS to align with recent changes in neo4j dataprep and retriever.
- upgraded tgi-gaudi image version
Related to GenAIComps repo issue #1025 (opea-project/GenAIComps#1025)

Original PR #1384
Original contributor is @rbrugaro

Signed-off-by: Xinyao Wang <[email protected]>
Co-authored-by: Liang Lv <[email protected]>
@chensuyue chensuyue closed this Jan 20, 2025
chyundunovDatamonsters pushed a commit to chyundunovDatamonsters/OPEA-GenAIExamples that referenced this pull request Mar 4, 2025
…project#1427)

- Updated ENV VARS to align with recent changes in neo4j dataprep and retriever.
- upgraded tgi-gaudi image version
Related to GenAIComps repo issue opea-project#1025 (opea-project/GenAIComps#1025)

Original PR opea-project#1384
Original contributor is @rbrugaro

Signed-off-by: Xinyao Wang <[email protected]>
Co-authored-by: Liang Lv <[email protected]>
Signed-off-by: Chingis Yundunov <[email protected]>
chyundunovDatamonsters pushed a commit to chyundunovDatamonsters/OPEA-GenAIExamples that referenced this pull request Mar 4, 2025
…project#1427)

- Updated ENV VARS to align with recent changes in neo4j dataprep and retriever.
- upgraded tgi-gaudi image version
Related to GenAIComps repo issue opea-project#1025 (opea-project/GenAIComps#1025)

Original PR opea-project#1384
Original contributor is @rbrugaro

Signed-off-by: Xinyao Wang <[email protected]>
Co-authored-by: Liang Lv <[email protected]>
Signed-off-by: Chingis Yundunov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
r1.2 OPEA 1.2 RELEASE TAG
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants