-
Notifications
You must be signed in to change notification settings - Fork 41
FEATURE: add OpenAI image generation and editing capabilities #1293
Conversation
|
odd, failure is a flake... having a think about how to fix |
this increases distance
Co-authored-by: Alan Guo Xiang Tan <[email protected]>
Co-authored-by: Alan Guo Xiang Tan <[email protected]>
| size: "auto", | ||
| api_key:, | ||
| api_url:, | ||
| n: 1, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if there is a more descriptive name we can use for this kwarg. Not sure what n stands for here without having to read through method.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah its tricky, I map to what open ai call the param, we can remap internally, but generally we keep it at 1 anyway so maybe we can even remove for now.
tgxworld
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me 👍 Thank you for the changes.
This commit enhances the AI image generation functionality by adding support for:
Technical changes include:
ai_openai_dall_e_3_urltoai_openai_image_generation_urlwith a migrationai_openai_image_edit_urlsetting for the image edit API endpoint