Closed ruebot closed 5 years ago
Since we have some discussion going on #13, we'll need to make sure we don't mark it as resolved in the commit message. Just partially addressed, since we're actually citing it now.
Commit message:
Title: Use NLTK stopwords, update README (#15)
Body:
- Resolve #14
- Partially address #13
- Resolve #17
- Update notebooks to use NLTK stopwords
- Add NLTK stopwords
Works great. One reminder is that if someone has a notebook up, the docker instructions may not show the correct output. Something to consider for the README.md.
@ianmilligan1 once dockercloud finishes, this should be good to merge with the commit message above.
Once that's done, I'm jump on some of the other open issues.
DO NOT MERGE UNTIL #12 is merged.