Skip to content

Added a little bit of language options (just French) and matrix output#5

Open
psorianom wants to merge 2 commits intojpotts18:masterfrom
psorianom:master
Open

Added a little bit of language options (just French) and matrix output#5
psorianom wants to merge 2 commits intojpotts18:masterfrom
psorianom:master

Conversation

@psorianom
Copy link

Hey man!
Thanks for the library. I forked it because I wanted two things extra that your code did not had:

  1. Handle languages from the constructor and be able to have a list of words (like the English ones) in French. I thought that it would be interesting to have stopwords as these words with the translation of the default words (and, but, if, that, etc) in English. This of course strays away from the paper you based the library on.
  2. I wanted access to the feature matrix created. Maybe this was implemented but I could not find it. I find it handy that you have your own sklearn methods but maybe people want just to get access to the feature data matrix.
  3. I also put the English words in a list and I changed some things related to that...

Cheers!
Pavel

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