You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mahout.apache.org by "Jeff Eastman (JIRA)" <ji...@apache.org> on 2008/03/06 18:56:58 UTC

[jira] Commented: (MAHOUT-12) Point formatting and parsing improved (StringBuilder, no need for trailing comma).

    [ https://issues.apache.org/jira/browse/MAHOUT-12?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12575809#action_12575809 ] 

Jeff Eastman commented on MAHOUT-12:
------------------------------------

The main reason I put in the trailing comma (and also aleading comma after [ (not yet in trunk)) is so that it is easy to slurp the resulting data into Excel spreadsheets. Without the extra delimiters, the [] characters mix with the data values and manual editing is required. I think this is a valid use case that needs to be supported.

> Point formatting and parsing improved (StringBuilder, no need for trailing comma).
> ----------------------------------------------------------------------------------
>
>                 Key: MAHOUT-12
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-12
>             Project: Mahout
>          Issue Type: Improvement
>          Components: Clustering
>    Affects Versions: 0.1
>            Reporter: Dawid Weiss
>            Assignee: Dawid Weiss
>            Priority: Trivial
>         Attachments: mah-12.patch
>
>
> Added test case to point class, improved parsing (no need to recompile the pattern all over again) and concatenation of points (stringbuilder used internally).

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Re: [jira] Commented: (MAHOUT-12) Point formatting and parsing improved (StringBuilder, no need for trailing comma).

Posted by Ted Dunning <td...@veoh.com>.
Just define all of [], as delimiter characters and collapse repeats.  Works
like a charm.  No leading and trailing needed.


On 3/6/08 9:56 AM, "Jeff Eastman (JIRA)" <ji...@apache.org> wrote:

> 
>     [ 
> https://issues.apache.org/jira/browse/MAHOUT-12?page=com.atlassian.jira.plugin
> .system.issuetabpanels:comment-tabpanel&focusedCommentId=12575809#action_12575
> 809 ] 
> 
> Jeff Eastman commented on MAHOUT-12:
> ------------------------------------
> 
> The main reason I put in the trailing comma (and also aleading comma after [
> (not yet in trunk)) is so that it is easy to slurp the resulting data into
> Excel spreadsheets. Without the extra delimiters, the [] characters mix with
> the data values and manual editing is required. I think this is a valid use
> case that needs to be supported.
> 
>> Point formatting and parsing improved (StringBuilder, no need for trailing
>> comma).
>> -----------------------------------------------------------------------------
>> -----
>> 
>>                 Key: MAHOUT-12
>>                 URL: https://issues.apache.org/jira/browse/MAHOUT-12
>>             Project: Mahout
>>          Issue Type: Improvement
>>          Components: Clustering
>>    Affects Versions: 0.1
>>            Reporter: Dawid Weiss
>>            Assignee: Dawid Weiss
>>            Priority: Trivial
>>         Attachments: mah-12.patch
>> 
>> 
>> Added test case to point class, improved parsing (no need to recompile the
>> pattern all over again) and concatenation of points (stringbuilder used
>> internally).