fuzzywuzzy-rs
fuzzywuzzy-rs copied to clipboard
process::extract_without_order must avoid re-invoking processor methods
See the original implementation where the processor function is only executed once if it's a known method.
I'm not sure if this is an optimization to prevent duplicate calls, or if multiple invocations could lead to some undesired state? i.e. the processor function is not idempotent.
wratio -> wratio and wratio_calc like ratio_full and simple_calc
same with partial_ratio_full for that matter
basically the current impl of extract_without_order doesn't double call the processors, but if the scorer calls processing, you're hosed. It just so happens the original scorers by default do because we havent split them out as concepts.
Does #28 split them out as concepts?
not quite, no. You have things like ratio_full which does processor and delegates to simple_ratio which could be considered a scorer, but that structure is not applied to, say, partial_ratio_full which would need to split out most of its body into a best_partial_ratio or something.