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

Support 'RGBA' mode images by converting to 'RGB' #14

Merged
merged 2 commits into from
Mar 6, 2019
Merged

Conversation

MLnick
Copy link
Contributor

@MLnick MLnick commented Mar 5, 2019

Closes #12, #13

@MLnick MLnick requested review from feihugis and ptitzler March 5, 2019 14:10
@MLnick MLnick changed the title Support 'RFBA' mode images by converting to 'RGB' Support 'RGBA' mode images by converting to 'RGB' Mar 5, 2019
@MLnick
Copy link
Contributor Author

MLnick commented Mar 5, 2019

cc @zuhito

@zuhito
Copy link
Contributor

zuhito commented Mar 5, 2019

It looks great! After merging this, I will use PNG files using this Docker image.

Copy link
Contributor

@ptitzler ptitzler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comment, which is not specific to this fix but possibly related.

core/model.py Show resolved Hide resolved
@ptitzler
Copy link
Contributor

ptitzler commented Mar 5, 2019

Created a "test" image by converting one of our asset files from jpeg to tiff. File is ZIPped because GH can't handle TIFF.
baby-bear.zip

@MLnick
Copy link
Contributor Author

MLnick commented Mar 5, 2019 via email

@MLnick
Copy link
Contributor Author

MLnick commented Mar 6, 2019

Added test case for TIFF, Travis is happy. Merging

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.

4 participants