instructor-ai / instructor

structured outputs for llms
https://python.useinstructor.com/
MIT License
8.16k stars 654 forks source link

jsonref missing from requirements.txt #1183

Open davesyms opened 5 hours ago

davesyms commented 5 hours ago

What Model are you using?

Describe the bug client_vertexai.py imports jsonref on line 9. jsonref is not included in the requirements.txt

To Reproduce If jsonref has not been installed on the venv running instructor.from_vertexai will result in a ValueError - instructor has no attribute from_vertexai. If directly importing: from instructor.client_vertexai import from_vertexai a module not found exception is thrown.

Expected behavior For jsonref to be included in the requirements.txt.

davesyms commented 5 hours ago

Fixed in PR: https://github.com/instructor-ai/instructor/pull/1184