strider1551 / djvubind

Combine multiple image files into an optimally compressed djvu file.
19 stars 6 forks source link

Deal with image density in pixel/cm. Round to integer dpi. #16

Closed ospalh closed 8 years ago

ospalh commented 9 years ago

This fixes the problem #15 for me. On the other hand, i am not sure that you want to pull it as-is. The ValueError does not really get caught anyway. When you have a third unit, it still stops.

ospalh commented 9 years ago

Oh, well. Looks like it will still fail with non-integer values, that is pixel/m as pixel/cm, when you use the front-cover or back-cover options. eta: There you will have to deal with the unit “Undefined“. And i have seen 0 Undefined and values like 100 Undefined or even some nonsense values like the image size repeated as image resolution.