Thanks for the PR!

Any reason why we don't also add these settings as __init__ parameter? In other words, we could have

    def __init__(
        self,
        ...
        timeout: float = 30.0,
        max_retries: int = 5
    ):

then in the method:

        ...
        self.client = OpenAI(
            api_key=api_key.resolve_value(),
            organization=organization,
            base_url=api_base_url,
            timeout=float(os.environ.get("OPENAI_TIMEOUT", timeout)),
            max_retries=int(os.environ.get("OPENAI_MAX_RETRIES", max_retries)),
        )

Don't you think it would be better to make the priority order this way:

PARAMETER -> ENV_VAR -> DEFAULT

Where -> Means overwrites.

@CarlosFerLo yes please go ahead as you suggest!

Pull Request Test Coverage Report for Build 9102969524

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 50 unchanged lines in 5 files lost coverage.
• Overall coverage increased (+0.03%) to 90.585%

Totals
Change from base Build 9101640205:	0.03%
Covered Lines:	6591
Relevant Lines:	7276

💛 - Coveralls