+1 It was never intended as a human readable format and was just barely readable before sparse vectors matured. It should be replaced with VectorHelper.vectorToString with an optional bindings arg.

I also think asJsonString() would be useful to preserve the existing semantics.


Jake Mannix wrote:
asFormatString() is horrible.  It prints out stuff which allows you to
barely
figure out what is in the vector is, since both sparse vector impls have
weird internal structures.  I much prefer the VectorHelper.vectorToString()
method (with possibly a null dictionary) - it's actually readable.

  -jake

On Mon, Mar 1, 2010 at 1:36 PM, Robin Anil <robin.a...@gmail.com> wrote:

It already does this, i think. But floats can be formatted better

On Tue, Mar 2, 2010 at 2:55 AM, Jeff Eastman <j...@windwardsolutions.com
wrote:
And check the asFormatString(bindings) implementation in ClusterBase. It
does this I think, though it has not yet been wired into
ClusterDumper.printClusters.  I wanted to give the ClusterDumper users a
chance to critique my formatting but it is like the below.

Jeff



Jake Mannix (JIRA) wrote:

VectorDumper should also do printing to simple {index : value, index :
value, ... } output, if no dictionary is specified.


--------------------------------------------------------------------------------------------------------------------------
                Key: MAHOUT-315
                URL: https://issues.apache.org/jira/browse/MAHOUT-315
            Project: Mahout
         Issue Type: Improvement
   Affects Versions: 0.2
           Reporter: Jake Mannix
           Assignee: Jake Mannix
            Fix For: 0.4


I've got a patch for this, tied up in other code.




Reply via email to