Skip to content
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

Missing dependencies (requests) #51

Closed
martin-martin opened this issue Aug 7, 2018 · 3 comments · Fixed by #83
Closed

Missing dependencies (requests) #51

martin-martin opened this issue Aug 7, 2018 · 3 comments · Fixed by #83

Comments

@martin-martin
Copy link

Hello!
When attempting to run vaderSentiment after installing it in my venv with the suggested pip install vaderSentiment, I got the following error:

Traceback (most recent call last):
  File "rate.py", line 1, in <module>
    from vaderSentiment.vaderSentiment import SentimentIntensityAnalyzer
  File "/Users/martin/Documents/codingnomads/nlpython/big_projects/rate-mds/env/lib/python3.6/site-packages/vaderSentiment/vaderSentiment.py", line 17, in <module>
    import requests
ModuleNotFoundError: No module named 'requests'

Seems that requests is an implicit(?) dependency.
I looked for a requirements.txt but not sure that's how it works with packages up on PyPI.

My pip freeze suggests that a requirements.txt for vaderSentiment could look like this:

certifi==2018.4.16
chardet==3.0.4
idna==2.7
requests==2.19.1
urllib3==1.23

If this is helpful, I can create one and make a PR? But maybe there are other reasons for this not to be included that I am not aware of.

Let me know if this is something I can help with - would gladly do so :)

@ianthebeard
Copy link

I have the same issue. I can't get vaderSentiment.py to run at all. Several errors, and being new to Python, I have no idea how to troubleshoot.

@martin-martin
Copy link
Author

Hi @ianthebeard, you can type pip install requests in your console to resolve this issue.

Then run VADER again. Hope this helps!

@ianthebeard
Copy link

ianthebeard commented Aug 12, 2018 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants