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

add gitignore, add requirements, correct format, add main, add modes … #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

diazcelsa
Copy link

…for show text with sentiment, pickle sentiment of a text, pickle sentiment of a list of texts

The changes are mainly:

  • Add requirements with package versions
  • Gitignore to avoid editor configuration and cache files
  • Add a main in visualize.py with 3 modes: current one "show" and two more for storing the sentiment of one text or a list of texts
  • pep-8 formating of the code to be more readable

…for show text with sentiment, pickle sentiment of a text, pickle sentiment of a list of texts
@diazcelsa
Copy link
Author

I evaluated your pre-trained model with my data set and with several sentences from Sutskever et al. 2017 and the sentiment looks coherent. You can have a look:
https://github.com/diazcelsa/NLP_politics_spiegel/blob/master/notebooks/sentiment/Evaluate%20Sentiment%20of%20migration-related%20selected%20articles.ipynb

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.

1 participant