mahout-user mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeff Eastman <j...@windwardsolutions.com>
Subject Re: Problems installing Mahout
Date Tue, 06 Apr 2010 16:42:38 GMT
+1 With all the formatted output needing to be checked this seems to be 
the lowest-hanging-fruit.

Sean Owen wrote:
> I'm ready to patch this issue, but I went the other way -- fixed the
> output to use Locale.ENGLISH.
>
> Either way works, what's preferred to you guys? Is it making the
> output deterministic, or locale-friendly? I opted for fixing it to
> Locale.ENGLISH because I like not depending on the platform, and
> because the project is hardly internationalized to begin with.
>
> On Tue, Apr 6, 2010 at 12:22 AM, Jeff Eastman
> <jdog@windwardsolutions.com> wrote:
>   
>> We've been seeing a lot of similar string comparison problems recently and
>> have made some progress in minimizing them. Its ironic that this problem is
>> in the Printable tests which were supposed to be an improvement to the
>> situation<grin>. The NormalModel asFormatString() uses
>> ClusterBase.formatVector() which itself uses String.format(). In this case I
>> think I would choose to fix the tests since the formatString *should* track
>> the local language settings as its intended to be user readable.
>>
>> I suggest deleting the test class until I can get a patch in for the tests,
>> or just run mvn install -DskipTests=true. Actually, it would be most useful
>> to delete the class and see if there are any other tests like that one to
>> bite us.
>>     
>
>   


Mime
View raw message