llama_index/llama-index-core/llama

At a glance

The community member is seeking help with an issue related to the KnowledgeGraphQueryEngine in the Llama Index library. They have noticed that the graph_query_synthesis_prompt parameter is optional in one part of the code, but mandatory in another part, which is causing an error when they try to run a query.

Another community member responds that this issue was introduced during a refactor in version 0.10.x, and that the newer property graph index feature may make this obsolete. They suggest the original poster look into the property graph index as a potential solution.

Useful resources

lllm_dev

Hey guys, any help on KG pls

In the given line for KnowledgeGraphQueryEngine, the graph_query_synthesis_prompt is an optional paramter
https://github.com/run-llama/llama_index/blob/01e5173f8a272e8b7e5ccb2ae3ff215eb6c4ca6a/llama-index-core/llama_index/core/query_engine/knowledge_graph_query_engine.py#L70

but at line 132
https://github.com/run-llama/llama_index/blob/01e5173f8a272e8b7e5ccb2ae3ff215eb6c4ca6a/llama-index-core/llama_index/core/query_engine/knowledge_graph_query_engine.py#L132

its a mandatory argument for _llm.predict, I'm seeing an error, when i run query_engine.query("sample query ?")

Any help here pls

2 comments

LLogan M

Yea during the v0.10.x refactor, some of the text2sql prompts disappeared 😓 Been slightly delaying fixing this because the newer property graph index kind of makes this obsolete

lllm_dev

ahhh i see, property graph, thx again @Logan M appreciate the help as always. I'll look into property graph. thx for the tip.

Add a reply

Find answers from the community

llama_index/llama-index-core/llama_index...