Fixed PorterStemmer to work with the word "ions".

EdwardRaff / JSAT

Java Statistical Analysis Tool, a Java library for Machine Learning

GNU General Public License v3.0

789 stars 205 forks source link

Closed Pedromdrp closed 8 years ago

Pedromdrp commented 8 years ago

Fix to use PorterStemmer with the string "ions".

EdwardRaff commented 8 years ago

Was this for an index out of bounds error or was there a word that wasn't stemmed when it should have been?

I'd appreciate if you added the test case that gets fixed to JSAT/test/jsat/text/stemming/PorterStemmerTest.java

Pedromdrp commented 8 years ago

It's an out of bounds error. I am going to add the test case then.

EdwardRaff commented 8 years ago

Looks, good. Thanks for the bug catch and work! Hope JSAT has otherwise been helpful!