Skip to content

feat(llm): support switch graph in api & add some query configs#184

Merged
imbajin merged 17 commits intoapache:mainfrom
afterimagex:master
Mar 3, 2025
Merged

feat(llm): support switch graph in api & add some query configs#184
imbajin merged 17 commits intoapache:mainfrom
afterimagex:master

Conversation

@afterimagex
Copy link
Copy Markdown
Collaborator

No description provided.

Jin and others added 7 commits January 2, 2025 19:24
Change-Id: I31e9532990c2e7e09ddf518ac9c35cdf996a5a25

doc: update repo & url link for inner repo

Change-Id: Ieac00904c4ab793de87c1cae7324e3d2d7946884

doc: update repo & url link for inner repo (GraphPlatform-4190)

doc: update the link info
sync

Change-Id: Id2138dfecfbd377dd39ec51fa9d46b630e9ea88f
Change-Id: I20c7fd6841df88c50933b4f54bf9bbb316b096a3
@dosubot dosubot Bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Feb 28, 2025
@dosubot dosubot Bot added the documentation Improvements or additions to documentation label Feb 28, 2025
@imbajin imbajin changed the title add api config feat(llm): support switch graph in api & add some query configs Feb 28, 2025
@imbajin imbajin force-pushed the master branch 2 times, most recently from d94f5f5 to c8b6af9 Compare February 28, 2025 06:21
gremlin_prompt: Optional[str] = None,
max_graph_items=30,
topk_return_results=20,
vector_dis_threshold=0.9,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

seems not used in the code?

method=rerank_method,
near_neighbor_first=near_neighbor_first,
custom_related_information=custom_related_information,
topk_return_results=topk_return_results
Copy link
Copy Markdown
Member

@imbajin imbajin Feb 28, 2025

Choose a reason for hiding this comment

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

this param should replace topk below ↓

# class MergeDedupRerank
def __init__(
        self,
        embedding: BaseEmbedding,
        # get from settings & override by params if user specified it ↓
        topk: int = huge_settings.topk_return_results, 
        graph_ratio: float = 0.5,
        method: Literal["bleu", "reranker"] = "bleu",
        near_neighbor_first: bool = False,
        custom_related_information: Optional[str] = None,
        priority: bool = False,  # TODO: implement priority
    ):

imbajin and others added 3 commits February 28, 2025 15:33
Change-Id: Idedc5f1d4fe97f7d9b8adfbfdc82d9aa36dafaa4
Comment thread hugegraph-llm/src/hugegraph_llm/demo/rag_demo/vector_graph_block.py Outdated
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Mar 3, 2025
@imbajin imbajin merged commit 759b963 into apache:main Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation lgtm This PR has been approved by a maintainer llm ml python-client size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants