Fall back to a default resolution when the target profile has none

[NilsLeo]

Selecting the generic 'Other'/'OTHER' device (profile resolution (0, 0)) without supplying custom width/height propagates a (0, 0) target into PIL resize/thumbnail, crashing every such conversion with a ZeroDivisionError.

This falls back to a sane default e-reader resolution (1264×1680) when the resolved profile has a zero dimension, so the generic profile no longer hard-crashes.

How to reproduce / test

Any convertible comic reproduces this — the crash is about the profile, not the file. You just need an archive that extracts cleanly and is converted with -p OTHER (no custom -u/width/height, so the target resolution stays (0, 0)).

Download: https://drive.google.com/file/d/1z7SlgQZxPTADimAOhDKxhC41lVmQOMM7/view?usp=drive_link

Save it next to where you run the commands (referred to below as 1400.cbz). It is a plain flat CBZ of valid pages; the only thing that matters is the -p OTHER profile.

The easiest way to A/B test any KCC branch without a local setup is kcc-run, which clones a given owner/repo:ref into a throwaway container at runtime:

export DOCKER_DEFAULT_PLATFORM=linux/amd64   # Apple Silicon; no-op on Intel/AMD
kcc() { docker run --platform linux/amd64 --rm -v "$PWD:/work" -w /work ghcr.io/nilsleo/kcc-run "$@"; }

1. Reproduce the crash on upstream master (-p OTHER, no custom size):

kcc ciromattia/kcc:master -p OTHER -f EPUB -o out 1400.cbz

→ aborts with ZeroDivisionError: division by zero in PIL/Image.py thumbnail (target resolution (0, 0)).

2. Same file on this branch — falls back and converts:

kcc NilsLeo/kcc:up/default-resolution-fallback -p OTHER -f EPUB -o out 1400.cbz

→ logs WARNING: profile "OTHER" has no resolution; falling back to 1264x1680, then writes a valid out/1400.epub.

Control — a real profile already works on master, confirming it is the profile and not the file:

kcc ciromattia/kcc:master -p KV -f EPUB -o out 1400.cbz

(Or clone this branch and run python kcc-c2e.py -p OTHER -f EPUB -o out 1400.cbz directly.)

[NilsLeo]

this works with any file but ive added one just in case

[NilsLeo]

Hey! Would it be possible to grant me Maintainer access to the repository?

I don't need to be able to push directly to main, but it would be good to have the ability to create feature branches in the repository and open PRs from those branches instead of having to work from a fork.

[axu2]

I don't think that's needed, I personally don't do any work on ciromattia/kcc, I do everything in my fork and PR.