Nishieee commented on code in PR #354:
URL: https://github.com/apache/hugegraph-ai/pull/354#discussion_r3381667831
##########
hugegraph-llm/src/hugegraph_llm/api/models/rag_requests.py:
##########
@@ -56,6 +56,7 @@ class RAGRequest(BaseModel):
extracted from the query, by default only
the most similar one is returned.",
)
client_config: Optional[GraphConfigRequest] = Query(None,
description="hugegraph server config.")
+ include_trace: bool = Query(False, description="Include retrieval
trace/debug info in the response.")
Review Comment:
Agreed on both points. I’ll add gremlin to WkFlowState (+ reset in setup())
and update the regression test to exercise assign_from_json() so real graph
flows preserve trace.gremlin. For include_trace, I’ll narrow the contract to
graph-backed modes and return 400 for unsupported combinations instead of
silently omitting trace.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
