Simplifying imports for llm_client and cross_encoder
Why?
- It didn’t make sense to expose only
OpenAIClientwhile omittingOpenAIGenericClient,AnthropicClient,GeminiClient, andGroqClient. This led to confusion for users. - A similar issue existed in
cross_encoder, so I addressed that as well. - I noticed related confusion mentioned in #331.
What’s fixed:
- Added the missing clients to
__init__.pyto ensure they’re properly exposed.
Note:
I didn’t open a separate issue to discuss this because I ran into the problem myself while using Graphiti, and it was quicker to submit a PR. Please feel free to close this if there are other considerations I missed.
[!IMPORTANT] Added missing client imports to
__init__.pyinllm_clientandcross_encoderto ensure proper exposure.
- Imports:
- Added
AnthropicClient,GeminiClient,GroqClient, andOpenAIGenericClientto__init__.pyinllm_client.- Added
BGERerankerClientto__init__.pyincross_encoder.- Exports:
- Updated
__all__inllm_client/__init__.pyto include all clients.- Updated
__all__incross_encoder/__init__.pyto includeBGERerankerClient.This description was created by
for 003bae5fe89a421e46674b6769210327a95df796. You can customize this summary. It will automatically update as commits are pushed.
All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.
I have read the CLA Document and I hereby sign the CLA
This has been superseded by #675