[ 
https://issues.apache.org/jira/browse/SAMOA-27?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14539560#comment-14539560
 ] 

ASF GitHub Bot commented on SAMOA-27:
-------------------------------------

Github user gdfm commented on the pull request:

    https://github.com/apache/incubator-samoa/pull/25#issuecomment-101205275
  
    All the tests pass.
    Tested various runs of Bagging, VHT, and AMRules on Storm local model and 
they all work.
    Also ran the ClusteringEvaluation. Seems to work but it's hard to evaluate 
:)
    
    In the samoa script, given that now storm-core.jar is in the /lib directory 
of storm, we can simplify the line creating the class path by removing the 
STORM_JAR entry.
    
    +1


> Upgrade Storm version to 0.9
> ----------------------------
>
>                 Key: SAMOA-27
>                 URL: https://issues.apache.org/jira/browse/SAMOA-27
>             Project: SAMOA
>          Issue Type: Task
>          Components: SAMOA-Storm
>            Reporter: Gianmarco De Francisci Morales
>
> From GitHub:
> Storm, and consequently Zookeeper and Kryo, have been updated.
> Samoa-storm compiles and passes the tests; the example of CoverType dataset 
> (classified with the bagging algorithm) runs without problems.
> Also the code of my fork with a word2vec implementation runs correctly.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to