search

philipperemy / tensorflow-1.4-billion-password-analysis

Deep Learning model to analyze a large corpus of clear text passwords.
1.91k stars 394 forks source link

MacOS - slugify NameError: name 'unicode' is not defined #3

Closed mveplus closed 6 years ago

mveplus commented 6 years ago

Thank you philipperemy, This is a great tool for analyzing how password changes over time!

I was having some issues because when I jumped on to it I did not read the dependency requirements.txt and I've installed just slugify module not uni-slugify: so running it would yield: python3 read.py --breach_compilation_folder BreachCompilation

Found 2035 files 0%| | 0/2035 [00:00<?, ?it/s] Traceback (most recent call last): File "read.py", line 36, in read() File "read.py", line 32, in read on_file_read_call_back_class=ReducePasswordsOnSimilarEmailsCallback) File "/Users/banan/Downloads/BreachCompilation/tensorflow-1.4-billion-password-analysis/utils.py", line 37, in read_n_files suffix = slugify(current_filename.split('data')[-1]) File "/usr/local/lib/python3.6/site-packages/slugify.py", line 24, in slugify unicode( NameError: name 'unicode' is not defined

Uninstalling slugify and installing uni-slugify solved the issue! It took about 2 hours on i7(2.7GHz 13-inch, Early 2011 with SSD) This is just to say thank you really and for others chaps like me that will bang their head around and wonder why is not working, just RTFM ;)

philipperemy commented 6 years ago

@mveplus hey thanks for the help and please feel to contribute :)

For your problem I guess you installed slugify while the relevant package is called python-slugify. One is not compatible with python3.

pip3 uninstall slugify
pip3 install python-slugify

Should work for you!