magentic icon indicating copy to clipboard operation
magentic copied to clipboard

Add tests against the Azure OpenAI API

Open jackmpcollins opened this issue 1 year ago • 1 comments

Add tests against the Azure OpenAI API to make sure this is working as expected.

As part of this improve usage experience of Azure. Some ideas/options

  • add inputs to the OpenaiChatModel class for the Azure-specific fields if only a few
  • create a separate AzureOpenaiChatModel for this
  • (also?) allow passing an OpenAI client into the init of OpenaiChatModel. This might be generally useful.

Related comment https://github.com/jackmpcollins/magentic/issues/261#issuecomment-2227657589

jackmpcollins avatar Jul 15 '24 03:07 jackmpcollins

(also?) allow passing an OpenAI client into the init of OpenaiChatModel. This might be generally useful.

Let's avoid this if possible so that ChatModel parameters remain easily serializable so in future it can potentially be a return type of a prompt-function.

jackmpcollins avatar Feb 05 '25 18:02 jackmpcollins