My favorites | Sign in
Project Home Wiki Issues Source
READ-ONLY: This project has been archived. For more information see this post.
Search
for
  Advanced search   Search tips   Subscriptions
Issue 110: Analytics: Code Review
1 person starred this issue and may be notified of changes. Back to list
 
Project Member Reported by kumarami...@gmail.com, May 2, 2010
I am making a list of issues related to the analytics code -need to address
these when Kris returns

1. FriedmansAnovaTkHsd -what's this class for? Can it be retired.
2. WriteCsvResultFile.java -Need to change this class name move the
functionality to utils
3. CopyFileFromClassPathToDisk.java Rename class -move functionality to a
util class
4. Move NemaFileType and nemaFileTypeImpl, OpaqueFileFormat and
RawAudioFile from the util package These are not util classes.
5. What are the classes in the util.process package, seem like either the
package is a misnomer or has redundant code or both...





May 4, 2010
Project Member #1 kris.west@gmail.com
1. FriedmansAnovaTkHsd - performs all our stats tests (Friedmans ANOVA with Tk HSD 
using matlab. It needs to be replaced with a java implementation, however, this is 
not a trivial job.

2. WriteCsvResultFile.java - Yes, change name as it creates tabular data that then be 
written to either an HTML table or CSV file. Is used in every evaluator (and only in 
evaluators) to prep result data.

3. CopyFileFromClassPathToDisk.java - Move to an IO class?

4. Move NemaFileType and nemaFileTypeImpl, OpaqueFileFormat and
RawAudioFile from the util package These are not util classes.
- agreed

5. What are the classes in the util.process package, seem like either the
package is a misnomer or has redundant code or both...
- all of these are classes related to setting up (remote) processes, 
encoding/decoding command formatting strings etc. MatlabExecutorImpl runs matlab 
processes and is used to run the stats tests
May 4, 2010
Project Member #2 kumarami...@gmail.com
Hey Kris -Stop working on your vacation!!! Enjoy Mexico :)

I created a 0.3.0-BRANCH that has the modifications to the package structure. Still
working on it.

Amit

Powered by Google Project Hosting