Nicer console output while enhancing

[vi]

file.png (tile    4 of    9)

instead of just printing dots

[alexjc]

Quick review for this if you're keen to listen:

• Can you compute the itertools.product only once and put it in a list (DRY principle).
• You can determine its length using Python's len rather than a counter.
• Also, in the main loop you can use enumerate rather than another counter.

This should simplify the code and make it more Pythonic. I'd welcome the change afterwards.

[thomasbarwanitz]

Nice

[thomasbarwanitz]

Nice

[scribblemaniac]

Based on the changes in this PR, it seems like numtiles would just be equal to math.ceil(original.shape[0] / s) * math.ceil(original.shape[1] / s), which would be faster and cleaner than itertools.product.

[vi]

• Rebased against master
• Implemented mentioned refactorings
• Printing numer of tiles before _{(for user to abort earlier if it one specifies wrong too big image)} and after _{(to make image size obvious from the log after enhancing and avoid clumsy "(tile 27 of 28)")} processing of each file
• Nicer output, usage of better working \r instead of hackier \x7f.

[vi]

Can it process tiles in parallel BTW? Currently it looks like using only one core.

[vi]

I'd welcome the change afterwards.

@alexjc, So, is the change welcome now?

[vi]

Rebased against v0.4