New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
OpenAI DALL·E support #298
Conversation
# Conflicts: # pom.xml
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.
@Heezer thanks a lot! Please check my comments.
I trust that it works, but I would add at least one integration test to make sure it keeps working in the future.
Thanks!
langchain4j-core/src/main/java/dev/langchain4j/data/image/Image.java
Outdated
Show resolved
Hide resolved
langchain4j-core/src/main/java/dev/langchain4j/model/chat/ImageProcessingModel.java
Outdated
Show resolved
Hide resolved
...-ai-dall-e/src/main/java/dev/langchain4j/image/openai/dalle/AuthorizationHeaderInjector.java
Outdated
Show resolved
Hide resolved
...j-open-ai-dall-e/src/main/java/dev/langchain4j/image/openai/dalle/OpenAiImageDownloader.java
Outdated
Show resolved
Hide resolved
...dall-e/src/main/java/dev/langchain4j/image/openai/dalle/OpenAiDalleImageProcessingModel.java
Outdated
Show resolved
Hide resolved
langchain4j-core/src/main/java/dev/langchain4j/data/image/Image.java
Outdated
Show resolved
Hide resolved
...dall-e/src/main/java/dev/langchain4j/image/openai/dalle/OpenAiDalleImageProcessingModel.java
Outdated
Show resolved
Hide resolved
...n4j-open-ai-dall-e/src/main/java/dev/langchain4j/image/openai/dalle/OpenAiDalleResponse.java
Outdated
Show resolved
Hide resolved
inte test added |
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.
@Heezer great job, thank you! Please check comments
langchain4j-core/src/main/java/dev/langchain4j/data/image/Image.java
Outdated
Show resolved
Hide resolved
langchain4j-core/src/main/java/dev/langchain4j/model/image/ImageModel.java
Outdated
Show resolved
Hide resolved
langchain4j-open-ai/src/main/java/dev/langchain4j/model/openai/OpenAiImageModel.java
Show resolved
Hide resolved
langchain4j-open-ai/src/main/java/dev/langchain4j/model/openai/OpenAiImageModel.java
Show resolved
Hide resolved
langchain4j-open-ai/src/main/java/dev/langchain4j/model/openai/OpenAiImageModel.java
Outdated
Show resolved
Hide resolved
langchain4j-open-ai/src/main/java/dev/langchain4j/model/openai/OpenAiImageModel.java
Show resolved
Hide resolved
langchain4j-open-ai/src/test/java/dev/langchain4j/model/openai/OpenAiImagesGenerationIT.java
Outdated
Show resolved
Hide resolved
langchain4j-open-ai/src/main/java/dev/langchain4j/model/openai/OpenAiImageModel.java
Outdated
Show resolved
Hide resolved
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.
@Heezer thank you so much, great job!
This uses the same LangChain classes as #298 by @Heezer - langchain4j-core/src/main/java/dev/langchain4j/data/image/Image.java - langchain4j-core/src/main/java/dev/langchain4j/model/image/ImageModel.java I copy/pasted those 2 files here as #298 isn't merged yet, and my goal is that we use the same API in the end.
Motivating generated picture 😄 ❤️