Skip to content

Conversation

@Yummy2016
Copy link

This commit is to remove unused kwargs that we include in the requests sent through AzureOpenAI class.

This seems to fix this issue: #543

This commit is to remove unused kwargs that we include in the requests sent through `AzureOpenAI` class.

This seems to fix this issue: stanfordnlp#543
@insop
Copy link
Contributor

insop commented Mar 7, 2024

@Yummy2016 thank you for the PR.
Let me check the change, first.

This code was updated recently due to this issue, #521.
I am using legacy openai.

@insop
Copy link
Contributor

insop commented Mar 7, 2024

Thank you for the change. Let's merge this as many are hit the issue, @okhat.

(updated since previous note was not valid)

I tested with this PR and I still hit the issue #521.
For my setup, I use llm_metric and teach lm that are all azure model and saw this error when I am using BootstrapFewShotWithRandomSearch

<function llm_metric at 0xxx> due to __init__() missing 2 required positional arguments: 'api_base' and 'api_version'.

But let's get this change in first, and I will figure out how to address #521

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants