neo4j / neo4j-graphrag-python

Neo4j GraphRAG for Python
https://neo4j.com/docs/neo4j-graphrag-python/current/
Other
231 stars 37 forks source link

Fix bug where AzureOpenAIEmbeddings inherits from OpenAIEmbeddings #203

Closed willtai closed 4 weeks ago

willtai commented 4 weeks ago

Description

This PR addresses the bug in the PR https://github.com/neo4j/neo4j-graphrag-python/pull/195 and this issue https://github.com/neo4j/neo4j-graphrag-python/issues/190 . Thank you @brandon-amg for raising this!

In this PR, a separate embed_query method is created for AzureOpenAIEmbeddings and it now inherits from BaseOpenAIEmbeddings, which would avoid the openai.OpenAI client being used.

Type of Change

Complexity

Complexity: Low

How Has This Been Tested?

Checklist

The following requirements should have been met (depending on the changes in the branch):

willtai commented 4 weeks ago

@CodiumAI-Agent /update_changelog

CodiumAI-Agent commented 4 weeks ago

Changelog updates: 🔄

2024-10-24

Fixed

to commit the new content to the CHANGELOG.md file, please type: '/update_changelog --pr_update_changelog.push_changelog_changes=true'