-
Notifications
You must be signed in to change notification settings - Fork 380
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add a requirements.txt #3
Comments
I'm similarly having trouble with the edit: aha, this seems to imply python3. so it would be good to note that. update: code now works fine for me with Python 3.5.3 and tensorflow (0.12.1). |
Using HTMLParser doesn't work exactly like you would want it to - I think you found this in #2 as well. I'm not sure what the way to fix it is in python 2 is, though. |
Use Added requirements.txt in this PR. #43 |
This should work well with python 3.6, but not python 2.7. I did a mistake with html.unescape(). Because I use python 2.7, I did change the code to HTMLParser.HTMLParser().unescape(text) After I import HTMLParser This turned out to be a bad idea. So again, my recommendation is to use python 3.6 to run this amazing work. Hope this helps other people who have this problem as well. |
Getting a weird module version issue with tensorflow and the method 'unpack' not being an attribute. Would be helpful if a requirements.txt file was included to use with virtualenv.
The text was updated successfully, but these errors were encountered: