Skip to content

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