Deal with warning about slow fuzzy matching

declension / squeeze-alexa

Squeezebox integration for Amazon Alexa

GNU General Public License v3.0

59 stars 20 forks source link

Deal with warning about slow fuzzy matching #105

Closed declension closed 5 years ago

declension commented 5 years ago

Steps

Run (cold) in production AWS (or even just run a local build & test)

Expected

No warnings

Actual

-python SequenceMatcher. Install python-Levenshtein to remove this warning                                                                  
    warnings.warn('Using slow pure-python SequenceMatcher. Install python-Levenshtein to remove this warning')                              

-- Docs: https://docs.pytest.org/en/latest/warnings.html

See https://github.com/seatgeek/fuzzywuzzy/issues/67

declension commented 5 years ago

Actually adding it is one option, but:

it comes at > 500k of weight itself (including C source code, annoyingly)

Pulls in dist_tools:

Collecting python-levenshtein==0.12.0 (from -r /home/nick/workspace/squeeze-alexa/requirements.txt (line 15))                                                    
Collecting setuptools (from python-levenshtein==0.12.0->-r /home/nick/workspace/squeeze-alexa/requirements.txt (line 15))

This makes the deployment zips ~150 files not ~47 !