You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mahout.apache.org by "Mat Kelcey (Created) (JIRA)" <ji...@apache.org> on 2011/12/23 07:33:32 UTC

[jira] [Created] (MAHOUT-936) Mismatch in case for arg handling in org.apache.mahout.vectorizer.collocations.llr.CollocDriver

Mismatch in case for arg handling in org.apache.mahout.vectorizer.collocations.llr.CollocDriver
-----------------------------------------------------------------------------------------------

                 Key: MAHOUT-936
                 URL: https://issues.apache.org/jira/browse/MAHOUT-936
             Project: Mahout
          Issue Type: Bug
          Components: Clustering
    Affects Versions: 0.6
            Reporter: Mat Kelcey
            Priority: Minor


the option minSupport is added with a capital S
   addOption("minSupport", "s", "(Optional) Minimum Support. Default Value: "

but the actual value retrieved from the arg map has a lower case s
   if (argMap.get("--minsupport") != null) {

Was tempted to attach a patch and then thought... well.... it might not be quite worth it :)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Resolved] (MAHOUT-936) Mismatch in case for arg handling in org.apache.mahout.vectorizer.collocations.llr.CollocDriver

Posted by "Sean Owen (Resolved) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/MAHOUT-936?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sean Owen resolved MAHOUT-936.
------------------------------

       Resolution: Fixed
    Fix Version/s: 0.6
         Assignee: Sean Owen

Good one, I submitted a fix.
                
> Mismatch in case for arg handling in org.apache.mahout.vectorizer.collocations.llr.CollocDriver
> -----------------------------------------------------------------------------------------------
>
>                 Key: MAHOUT-936
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-936
>             Project: Mahout
>          Issue Type: Bug
>          Components: Clustering
>    Affects Versions: 0.6
>            Reporter: Mat Kelcey
>            Assignee: Sean Owen
>            Priority: Minor
>             Fix For: 0.6
>
>
> the option minSupport is added with a capital S
>    addOption("minSupport", "s", "(Optional) Minimum Support. Default Value: "
> but the actual value retrieved from the arg map has a lower case s
>    if (argMap.get("--minsupport") != null) {
> Was tempted to attach a patch and then thought... well.... it might not be quite worth it :)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (MAHOUT-936) Mismatch in case for arg handling in org.apache.mahout.vectorizer.collocations.llr.CollocDriver

Posted by "Hudson (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/MAHOUT-936?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13175685#comment-13175685 ] 

Hudson commented on MAHOUT-936:
-------------------------------

Integrated in Mahout-Quality #1268 (See [https://builds.apache.org/job/Mahout-Quality/1268/])
    MAHOUT-936 fix 'minsupport' typo

srowen : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1222685
Files : 
* /mahout/trunk/core/src/main/java/org/apache/mahout/vectorizer/collocations/llr/CollocDriver.java

                
> Mismatch in case for arg handling in org.apache.mahout.vectorizer.collocations.llr.CollocDriver
> -----------------------------------------------------------------------------------------------
>
>                 Key: MAHOUT-936
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-936
>             Project: Mahout
>          Issue Type: Bug
>          Components: Clustering
>    Affects Versions: 0.6
>            Reporter: Mat Kelcey
>            Assignee: Sean Owen
>            Priority: Minor
>             Fix For: 0.6
>
>
> the option minSupport is added with a capital S
>    addOption("minSupport", "s", "(Optional) Minimum Support. Default Value: "
> but the actual value retrieved from the arg map has a lower case s
>    if (argMap.get("--minsupport") != null) {
> Was tempted to attach a patch and then thought... well.... it might not be quite worth it :)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira