Skip to content

fix(llm): extract_graph_data use wrong method#145

Merged
imbajin merged 1 commit intoapache:mainfrom
MrJs133:fix_extract_graph_data
Dec 26, 2024
Merged

fix(llm): extract_graph_data use wrong method#145
imbajin merged 1 commit intoapache:mainfrom
MrJs133:fix_extract_graph_data

Conversation

@MrJs133
Copy link
Copy Markdown
Contributor

@MrJs133 MrJs133 commented Dec 25, 2024

the InfoExtract corresponding to 'tripls' was incorrectly called; instead, the PropertyGraphExtract corresponding to 'property_graph' should have been called

So, change extract_info(example_prompt, "triples") to extract_info(example_prompt, "property_graph")

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Dec 25, 2024
@github-actions github-actions Bot added the llm label Dec 25, 2024
@dosubot dosubot Bot added the bug Something isn't working label Dec 25, 2024
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Dec 26, 2024
@imbajin imbajin changed the title fix(llm): fix extract_graph_data fix(llm): extract_graph_data use wrong method Dec 26, 2024
@imbajin imbajin merged commit 030fd2e into apache:main Dec 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working lgtm This PR has been approved by a maintainer llm size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants