Skip to content

Conversation

@snimu
Copy link
Contributor

@snimu snimu commented Apr 4, 2024

Fixes issue #591 without re-creating issue #543

The problem was that AzureOpenAI.copy used LM.copy, but because api_key etc. aren't in the AzureOpenAI.kwargs, and LM.copy per default only propagates self.__class__.kwargs.

I now gave AzureOpenAI its own copy-method. This isn't really beautiful, but it seems to work pretty well.

@arnavsinghvi11 arnavsinghvi11 merged commit 5ed6f5e into stanfordnlp:main Apr 4, 2024
@arnavsinghvi11
Copy link
Collaborator

Thanks @snimu !

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.

2 participants