mienaiyami / yomikiru

An offline desktop reader for manga, comics, and novels. Offers a customizable reading experience with extensive settings, shortcuts, themes and layouts. Manga reader / Manhwa reader / Webtoon Reader / EPUB Reader. **Not a downloader**
MIT License
217 stars 10 forks source link

"Error occurred while loading image" when trying to open Full Colored pages #351

Closed himouto closed 2 months ago

himouto commented 3 months ago

Type of installation

Setup (.exe)

Type of reader (if reader related)

None

Steps to reproduce

Reader works fine on normal grayscale mangas but getting an error whenever I try to open a manga with Full Colored pages. It doesn't load the images and just displays "Error occurred while loading image". Already tried enabling "Canvas Based Rendering" but it didn't help.

✔️ Expected Behavior

No response

❌ Actual Behavior

No response

mienaiyami commented 3 months ago

Is there any error when "Canvas Based Rendering" is disabled? "Error occurred while loading image" is only showed when its enabled. You will need to close and reopen reader for changes to take place. If possible, please send the average image resolution and size under.

himouto commented 3 months ago

Getting the error with "Canvas Based Rendering" enabled or disabled

mienaiyami commented 3 months ago

That would be weird because there is no error defied when "Canvas Based rendering" is disabled. Can you show some screenshots, or screen recording if possible? Also make sure the images are not corrupted by opening them inside an image viewer.

himouto commented 3 months ago

Not the same error, reader won't load the images if they're full colored for some reason

"Canvas Based Rendering" enabled:

Screenshot 2024-02-04 132111

"Canvas Based Rendering" disabled:

Screenshot 2024-02-04 132026

Try to recreate it with any full colored manga/comics .cbz file with .png pages with at least 1400px horizontal resolution

himouto commented 3 months ago

These errors only occur with .cbz files - I cant recreate it with the folder extracted then dragged to the reader

zeoint commented 3 months ago

This might be related to #141, one of the reasons why it is recommended to use folder as chapters instead of archives.

mienaiyami commented 3 months ago

I will add option to use raw image data instead of filename to bypass this issue. But not that it will greatly increase RAM and CPU usage.

zeoint commented 3 months ago

I will add option to use raw image data instead of filename to bypass this issue. But not that it will greatly increase RAM and CPU usage.

The user above states that this error only occurs with archive and not with folders. So wouldn't just using the folder as chapters solve the issue. My thoughts.

mienaiyami commented 3 months ago

@zeoint Thats because most of the official cbz I encountered are designed like that, each page has whole manga name followed by page number, and if the manga name is too large then everything breaks.

himouto commented 3 months ago

I will add option to use raw image data instead of filename to bypass this issue. But not that it will greatly increase RAM and CPU usage.

The user above states that this error only occurs with archive and not with folders. So wouldn't just using the folder as chapters solve the issue. My thoughts.

Extracting all my cbz archives to folders just to use the reader would be hassle - also lose cover thumbnails, higher file size etc.

zeoint commented 3 months ago

@zeoint Thats because most of the official cbz I encountered are designed like that, each page has whole manga name followed by page number, and if the manga name is too large then everything breaks.

I see, then this would option would be helpful for cbz readers.

himouto commented 2 months ago

Any updates with the fix @mienaiyami ?

himouto commented 2 months ago

Thanks