Conversation
| def _get_client(self) -> openai.OpenAI: | ||
| """Get or create OpenAI client instance.""" | ||
| import openai | ||
| openai = require("openai", "openai") |
There was a problem hiding this comment.
Это в init уже проверяется. Мне кажется что можно просто импортировать
There was a problem hiding this comment.
без этих штук ruff ругается что не установлен openai + так консистентнее, потому что в большинстве случаев именно через require импорт идет
There was a problem hiding this comment.
Я думаю ты про mypy и такой способ не позволяет валидировать типизацию
There was a problem hiding this comment.
- окей, это не ruff, это какой-то другой линтер
- хорошо это кажется действительно, тогда я еще подумаю - потому что тогда мы должны отказаться от использования require во всех местах где только можно
There was a problem hiding this comment.
тогда мне кажется что стоит отказаться от использования require для импорта - стоит его использовать только чтобы выводить информационное сообщение об отсутствии необходимой зависимости
импорты опциональных зависимостей стоит делать только внутри TYPE_CHECKING секции и внутри функций и методов классов. первое обеспечит тайпчекинг, второе позволит лениво подгружать опциональные зависимости
а на эти проблемы reportMissingImports придется забить
что думаешь?
There was a problem hiding this comment.
Да, согласен. В mteb так и делаем
2 participants
No description provided.