fix!: switch DALLEImageGenerator to gpt-image-2#11321
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Coverage reportClick to see where and how coverage changed
This report was generated by python-coverage-comment-action |
||||||||||||||||||||||||||||||||||||
Code reviewChecked for bugs and CLAUDE.md compliance. One issue found:
AGENTS.md states: "To fix type issues, avoid The new assert self.client is not None # warm_up() sets self.client; hint for mypy |
bogdankostic
left a comment
There was a problem hiding this comment.
Looking good, I'm just wondering if at some point we should also rename the component to something like OpenAIImageGenerator.
Noted for Haystack 3 on Notion |
Related Issues
Proposed Changes:
DALLEImageGeneratortogpt-image-2, adapt for API changes but trying to keep backward compatibilityHow did you test it?
CI, new (slow) integration test
Checklist
fix:,feat:,build:,chore:,ci:,docs:,style:,refactor:,perf:,test:and added!in case the PR includes breaking changes.