Nicolas Santini

Still nothing. Not only that. By opening the reply from the notifications page, and replying to it, I have branched the thread. See screenshots from Planetary and Perihelion Original message...

Hi, I took the liberty of forking this great repo to add some features I needed, and they go inline with whats described in this issue. I added node-spellchecker to...

PR https://github.com/thisandagain/sentiment/pull/144 It has some merge conflicts. I'll try go solve them, but feel free to take over, I havent looked into this for a while

solved the merge conflicts, but somebody more familiar with the changes that happened since I forked need to validate them :)

So, the sync_negation test case is failing. Looks like since I forked the logic to deal with negation of sentence has changed, so my change is double negating the score....

Ok - moved the negation backwards strategy to the language model - added unit test for it

Finally, made spell checking optional, and unit tested it

The current implementation uses https://www.npmjs.com/package/spellchecker , which are native bindings to NSSpellChecker, Hunspell, or the Windows 8 Spell Check API, depending on your platform. Windows 7 and below as well...

There is a mechanism to add a dictionary to the spellchecker, I believe thats how a language gets "set"

I have refactored the spell checking functionality to use a different library [nspell](https://www.npmjs.com/package/nspell) that can support multiple languages. Also updated README with examples of how to use spell checking