Kaggle / docker-python

Kaggle Python docker image
Apache License 2.0
2.45k stars 948 forks source link

Add vaderSentiment package #457

Closed NazarioR9 closed 5 years ago

NazarioR9 commented 5 years ago

Hello Kaggle Team,

I've installed 'vaderSentiment' on my custom docker. But I could'nt use it because it was't allowed in the competion. I also tried using it via 'NLTK' (by using "import nltk.sentiment.vader" ) but it failed to load it. So can you please add it (It's a sentiment analysis library). Thanks

rosbo commented 5 years ago

Hi @NazarioR9,

Given that this is a very popular package (1000+ starts on GitHub) we would accept a PR adding this package with some tests. See instructions here: https://github.com/Kaggle/docker-python#opening-a-pull-request

You can tag me in the PR and I will make sure to review it promptly.

Thank you

rosbo commented 5 years ago

If you are still interested in this package, feel free to reopen this issue and follow the instructions I have posted above.

Thank you