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

Color gem expects 4bit or 8bit color strings #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mrloop
Copy link

@mrloop mrloop commented Jan 3, 2017

The color gem expects hex color string with 3 or 6 digits, see:
https://github.com/halostatue/color/blob/2159ca233851ca3304414b831d9da368dcbafb04/lib/color/rgb.rb#L609

When miro generates a histogram it uses Color::RGB.from_html however
image magick can generate 16bit color strings. Using these and
from_html raises an error.

The PR tell image magick we want 8bit color strings.

Had been experiencing this issue with Version: ImageMagick 6.8.9-9 Q16
x86_64 2016-11-29 http://www.imagemagick.org

The color gem expects hex color string with 3 or 6 digits, see:
https://github.com/halostatue/color/blob/2159ca233851ca3304414b831d9da368dcbafb04/lib/color/rgb.rb#L609

When miro generates a histogram it uses `Color::RGB.from_html` however
image magick can generate 16bit color strings. Using these and
`from_html` raises an error.

The PR tell image magick we want 8bit color strings.

Had been experiencing this issue with Version: ImageMagick 6.8.9-9 Q16
x86_64 2016-11-29 http://www.imagemagick.org
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.

1 participant