Suneel Marthi created SAMOA-14:
----------------------------------

             Summary: 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