Added italian porter stemmer #80

Merged
merged 1 commit into from Dec 7, 2012

Conversation

Projects
None yet
2 participants
Contributor

genialeo commented Dec 5, 2012

My small contribution to a great project.

Owner

chrisumbel commented Dec 7, 2012

Thank you!

@chrisumbel chrisumbel added a commit that referenced this pull request Dec 7, 2012

@chrisumbel chrisumbel Merge pull request #80 from leofenu/italian_stemmer
Added italian porter stemmer
7b8a7c2

@chrisumbel chrisumbel merged commit 7b8a7c2 into NaturalNode:master Dec 7, 2012

Owner

chrisumbel commented on 31afbae Dec 7, 2012

I didn't poke around but I get some test failures:

  1. should tokenize and stem attached
    Message:
    Expected [ 'sopr', 'la', 'panc', 'la', 'capr', 'camp' ] to equal [ 'sopr', 'panc', 'capr', 'camp' ].

  2. should tokenize and stem attached
    Message:
    Expected [ 'sotto', 'la', 'panc', 'la', 'capr', 'crep' ] to equal [ 'sott', 'panc', 'capr', 'crep' ].

Contributor

genialeo replied Dec 7, 2012

Owner

chrisumbel replied Dec 7, 2012

Ah, I got it fixed. It was the test that was goofed, not the stemmer. I'll explain in the commit.

Thanks again for your contribution! I'm very excited about it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment