[
https://issues.apache.org/jira/browse/SAMOA-14?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15192207#comment-15192207
]
ASF GitHub Bot commented on SAMOA-14:
-------------------------------------
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-samoa/pull/51
> 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)
