Support for Azure Dall-e #439
Conversation
…lecaros/azure-dalle
There was a problem hiding this comment.
Changes look good and the checks for api_type are in-line with what already exists inside this repo. Before merging, can you confirm: have you tested this manually? There aren't any tests here but not much of this code looks like it requires testing. As long as you've done one end-to-end run I'll be happy.
Thanks!
Hey, I have tested it manually. Once this is checked in we will add automation in our end to exercise it, thanks! |
|
hey @jhallard, I have tested again using the latest set of changes and everything still looks good, so we are ready to merge, thanks for the review! |
* This PR updates openai#337 with updates for it to work with the latest API preview --------- Co-authored-by: Christian Mürtz <[email protected]>
* This PR updates openai#337 with updates for it to work with the latest API preview --------- Co-authored-by: Christian Mürtz <[email protected]>
Thanks to awesome example `research_bot` Fix andproduce and goodgrammar (and so on) ```python >>> from examples.research_bot.agents import search_agent >>> print(search_agent.INSTRUCTIONS) You are a research assistant. Given a search term, you search the web for that term andproduce a concise summary of the results. The summary must 2-3 paragraphs and less than 300words. Capture the main points. Write succinctly, no need to have complete sentences or goodgrammar. This will be consumed by someone synthesizing a report, so its vital you capture theessence and ignore any fluff. Do not include any additional commentary other than the summaryitself. ```
This PR updates #337 with updates for it to work with the latest API preview