[ 
https://issues.apache.org/jira/browse/SAMOA-14?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Gianmarco De Francisci Morales updated SAMOA-14:
------------------------------------------------
        Status: Done  (was: To Do)
    Resolution: Done

> Disambiguate ARFFFileStream
> ---------------------------
>
>                 Key: SAMOA-14
>                 URL: https://issues.apache.org/jira/browse/SAMOA-14
>             Project: SAMOA
>          Issue Type: Task
>          Components: SAMOA-API
>            Reporter: Suneel Marthi
>
> There are presently 2 ARFFFileStream classes
> {Code}
> com.yahoo.labs.samoa.streams.ARFFFileStream
> com.yahoo.labs.samoa.moa.streams.ARFFileStream
> {Code}
> Pasting here Gianmarco's comments:
> -----------------------------------------------------------
> At least one of the two is being used, even though not statically from the 
> codebase. When you launch SAMOA from the command line, e.g.:
> bin/samoa local target/SAMOA-Local-0.0.1-SNAPSHOT.jar "PrequentialEvaluation 
> -l classifiers.ensemble.Bagging -s (ArffFileStream -f covtypeNorm.arff) -f 
> 100000"
> you are instantiating an InstanceStream (the -s option).
> Given that InstanceStream is in com.yahoo.labs.samoa.moa.streams, right now 
> we are using com.yahoo.labs.samoa.moa.streams.ArffFileStream.
> However com.yahoo.labs.samoa.stream.ArffFileStream is the new one with the 
> new class architecture. This ArffFileStream extends FileStream and should 
> allow to read several file formats.
> So I guess we should keep only this latter one and make sure it works.



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

Reply via email to