Error: Command failed: convert: unrecognized dither method `-background' about hass-lovelace-kindle-screensaver HOT 8 CLOSED

Yep, spotted this after testing and talking to @sibbl about another issue with env's so looks like it's due to the line 308 in index.js ( .dither(pageConfig.dither))

I may get time tomorrow to take a quick look.

from hass-lovelace-kindle-screensaver.

The cause of this might be that our Dockerfile uses imagemagick instead of graphicsmagick. While the used npm module gm says to be compatible with both, it doesn't seem to support the differences in the dither parameter. While +dither works for both, the -dither parameter requires the name of the dither algorithm to use in imagemagick, while graphicsmagick only uses it as a toggle.

Maybe it's worth to give Graphicsmagick another try in the Dockerfile but I'm not sure what other incompatibilities will occur.
An alternative would be to open an issue in the gm npm package repository.
Finally, we could also just use the "custom arguments" feature of the gm npm package to support -dither ALGORITHM and ´+dither` by our own. That's probably what I'd prefer.

from hass-lovelace-kindle-screensaver.

Yep, spotted this after testing and talking to @sibbl about another issue with env's so looks like it's due to the line 308 in index.js ( .dither(pageConfig.dither))

I may get time tomorrow to take a quick look.

That would be amazing, thanks for taking a look!

from hass-lovelace-kindle-screensaver.

This only seemed to come with the update to Home Assistant yesterday, as well as the v1.0.7 update today. I'm not sure if that helps at all. Node.js v18.19.1 running HAOS on a RPB4.

from hass-lovelace-kindle-screensaver.

Oh, now I understand. After the update, it doesn't work either way - with and without the toggle? I thought it was just a problem after enabling it.

As a quick fix, I'll remove the option again until we can find out the real cause. Version 1.0.8 will be available in a few minutes.

from hass-lovelace-kindle-screensaver.

Oh, now I understand. After the update, it doesn't work either way - with and without the toggle? I thought it was just a problem after enabling it.

As a quick fix, I'll remove the option again until we can find out the real cause. Version 1.0.8 will be available in a few minutes.

Yep that's correct! No matter the setting, or after reinstalling it's the same error. Thanks for taking a look! Will let you know about v1.0.8. Thanks for the quick responses!

from hass-lovelace-kindle-screensaver.

Can confirm this has now fixed it. Great work!

from hass-lovelace-kindle-screensaver.

Thanks for the confirmation and sorry that it happened in the first place!

from hass-lovelace-kindle-screensaver.