You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mahout.apache.org by "Karl Wettin (JIRA)" <ji...@apache.org> on 2008/04/10 02:58:04 UTC
[jira] Updated: (MAHOUT-33) Matrix tests share code that can be
placed in an abstract class
[ https://issues.apache.org/jira/browse/MAHOUT-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Karl Wettin updated MAHOUT-33:
------------------------------
Attachment: MAHOUT-33.patch
This was spawned by MAHOUT-26 and by mistake this patch also contains the contents of that patch. I did not mean to do that, but there you go. I'm ready to commit that too.
I did not check everything with my eyes, it was too much but everything pass though.
> Matrix tests share code that can be placed in an abstract class
> ---------------------------------------------------------------
>
> Key: MAHOUT-33
> URL: https://issues.apache.org/jira/browse/MAHOUT-33
> Project: Mahout
> Issue Type: Improvement
> Reporter: Karl Wettin
> Assignee: Karl Wettin
> Attachments: MAHOUT-33.patch
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
Re: [jira] Updated: (MAHOUT-33) Matrix tests share code that can
be placed in an abstract class
Posted by Jeff Eastman <je...@windwardsolutions.com>.
Karl Wettin (JIRA) wrote:
> [ https://issues.apache.org/jira/browse/MAHOUT-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
>
> Karl Wettin updated MAHOUT-33:
> ------------------------------
>
> Attachment: MAHOUT-33.patch
>
> This was spawned by MAHOUT-26 and by mistake this patch also contains the contents of that patch. I did not mean to do that, but there you go. I'm ready to commit that too.
>
> I did not check everything with my eyes, it was too much but everything pass though.
>
>
>
>
>
>> Matrix tests share code that can be placed in an abstract class
>> ---------------------------------------------------------------
>>
>> Key: MAHOUT-33
>> URL: https://issues.apache.org/jira/browse/MAHOUT-33
>> Project: Mahout
>> Issue Type: Improvement
>> Reporter: Karl Wettin
>> Assignee: Karl Wettin
>> Attachments: MAHOUT-33.patch
>>
>>
>>
>
>
>
-1 It occurred to me as I was copying and pasting the unit tests that
many of the methods could indeed be refactored. Unfortunately, you must
have forgotten to svn add the new MatrixTest superclass before you
diffed <grin>. I do that all the time myself.