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

Paritosh Ranjan commented on MAHOUT-1059:
-----------------------------------------

I thought Mocking was the best way to test DelegatingVector. 

I also think its necessary to test DelegatingVector ( though it looks pretty 
simple ), as there are lots of methods, and it would be really easy to inject 
bug but would be difficult to debug it.

I personally do not believe in creating generalized classes for testing. I 
think class ATest should be used to test A, BTest for B and so on. However, A 
should only test A's code, B should only test B's code and so on. Generalizing 
a class for writing test cases for a particular genre makes it difficult for a 
new guy to find the test cases.

However, in the end, what actually matters is that the code should work, so, 
all approaches are fine.
                
> New matrix extensions
> ---------------------
>
>                 Key: MAHOUT-1059
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-1059
>             Project: Mahout
>          Issue Type: Improvement
>          Components: Math
>            Reporter: Ted Dunning
>             Fix For: 0.8
>
>         Attachments: 
> 0001-MAHOUT-1059-Added-Centroid-WeightedVector-Delegating.patch, 
> 0001-MAHOUT-1059-Added-Centroid-WeightedVector-Delegating.patch, 
> 0002-MAHOUT-1059-Stylistic-cleanups.patch, DelegatingVectorTest.java
>
>
> The upcoming clustering needs several capabilities to support different 
> operations.  These include some matrix extensions for adding behaviors to 
> different kinds of matrices.  Also there is a file based matrix that uses 
> mmap to access a file as if it were a matrix in shared memory.  Since this is 
> off-heap and shared between processes, it can seriously help some programs.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to