type checker cannot see any attributes of retry.py

[kojiromike]

• [x] This is actually a bug report.
• [ ] I am not getting good LLM Results
• [ ] I have tried asking for help in the community on discord or discussions and have not received a response.
• [ ] I have tried searching the documentation and have not found an answer.

What Model are you using?

• [ ] gpt-3.5-turbo
• [ ] gpt-4-turbo
• [ ] gpt-4
• [x] Other (please specify)

Describe the bug The type: ignore[all] comment at the top of retry.py teaches mypy that retry has no attributes at all.

To Reproduce When I write a python module with:

from instructor.retry import reask_messages

and type check it with mypy in an environment with instructor installed, mypy complains:

error: Module "instructor.retry" has no attribute "reask_messages"  [attr-defined]

Expected behavior Since instructor ships with a PEP 561 py.typed file, I expected type checking to be largely correct across the package.

[ionflow]

@kojiromike it's in from instructor.exceptions import InstructorRetryException

[kojiromike]

Aha, thanks! Perhaps this was fixed in #710 three weeks ago and released in 1.3.3 then. I will test.

[kojiromike]

Unfortunately this is not really fixed. mypy still cannot see any attributes of retry.py, because it still has type: ignore[all] at the top. The example case is avoidable now because we can (and should) get the exception class from another module, but from instructor.retry import reask_messages still causes mypy to give an error.

I have edited the example to use reask_messages instead of InstructorRetryException.