-
Notifications
You must be signed in to change notification settings - Fork 72
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
score zero to any non-english sentence #26
Comments
This is the dev version on GitHub and we're still working to test all the various languages that are available via the expanded NRC lexicon. So far Spanish and German have been tested for functionality. Will keep this issue open until portuguese has been tested. Thanks. |
Hi Matthew - I'm happy to take up this issue, and continue testing in other languages. Is there a standardised way of running tests for this project that I should use? If not, I'll document my approach. I'm very new to contributing to Open Source, so do let me know if there are things that I have missed / OSS etiquette. Thanks |
Thanks codellama23. If you have fluency/expertise in another language and would like to test it, that would be great. I don't have a standardized test (though that is a good idea), so the way we have been testing the output is by having native speakers process a fairly well known text (i.e. Don Quixote was used for Spanish and Kafka's Metamorphoses for German) and then scrutinize the results to see if the program's output aligns with your expectations of the sentiment as a native speaker of the language. |
Hi @hichemmkhalyd, I was unable to recreate your error. I used the text of Os Maias by Eça de Queirós. os_maias_text.txt Here is the code I used:
And here is the output I get:
Could you post the code you used that caused your error? Thanks :) |
I tried to use get_nrc_sentiment with lang = "portuguese" and NRC method but it aways returns zero for all sentiments. I was wondering if I shoud do something more to use it.
The text was updated successfully, but these errors were encountered: