FYI @Charles: article sort tweak and autosort/automerge

Starson17 · 10-21-2010, 11:50 AM

A minor update/change:

Responding to a bug report/enhancement request, the autosort/automerge option in Adding Books|"If books with similar author ..found" (used in Adding Books and Copy to Library to identify matching books in the library) now uses Charles' title_sort_articles tweak to identify articles to be ignored during the compare. (Other fuzzy title matching is still done to ignore punctuation, multiple spaces, etc.)

The default tweak is: title_sort_articles=r'^(A|The|An)\s+' which means the default autosort/automerge behavior is unchanged even if the tweak is undefined.

The comparison is done with case ignored. Changing the tweak to title_sort_articles=r'^(A|The|An|XYz)\s+' means that a book with the title "XYZ Circus" will match all these book titles:
"Circus"
"circus"
"A Circus"
"a Circus"
"the Circus"
"The Circus"
"An circus"
"AN Circus"
"xyz Circus"
etc., provided the author(s) field also matches.

chaley · 10-21-2010, 11:56 AM

@Starson17: Is this an FYI? If so, thanks.

If it is to point me at a problem, I don't see what the problem is.

Starson17 · 10-21-2010, 12:02 PM

Quote:

Originally Posted by chaley

@Starson17: Is this an FYI? If so, thanks.

If it is to point me at a problem, I don't see what the problem is.

Sorry, I was unclear. The purpose wasn't to point at a problem. It was just to let you know your code had a new effect in case you have a reason to change that tweak in any way.

Manichean · 10-21-2010, 12:34 PM

I think that was in response to a bug I filed some time ago. Charles, I believe we had a brief discussion about this, though I can't for the life of me remember in which thread it happened.

chaley · 10-21-2010, 12:42 PM

Quote:

Originally Posted by Manichean

I think that was in response to a bug I filed some time ago. Charles, I believe we had a brief discussion about this, though I can't for the life of me remember in which thread it happened.

You would look for threads for a long time, because it was in a PM.

Starson17 · 10-21-2010, 01:38 PM

Quote:

Originally Posted by Manichean

I think that was in response to a bug I filed some time ago. Charles, I believe we had a brief discussion about this, though I can't for the life of me remember in which thread it happened.

Here's the bug report: http://bugs.calibre-ebook.com/ticket/7147
I didn't notice it was by you, and didn't see it until yesterday. Kovid assigned it to me, but I missed it somehow. It only took a moment to change the code.

Manichean · 10-21-2010, 04:48 PM

The PM explains why I didn't know the thread

Yeah, to summarize: It was a behaviour I stumbled upon while translating and wanted a clarification on to know what exactly to translate. Thanks for fixing, though.

Starson17 · 10-22-2010, 07:55 AM

Quote:

Originally Posted by Manichean

The PM explains why I didn't know the thread

Yeah, to summarize: It was a behaviour I stumbled upon while translating and wanted a clarification on to know what exactly to translate. Thanks for fixing, though.

At several points the code Charles wrote and the code I wrote have similarities (identifying leading articles and identifying similar/identical books being the two most prominent.) This was the first spot where I thought the two could be linked instead of being entirely separate/parallel. Thanks for your bug post.

10-21-2010, 11:50 AM	#1
Starson17 Wizard Posts: 4,004 Karma: 177841 Join Date: Dec 2009 Device: WinMo: IPAQ; Android: HTC HD2, Archos 7o; Java:Gravity T	FYI @Charles: article sort tweak and autosort/automerge A minor update/change: Responding to a bug report/enhancement request, the autosort/automerge option in Adding Books\|"If books with similar author ..found" (used in Adding Books and Copy to Library to identify matching books in the library) now uses Charles' title_sort_articles tweak to identify articles to be ignored during the compare. (Other fuzzy title matching is still done to ignore punctuation, multiple spaces, etc.) The default tweak is: title_sort_articles=r'^(A\|The\|An)\s+' which means the default autosort/automerge behavior is unchanged even if the tweak is undefined. The comparison is done with case ignored. Changing the tweak to title_sort_articles=r'^(A\|The\|An\|XYz)\s+' means that a book with the title "XYZ Circus" will match all these book titles: "Circus" "circus" "A Circus" "a Circus" "the Circus" "The Circus" "An circus" "AN Circus" "xyz Circus" etc., provided the author(s) field also matches.

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
Article tweak for title sort not working	Manichean	Calibre	2	10-04-2010 11:56 AM
Sorting Tweak	atjnjk	Calibre	6	08-31-2010 06:36 AM
Tweak to User Manual	edbro	Calibre	1	07-24-2010 11:26 AM
Firefox Tweak Guide	Colin Dunstan	Lounge	0	07-20-2004 12:01 PM

10-21-2010, 11:56 AM	#2
chaley Grand Sorcerer Posts: 11,742 Karma: 6997045 Join Date: Jan 2010 Location: Notts, England Device: Kobo Libra 2	@Starson17: Is this an FYI? If so, thanks. If it is to point me at a problem, I don't see what the problem is.

10-21-2010, 12:34 PM	#4
Manichean Wizard Posts: 3,130 Karma: 91256 Join Date: Feb 2008 Location: Germany Device: Cybook Gen3	I think that was in response to a bug I filed some time ago. Charles, I believe we had a brief discussion about this, though I can't for the life of me remember in which thread it happened.

10-21-2010, 04:48 PM	#7
Manichean Wizard Posts: 3,130 Karma: 91256 Join Date: Feb 2008 Location: Germany Device: Cybook Gen3	The PM explains why I didn't know the thread Yeah, to summarize: It was a behaviour I stumbled upon while translating and wanted a clarification on to know what exactly to translate. Thanks for fixing, though.

Advert

Advert