Skip to content

Conversation

@cyrannano
Copy link
Contributor

Description

This PR addresses a critical issue with the import order within the mindsql.llms module, where a bad import ordering led to a circular import error in Python. The ILlm import has been reorganized across the module to resolve this issue, ensuring smooth operation and compliance with PEP 8 standards.

Related Issue

N/A

Proposed Changes

  • Reordered the ILlm import in init.py to the top of the local imports to prevent circular import errors and improve code readability.
  • Updated relative imports of ILlm in anthropic.py, googlegenai.py, and open_ai.py to explicit import paths. This change clarifies the import structure and helps prevent similar circular import issues in the future.

@Sammindinventory Sammindinventory merged commit 8c6a90a into Mindinventory:master Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants