Skip to content

Clean up#47

Merged
marcosmilzo merged 11 commits into
devfrom
clean_up
Nov 26, 2017
Merged

Clean up#47
marcosmilzo merged 11 commits into
devfrom
clean_up

Conversation

@mkao006

@mkao006 mkao006 commented Nov 21, 2017

Copy link
Copy Markdown
Contributor

A simple clean up for deployment.

@marcosmilzo marcosmilzo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's what they call a real clean up!

analyzer = SentimentIntensityAnalyzer()
sentiment = analyzer.polarity_scores(sentence)
return sentiment
from vaderSentiment.vaderSentiment import SentimentIntensityAnalyzer

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Neat!

Comment thread requirements.txt
@@ -100,6 +100,7 @@ Twisted==17.5.0
twitter==1.17.1
Unidecode==0.4.21
urllib3==1.22

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we double check all the packages to remove any useless one?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, good idea. will do that!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just checked, the twitter package is the only one that is not required......

The remaining list is the bare minimum packages required for the reading machine
@marcosmilzo marcosmilzo merged commit 65d2edb into dev Nov 26, 2017
@mkao006 mkao006 deleted the clean_up branch November 27, 2017 14:56
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 this pull request may close these issues.

2 participants