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.
administrator2012-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
administrator2012-08-28 10:27
Professor,
I have made the changes in documentation and the tests are now run on a larger vocabulary.
administrator2012-08-28 10:29
the revised patch still has tabs everywhere. you haven't phpdoc'd your code as per (5) above.
administrator2012-09-14 19:38
September patch applied 62a5d9d.
Good work Akshat!
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.
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
/**
(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
Professor,
I have made the changes in documentation and the tests are now run on a larger vocabulary.
the revised patch still has tabs everywhere. you haven't phpdoc'd your code as per (5) above.
September patch applied 62a5d9d. Good work Akshat!
Attachments:
italian_stemmer.patch
italian_stemmer_Aug312012.patch
italian_stemmer_Sep132012
italian_stemmer_revised.patch
italian_stemmer_Sep132012.patch