MobileRead Forums - View Single Post

kiwidude · 10-12-2022, 04:17 PM

@TheMadGuy - Thanks for providing those examples. It has been a very long time since I have looked at the code in this plugin for the algorithms myself.

Are you talking about using the "Find Metadata Variations" for these Authors? From my test with it now:

Similar - finds the Biggle (jr.), Lloyd case - no surprise as it strips out common author words like "jr", "sr", "van" etc.
Soundex (length 8) - also finds the Pohl, Frederic/k case.
Fuzzy - all except the Hohlbein comma space case

I agree it should be finding that Hohlbein spacing issue so I can look to tweak things for that. I will also have to remind myself of exactly what all those algorithms are doing under the hood as to whether anything else should be tweaked.

EDIT: I can see why that Hohlbein case is going wrong, that is definitely something I will fix for the next version. It is stripping out the "," before it starts breaking the author name into space delimited parts, an easy fix. Thanks for reporting this!

10-12-2022, 04:17 PM	#987
kiwidude Calibre Plugins Developer Posts: 4,733 Karma: 2197770 Join Date: Oct 2010 Location: Australia Device: Kindle Oasis	@TheMadGuy - Thanks for providing those examples. It has been a very long time since I have looked at the code in this plugin for the algorithms myself. Are you talking about using the "Find Metadata Variations" for these Authors? From my test with it now: Similar - finds the Biggle (jr.), Lloyd case - no surprise as it strips out common author words like "jr", "sr", "van" etc. Soundex (length 8) - also finds the Pohl, Frederic/k case. Fuzzy - all except the Hohlbein comma space case I agree it should be finding that Hohlbein spacing issue so I can look to tweak things for that. I will also have to remind myself of exactly what all those algorithms are doing under the hood as to whether anything else should be tweaked. EDIT: I can see why that Hohlbein case is going wrong, that is definitely something I will fix for the next version. It is stripping out the "," before it starts breaking the author name into space delimited parts, an easy fix. Thanks for reporting this! Last edited by kiwidude; 10-12-2022 at 04:38 PM.