![]() |
#181 |
US Navy, Retired
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 9,897
Karma: 13806776
Join Date: Feb 2009
Location: North Carolina
Device: Icarus Illumina XL HD, Kindle PaperWhite SE 11th Gen
|
Next time put the report in the post in a spoiler tag (eye with a red X through it in the advanced editor.) Last edited by DoctorOhh; 09-09-2011 at 06:51 AM. |
![]() |
![]() |
![]() |
#182 |
Groupie
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 193
Karma: 266070
Join Date: Dec 2010
Location: Spain
Device: Win10,Win11,Ubuntu,PockbookLux44
|
Report in spoilers
Trying to:
Remove calibre bookmarks/itune list (both present); duplicate/legacy jackets --> update; update cover Spoiler:
Plus general CSS/fonts cleanup: Spoiler:
Exit (untrue) statement: Spoiler:
Note: have tried to split the spoiler for ease of perusal; maybe that's wrong? |
![]() |
![]() |
![]() |
#183 |
US Navy, Retired
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 9,897
Karma: 13806776
Join Date: Feb 2009
Location: North Carolina
Device: Icarus Illumina XL HD, Kindle PaperWhite SE 11th Gen
|
Maybe you should try one thing at a time.
I have never had trouble with removing legacy jackets (attached is what a legacy jacket looks like) or removing all jackets or adding/updating jackets. Removing calibre bookmarks also always works for me. Updating cover will only work if the cover is already correctly set. If you do a save to disk and the cover doesn't update then I don't believe this plugin will update the cover either. |
![]() |
![]() |
![]() |
#184 | ||
US Navy, Retired
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 9,897
Karma: 13806776
Join Date: Feb 2009
Location: North Carolina
Device: Icarus Illumina XL HD, Kindle PaperWhite SE 11th Gen
|
Quote:
Quote:
I recently purchased a prs-950 and love how it works. The only draw back was that now all of my books are justified. I thought it was the reader but in looking into this it turns out it is just the way the books are formatted. This is easy to fix by replacing the text-align: justify entries with text-align: left entries and the book again defaults to left justification. Since I have older eyes I use larger fonts. Large fonts and full justification is ugly. You end up with big gaps between the words that screw with the flow of reading. My request is an option to change books to left justify by replacing all of the text-align: justify entries with text-align: left entries. I can do this one book at a time but the extra css in calibre isn't enough to ensure it isn't fully justified and the search and replace in the conversion doesn't seem to touch the css (for good reason I'm sure). Thanks for the work you're doing on this plugin and thanks for considering my request. |
||
![]() |
![]() |
![]() |
#185 |
creator of calibre
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 45,400
Karma: 27756918
Join Date: Oct 2006
Location: Mumbai, India
Device: Various
|
@dwannthy: Use the Text Justification potion under Conversion->Look & Feel, that should do the trick for you.
|
![]() |
![]() |
![]() |
#186 | |
US Navy, Retired
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 9,897
Karma: 13806776
Join Date: Feb 2009
Location: North Carolina
Device: Icarus Illumina XL HD, Kindle PaperWhite SE 11th Gen
|
Quote:
![]() Last edited by DoctorOhh; 09-13-2011 at 02:44 AM. |
|
![]() |
![]() |
![]() |
#187 |
Groupie
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 193
Karma: 266070
Join Date: Dec 2010
Location: Spain
Device: Win10,Win11,Ubuntu,PockbookLux44
|
Good plugin!! - abd small suggestion
Good work indeed: thanks.
My suggestion: CHANGE THE QUERY FOR APPLYING CHANGES to something more conventional and understandable. REASON: did my choices, ran plugin, and read: "Modify Epub modified 1 files: proceed with updating your library?" And I interpreted (my thickheadedness, no doubt, but the wording is error-prone): these changes ALREADY made --> View log (which I did - I had backed up my epub for the trial to be on the safe side). So, I pressed NO to "proceed with updating your library?", thinking that the plugin offered to apply same changes to the REST of my books. Silly, I dare say -- but it took me some time and trials to understand why I wasn't seeing any difference in the epub. I think that wording the query as "Modify Epub modified 1 files: apply the changes?" would be more simple for silly blunderers like myself. |
![]() |
![]() |
![]() |
#188 |
Calibre Plugins Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 4,731
Karma: 2197770
Join Date: Oct 2010
Location: Australia
Device: Kindle Oasis
|
@carmenchu - sorry the wording confused you, but it is intentionally consistent with the wording on the bulk metadata download dialog (which most users should be familiar with) and other plugins like Count Pages and Extract ISBN.
This is what the bulk metadata download text says: "Finished downloading metadata for 1 book(s). Proceed with updating the metadata in your library?" |
![]() |
![]() |
![]() |
#189 |
Guru
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 776
Karma: 2751519
Join Date: Jul 2010
Location: UK
Device: PW2, Nexus7
|
|
![]() |
![]() |
![]() |
#190 | |
Groupie
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 193
Karma: 266070
Join Date: Dec 2010
Location: Spain
Device: Win10,Win11,Ubuntu,PockbookLux44
|
Quote:
It was the bit about "proceed to update your Library" that sent me on the fool's track. And THANKS to YOU for your attention! |
|
![]() |
![]() |
![]() |
#191 |
Addict
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 293
Karma: 21022
Join Date: Mar 2011
Location: NL
Device: Sony PRS-650
|
Kiwidude,
Seems this project is on a high (forgotten?) shelf at the moment. At the other hand, included functions seem to work just fine, so why not adding it to the downloadable plugins? I've got 2 new function requests. A lot of my books seemed duplicates to me, after applying all fixes and do all quality checks, some just where not marked as duplicates using the duplicate checker or an external duplicate file finder. Then I compared the files using winmerge. This resulted in 4 types of differences, 3 of which could be easily fixed by an extension of this plugin. Type 1: File-structure. While comparing the files, all internal files where marked equal, the only difference was the internal structure. Flat vs folders. It would be nice to change this structure without having to convert the complete file (causing other changes to be executed as well). Type 2: comment. A very bad one. The files where compete equal, the only difference was based on 2 lines commented out in my opf file!!! While comments in the opf are not used (in html it is possible e.g. javascript parts), It would be nice to have the option to strip html comment out of the opf and toc files. Type 3: extra line. Another very ugly one. Files where equal excluding one empty line at the end of my mimetype file. (Based on the specs of 2.01 this is even forbidden!) While empty lines are not shown in html and not used in opf, I'd suggest to add an option to remove empty lines or lines that only contain white-characters \s (space, tab, \n, \r, other non-visible characters) Type 4: metadata in opf The last difference I found was the metadata in the opf-file. E.g. lang UND vs EN or comments versus no comments (about the book). And sometimes only a difference in sigil-version. This last one is a task that is not easy to implement. While you're able to update the file, it is not easy to update the comment based on metadata of another file. If it would be possible to implement function 1 2 and 3, I'd be very happy. Yet another request is to split the metadata-update function. I'd like to update my epub-files, but I always crop my covers for calibre. So if I update my epub, I will always change my cover in a cover with less quality. Splitting this function would make it possible to update the metadata without first having to remove the cover and later on extract the cover again. |
![]() |
![]() |
![]() |
#192 |
Calibre Plugins Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 4,731
Karma: 2197770
Join Date: Oct 2010
Location: Australia
Device: Kindle Oasis
|
@drMerry - this plugin hasn't been forgotten - it is just that I handed it over to Idolse to add some functionality, and "real life" has gotten in the way of much happening on that front for him. In fact Idolse and I have been exchanging a few messages about it over the last few weeks because I want to officially release it, even though it doesn't have everything in there that I ultimately want it to. What is in there currently works well enough, Idolse is making a few changes to the work he did around margins, and then I will do the support for keyboard shortcuts and release it. The #1 outstanding thing I would like it to have is the ability to support replacing the cover in *all* situations just like doing a conversion does - currently the "update metadata" command will only replace the cover if the cover can be "identified". However Idolse hasn't the time and I don't have the knowledge/time to gather it, so we will leave that as a future rainy day feature to add.
In answer to your suggestions - at the outset I will say that adding functionality "just" for the purposes of trying to do winmerge duplicate comparisons is not high on my priority list. That is the smallest niche of uses in my opinion. Specially my comments: (1) Changing file locations means having to rewrite all of the html, toc, etc to update links. And who is to say what "file structure" is correct? Calibre has one way, Sigil has another, raw book publishers have another again. Not gonna happen. (2) & (4) There already exists the "Update metadata" function, which rewrites the top part of the opf file. This should take care of bad or out of date being present in there, which is the primary use of this plugin. (3) I understand it causes an issue for your comparison (though surely not if you turn off whitespace checking?) but AFAIK it causes zero harm as far as ePub validity is concerned for readability. If someone states that their books are being rejected from a store because of this then I will add support, otherwise it is way down the list sorry. |
![]() |
![]() |
![]() |
#193 |
Addict
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 293
Karma: 21022
Join Date: Mar 2011
Location: NL
Device: Sony PRS-650
|
Hi Kiwidude,
Thanks for the response. Maybe I wasn't clear enough. The problem is not that I can't compare with winmerge. Winmerge just did it's Job very well, that's how I found these problems. Lets re-explain my case. I see 2 books. In my opinion, the 2 books are equal. When I do a binary duplicate check, it turns out they are not equal. That's where Winmerge jumps in. I used it to compare the files and I found the problems described. Winmerge tells me both files are equal, except for 1, 2, 3 or 4. So if I change it, a binary duplicate compare tells me that the files are equal. While comments and empty lines are only useless load that also takes (some) memory on my reader, it would be great to have it removed. And if it is, I'd also be able to remove a lot of my books because in that case it would turn out that they are duplicates. For example, I have x books with same author, the same title and (app.) the same size. While they are not binary equal, I have to open all the books to see if there is a difference. If not at first sight, I have to do a compare with e.g. winmerge to see if the difference is on the metadata files or on the real text. If the plugin could remove just the comments and empty lines, I would only have left 10 percent of my current duplicates that are not binary equal and that I will have to check. Did this text make the problem clear? (So lets concentrate on 2 and 3). 1 is indeed a discussion and could only be solved if you implemented a lot of options so the user should tell which one to use (not worth the effort). For option 4 are other ways. On the cover problem. I'd really appreciate to have the cover update function split. The reason is clear (to me). I have a lot of books. I want them to be identifiable in calibre by using the cover browser. These extracted covers consume a lot of space so I set all my covers to be max 250x250. If I update the metadata on my books, I want to keep the original big cover. To display on my reader or to import on a later moment in calibre. Reducing the cover-size saved me about 1GB!!!!! on files so you can imagine that I do not want to reset all the covers just to be able to update my metadata. At the other hand, I do sometimes have metadata I'm not completely sure about. But the cover is better than the cover currently in the epub file. In that case, I'd like to easily add the cover to the epub by pushing the update button in the plugin without exploding the epub, adding the file and than again compacting the epub. |
![]() |
![]() |
![]() |
#194 |
Calibre Plugins Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 4,731
Karma: 2197770
Join Date: Oct 2010
Location: Australia
Device: Kindle Oasis
|
@drMerry - I understood what you meant perfectly fine, it doesn't change my mind about it
![]() You *cannot* make two ePubs *binary* equal for the Find Duplicates plugin by manipulating them with a plugin like this. They will have different timestamps on the files for a start, which means Find Duplicates will report them as different, no matter whether the text inside is the same or not. So you are still going to have to run some other tool (like WinMerge) to compare them if you are going down that route. As I said above, I see it as too niche a requirement to justify the effort. However I know you have dabbled with writing plugins yourself, so feel free to create one for your own needs. ![]() As for covers, it may be clear to you but sadly very fuzzy to me ![]() |
![]() |
![]() |
![]() |
#195 | ||
Addict
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 293
Karma: 21022
Join Date: Mar 2011
Location: NL
Device: Sony PRS-650
|
Quote:
A modification of your plugin will help me in that case, but won't most of the other users (that will use your duplicate scanner). Thanks for the explanation. Quote:
![]() In my idea, this is what your plugin does (for updating metadata): When updating metadata, the opf meta part is REPLACED with data stored in calibre (or, the calibre opf-file). Besides that, the cover in the folder is added to the epub. If a cover already exists, it is replaced. This would be a good option for books having no cover, a projectgutenberg cover or a bad cover. But in my case, the cover used for calibre is, in most cases, of lower quality than the cover that is stored inside my epub. So I do not want these lower quality covers to be included in my epub (not even as extra cover, in case it does not overwrite the original cover). But, maybe I'm all wrong about this function. This is a case that I saw using one of the first versions of the plugin. Maybe the current version doesn't work that way. In that case, I'm the one who is mistaken (and made you confused). |
||
![]() |
![]() |
![]() |
|
![]() |
||||
Thread | Thread Starter | Forum | Replies | Last Post |
Any web-to-epub plugin for internet browser? | bthoven | ePub | 7 | 07-10-2011 05:14 AM |
[Old Thread] Reading epub on viewer inexplicably changes the time stamp of epub | greenapple | Library Management | 20 | 03-19-2011 10:18 PM |
Easy way to modify thread subscription emails in bulk? | snipenekkid | Feedback | 11 | 02-06-2011 03:47 AM |
Another plugin dev question | DiapDealer | Plugins | 2 | 12-11-2010 01:46 PM |
Epub plugin dev | DiapDealer | Plugins | 15 | 11-12-2010 09:36 AM |