Mailing-List: contact mahout-dev-help@lucene.apache.org; run by ezmlm
Precedence: bulk
Reply-To: mahout-dev@lucene.apache.org
Received-SPF: pass (athena.apache.org: domain of robin.anil@gmail.com
 designates 209.85.216.176 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:in-reply-to:references:from:date:message-id:subject:to
         :content-type;
        b=nu1rirVT0IqiBnfdiFUL+D2egCnv5UndVh4FauGdCIid0UnyPuS4ErvxtYa1zWslKt
         ZUr1Ud6Q0Ow5TnsJoe+ikpWdkobS8Mc5VtFv9hKJxTeaoe3NuYQPpvNyV5Jesdh/0GZp
         QD6nVe/YgrspaNXLZx+OFu7fQvRdoZiQ+9Lg8=
MIME-Version: 1.0
In-Reply-To: <4B7ACE0A.4010902@windwardsolutions.com>
References: <7d7600c51002160538od66b799o7235ecab2869b542@mail.gmail.com>
	<4B7ACE0A.4010902@windwardsolutions.com>
From: Robin Anil <robin.anil@gmail.com>
Date: Tue, 16 Feb 2010 22:28:34 +0530
Message-ID: <7d7600c51002160858p61e1fd83kbc6b1abc27a65d50@mail.gmail.com>
Subject: Re: Fuzzy K Means
To: mahout-dev@lucene.apache.org
Content-Type: multipart/alternative; boundary=0016369207c618a2a3047fbaa6fd

--0016369207c618a2a3047fbaa6fd
Content-Type: text/plain; charset=UTF-8

On Tue, Feb 16, 2010 at 10:25 PM, Jeff Eastman
<jdog@windwardsolutions.com>wrote:

> Looks to me like the unit tests are the only calls to recomputeCenter,
> which is where the center is set. The clusterer seems to be calling
> computeCentroid, which sets the centroid, instead. I'm not sure why it needs
> both instance variables, as the pointProbSum and weightedPointTotal
> variables take the place of the single pointTotal in ClusterBase. I think
> perhaps center and centroid need to be merged?
>
> In k-means and canopy, the center is the (read-only) current centroid which
> is used for the distance calculations during an iteration, and it is
> recomputed by computeCentroid (using pointTotal and numPoints) at the end of
> the iteration.
>
> So just writing computeCentroid should do right? Which is what its doing.

@Override
  public void write(DataOutput out) throws IOException {
    out.writeInt(clusterId);
    out.writeBoolean(converged);
    Vector vector = computeCentroid();
    VectorWritable.writeVector(out, vector);
  }

  @Override
  public void readFields(DataInput in) throws IOException {
    clusterId = in.readInt();
    converged = in.readBoolean();
    VectorWritable temp = new VectorWritable();
    temp.readFields(in);
    setCenter(temp.get());
    this.pointProbSum = 0;
    this.weightedPointTotal = getCenter().like();
  }

 Jeff
>
>
>
> Robin Anil wrote:
>
>> I have been trying to convert FuzzyKMeans SoftCluster(which should be
>> ideally be named FuzzyKmeansCluster) to use the ClusterBase.
>>
>> I am getting* the same center* for all the clusters. To aid the conversion
>> all i did was remove the center vector from the SoftCluster class and
>> reuse
>> the same from the ClusterBase. These are essentially making no change in
>> the
>> tests which passes correctly.
>>
>> So I am questioning whether the implementation keeps the average center at
>> all ? Anyone who has used FuzzyKMeans experiencing this?
>>
>>
>> Robin
>>
>>
>>
>
>

--0016369207c618a2a3047fbaa6fd--