Skip to content
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

[FEAT] Created add_image_text_converter and unit tests #328

Merged
merged 4 commits into from
Aug 16, 2024

Conversation

jsong468
Copy link
Contributor

Created a converter called AddImageTextConverter which is similar to AddTextImageConverter except that it takes in an image path instead of text in its constructor, and the convert_async function takes in a text prompt as opposed to a file path.

Testing: Used test_add_text_image_converter.py as a baseline (including tests for initialization, font checking, input text checking, conversion) and added a couple of additional assert statements and unit tests, including one that compares the outputted image pixels of AddImageTextConverter to those of AddTextImageConverter when inputting the same text and base image.

@jsong468 jsong468 marked this pull request as ready for review August 15, 2024 20:59
@jsong468 jsong468 merged commit 9c3be20 into Azure:main Aug 16, 2024
5 checks passed
@jsong468 jsong468 deleted the 3156-AddImageToText branch August 16, 2024 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants