Skip to content

feat(llm):support graph checking before updating vid embedding#205

Merged
imbajin merged 13 commits intoapache:mainfrom
MrJs133:fix_update_vids
Apr 7, 2025
Merged

feat(llm):support graph checking before updating vid embedding#205
imbajin merged 13 commits intoapache:mainfrom
MrJs133:fix_update_vids

Conversation

@MrJs133
Copy link
Copy Markdown
Contributor

@MrJs133 MrJs133 commented Apr 4, 2025

The vertex embedding is updated every hour.

When a new user starts the program for the first time, if the graph database related information is not configured, many errors will occur, which affects the user experience.

Therefore, before updating the vertex embedding, it will check whether the current graph link is reachable.

When the link is unreachable, the effect is as follows:

image

@dosubot dosubot Bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Apr 4, 2025
@github-actions github-actions Bot added the llm label Apr 4, 2025
@dosubot dosubot Bot added the enhancement New feature or request label Apr 4, 2025
@imbajin imbajin requested a review from Copilot April 7, 2025 08:21
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

hugegraph-llm/src/hugegraph_llm/demo/rag_demo/vector_graph_block.py:161

  • [nitpick] Consider renaming the parameter 'pwd' to 'password' for improved clarity and consistency.
"pwd": huge_settings.graph_pwd,

Comment thread hugegraph-llm/src/hugegraph_llm/utils/hugegraph_utils.py Outdated
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Apr 7, 2025
@imbajin imbajin merged commit 36d5444 into apache:main Apr 7, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request lgtm This PR has been approved by a maintainer llm size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants