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

IC pipeline to accept PNG images #870

Merged
merged 3 commits into from
Jan 20, 2023
Merged

IC pipeline to accept PNG images #870

merged 3 commits into from
Jan 20, 2023

Conversation

dbogunowicz
Copy link
Contributor

@dbogunowicz dbogunowicz commented Jan 20, 2023

PIL reads PNG images as RGBA by default, causing an error later on in the pipeline as all images are assumed to have three channels. This PR makes sure that whatever is being returned by PIL, has 3 dimensions

Testing:

  • manually tested the result of the pipeline for an input JPG image (the result does not change after introducing this PR)
  • made sure that the PNG image gets properly inferred (the image below results in top 3 labels labels=["banana", "orange", "lemon"])image

@mwitiderrick
Copy link
Contributor

Worked
image

mwitiderrick
mwitiderrick previously approved these changes Jan 20, 2023
@dbogunowicz dbogunowicz merged commit 1634419 into main Jan 20, 2023
@dbogunowicz dbogunowicz deleted the feature/damian/pngs branch January 20, 2023 12:33
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.

3 participants