openai[patch]: add disabled_params

[baskaryan]

• enable json schema mode in azure openai
• add ability to disable different params

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
langchain	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 20, 2024 6:52pm

[ccurme]

As a class attribute this seems somewhat mysterious, as it only applies to with_structured_output. My first thought is to add a bind_kwargs: Optional[dict] kwarg to with_structured_output, e.g.,

if bind_kwargs is None:
    bind_kwargs = {
        "tool_choice": tool_name,
        "parallel_tool_calls": False,
        "strict": strict,
    }

then Azure can override with_structured_output and pass this in to the parent class method.

Alternatively, can update the naming here / clarify in documentation.

[baskaryan]

i think disabled_params is something we're gonna find useful beyond structured output if/when azure and openai apis diverge again

if it works well could generalize it to others integrations like bedrock (where apis are model-specific) and vertexai

[baskaryan]

outdated given latest azure oai api versions, will open new pr