« Back Issue #101
  • 2012-08-14 18:16 Reported by akshat.kukreti
  • 2012-09-14 19:38 Marked fixed
Status: Marked fixed
Priority: Low

Stemmer for Italian language

akshat.kukreti 2012-08-14 18:16

An implementation of the snowball stemmer for Italian language in PHP. The implementation follows the rules given in snowball.

Additional information: I would like my code to be considered for the CS297 coding challenge as well as for being added to Yioop.

administrator 2012-08-17 14:01

Hey Akshat,

Thanks for the initial submission. Here is what you need to correct to get it accepted:

(1)There are a lot of tab characters in the patch rather than 4 spaces in your patch. There is also a lot of stray whitespace. Please remove these.

(2) You need to have better class level phpdocs on what your stemmer does. Say that you are implementing this Italian stemmer based on where you found it, etc.

(3) If you are making phpdoc's make sure the comment begins: /** not /*

(4) Make sure that the php function array is lower case not Array

(5) Don't repeat the method name in your phpdoc for a method. i.e., rather than: /*Function stem computes the ...

instead

/**

*  Computes the ...

(6) Class level variables (fields and static fields) should be preceded by a phpdoc giving description and type.

(7) You need to have you test cases run against the larger vocabulary on the tatarus site: http://snowball.tartarus.org/algorithms/italian/voc.txt

Best, Chris

administrator 2012-08-28 10:27

Professor,

I have made the changes in documentation and the tests are now run on a larger vocabulary.

administrator 2012-08-28 10:29

the revised patch still has tabs everywhere. you haven't phpdoc'd your code as per (5) above.

administrator 2012-09-14 19:38

September patch applied 62a5d9d. Good work Akshat!

X