mahout-user mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sean Owen <sro...@gmail.com>
Subject Re: Problems installing Mahout
Date Tue, 06 Apr 2010 08:34:03 GMT
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