Initilized Property Graph Index with llm parameter #305

Eric-Shang · 2024-12-27T23:21:44Z

Why are these changes needed?

Property Graph Index use default openai llm internally to create node retriever. We need to pass in llm parameter specifically.

I've included any doc changes needed for https://docs.ag2.ai/. See https://docs.ag2.ai/docs/contributor-guide/documentation to build and test documentation locally.
I've added tests (if relevant) corresponding to the changes introduced in this PR.
I've made sure all auto checks have passed.

qingyun-wu

LGTM! Thanks!

Eric-Shang added 2 commits December 27, 2024 23:15

Initilized Property Graph Index with llm parameter

7299471

Merge branch 'main' into neo4j_fix_llm

199d0ed

Eric-Shang requested a deployment to openai1 December 27, 2024 23:21 — with GitHub Actions Waiting

Eric-Shang requested a deployment to openai1 December 27, 2024 23:22 — with GitHub Actions Waiting

Eric-Shang requested a deployment to openai1 December 27, 2024 23:25 — with GitHub Actions Waiting

qingyun-wu approved these changes Dec 28, 2024

View reviewed changes

qingyun-wu added this pull request to the merge queue Dec 28, 2024

Merged via the queue into main with commit 721ee7a Dec 28, 2024
207 of 215 checks passed

Eric-Shang deleted the neo4j_fix_llm branch January 7, 2025 01:28