Cortys / comic-backup

Back up your comics as CBZ.
https://cmxlgy.wordpress.com
GNU General Public License v3.0
296 stars 33 forks source link

Support for Right to Left reading (Manga) #13

Closed Kataomoi closed 9 years ago

Kataomoi commented 10 years ago

I'm hoping in a future version you can add support for Manga that reads right to left. When I try to save a manga, it stalls at 0% and does nothing. I believe this is due to the different way pages advance.

Kimship commented 9 years ago

First, thanks so much for this extension. Second, I'm having a similar issue with the manga stalling at 0%, but I think it's because every time it opens the book, there is a pop-up box that warns you that it is in "manga fixed format" and forces you to hit "OK" to continue. I think that this is interrupting the backup process somehow. mangamixedmessage

shinji257 commented 9 years ago

I'm not sure if that is doing it but I notice that on graphic novels where they have you going left for the next page that the script is numbering them backwards as it is starting at the last page and moving towards the first one instead of the other way around.

A good free example of this is the Inazuman series where they have free previews that exhibit this exact behavior.

EDIT: Ok this issue isn't related to the above. Well sorta. The issue with Hikaru no go (and the manga fixed format) is that it forces the page to two page layout instead of single page. Chances are though even if the 2 page layout issue was cleared up it still would suffer from reversed page numbering due to the back to front / right to left reading style of japanese manga.

Cortys commented 9 years ago

I was busy working on other projects lately.

Manga support will be pushed to master soon. Just started working on it in the 2.1.0 branch.

Cortys commented 9 years ago

Could anyone give me the names of a few comics that use manga fixed format (ideally free previews)?

Thanks!

Kataomoi commented 9 years ago

https://www.comixology.com/Boys-Over-Flowers-Season-2/comics-series/40497

There are a few chapters here that are free and read right to left.

Cortys commented 9 years ago

@Kataomoi Thanks! Would it be acceptable if fixed format comics were exported as dual-page-images (not split up to individual pages)?

Kataomoi commented 9 years ago

Single page preferred, but dual is definitely workable. Thanks for looking into this!

Cortys commented 9 years ago

Fixed format mangas should be supported now.

Could you verify that (the beta is on the 2.1.0 branch)?

shinji257 commented 9 years ago

I'd like to see Kataomoi's input on this but I checked this issue against a couple of mangas. One was one I mentioned earlier that may of been related although not forced 2 page then the other was the original that used a forced "manga" format. Both seemed to backup ok with proper numbering. However I saw an odd issue when backing up the forced manga format.

The first time I backed it up the 1st page right after the cover was skipped. Then it used odd numbering the rest of the way through for the images so it was numbered like this...

page000 page002 page005 page007 etc...

I turned off transitions just in case then redid it and it got it properly the next time and used even numbers. for the run. page000 page001 page002 page004 etc...

(while typing this test a 3rd time -- also without transitions) It did it again... Skipped the 2nd page.

(watches the download) Oh.... it looks like it is going directly from the cover to the first 2-page spread for some reason and skipping the 2nd single page. If you send me a message I'll gift you a copy of the manga being affected since it may be acting different than what you tested with. The interesting bit is the issue is intermittent. Usually re-doing the scan is enough but it doesn't appear the delay is the issue at all. As part of my testing I did increase the delay to 700ms.

(tests using the freebies above) Ok... I checked using the freebies and they did the same thing so there is no need to actually do the gift but the offer does still stand anyways. There is the cover then a single page (page 2) then it goes to 2-page format. The script appears to be going from page 1 (cover) then directly to page 3 (first 2-page layout) skipping over page 2.

Cortys commented 9 years ago

Thanks for the report.

I thought I fixed the page skipping bug you described. Which delay were you using when it occurred? Always 700ms?

shinji257 commented 9 years ago

At first I saw it on the default then extended it to 700. Once I started actually watching it I never saw it hit page 2. During the short period when the page selection was at the bottom I saw it go directly from page "1" to page 3 then continue from there.

Cortys commented 9 years ago

Hm. That's odd. I could not reproduce the bug (tested with the freebies).

With enabled animations the cover pages were missing in my scans though. Looks like there is yet another general timing issue not related to any specific page.

shinji257 commented 9 years ago

Did you clear your cache? It seems to only affect the first iteration. It's still really strange. It was reproducible after clearing cache.

Cortys commented 9 years ago

Yes I did. Since both issues are quite similar I'll try to fix the ones I'm experiencing, which will hopefully fix the ones you described as well.

Kataomoi commented 9 years ago

I can't get it past the initial setup scan (pressing single page, multipage).

Cortys commented 9 years ago

@Kataomoi You have to do the setup with a regular (left to right) comic.

Kataomoi commented 9 years ago

I compared the first 2 chapters (about 60 pages) and did not notice any dropped pages. From what I can tell, everything is working.

Cortys commented 9 years ago

I just committed a major change in the latency compensation logic, which should still have the improved backup speed of the last implementation while supporting guided view narrative and mangas.

@shinji257 The bugs I experienced are fixed in the new implementation. Could you check if yours are fixed as well?

Cortys commented 9 years ago

As I'm quite confident in the quality of the 2.1.0 branch, I'll release it now and consider this bug fixed.

If you should still experience issues with mangas, I'll reopen of course.

Kataomoi commented 9 years ago

Great! Thank you so much for the work on this.