You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by md...@apache.org on 2003/06/20 23:05:48 UTC

cvs commit: jakarta-commons-sandbox/math/src/java/org/apache/commons/math/stat AbstractStoreUnivariate.java StatUtils.java

mdiggory    2003/06/20 14:05:47

  Modified:    math/src/java/org/apache/commons/math/stat
                        AbstractStoreUnivariate.java StatUtils.java
  Log:
  PR: http://nagoya.apache.org/bugzilla/show_bug.cgi?id=20964
  Submitted by:	phil@steitz.com
  
  Revision  Changes    Path
  1.4       +0 -10     jakarta-commons-sandbox/math/src/java/org/apache/commons/math/stat/AbstractStoreUnivariate.java
  
  Index: AbstractStoreUnivariate.java
  ===================================================================
  RCS file: /home/cvs/jakarta-commons-sandbox/math/src/java/org/apache/commons/math/stat/AbstractStoreUnivariate.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- AbstractStoreUnivariate.java	11 Jun 2003 14:50:30 -0000	1.3
  +++ AbstractStoreUnivariate.java	20 Jun 2003 21:05:47 -0000	1.4
  @@ -62,16 +62,6 @@
    */
   public abstract class AbstractStoreUnivariate implements StoreUnivariate {
   
  -    /** 
  -     * Returns the most frequently occuring value
  -     * @see org.apache.commons.math.stat.StoreUnivariate#getMode()
  -     */
  -    public double getMode() {
  -        // Mode depends on a refactor Freq class
  -        String msg = "getMode() is not yet implemented";
  -        throw new UnsupportedOperationException(msg);
  -    }
  -
       /**
        * Returns the skewness of this collection of values
        * @see org.apache.commons.math.stat.StoreUnivariate#getSkewness()
  
  
  
  1.8       +1 -23     jakarta-commons-sandbox/math/src/java/org/apache/commons/math/stat/StatUtils.java
  
  Index: StatUtils.java
  ===================================================================
  RCS file: /home/cvs/jakarta-commons-sandbox/math/src/java/org/apache/commons/math/stat/StatUtils.java,v
  retrieving revision 1.7
  retrieving revision 1.8
  diff -u -r1.7 -r1.8
  --- StatUtils.java	18 Jun 2003 12:42:24 -0000	1.7
  +++ StatUtils.java	20 Jun 2003 21:05:47 -0000	1.8
  @@ -282,26 +282,4 @@
           }
           return min;
       }
  -
  -    /** 
  -     * Returns the mode of the values that have been added.  The mode is
  -     * the element which occurs with the most frequency
  -     * @return the mode
  -     */
  -    public static double mode() {
  -        // Mode depends on a refactor Freq class
  -        String msg = "mode() is not yet implemented";
  -        throw new UnsupportedOperationException(msg);
  -    }
  -
  -    /** 
  -     * Returns the mode of the values that have been added.  The mode is
  -     * the element which occurs with the most frequency
  -     * @return the mode
  -     */
  -    public static double median(double[] values) {
  -        // Mode depends on a refactor Freq class
  -        String msg = "median() is not yet implemented";
  -        throw new UnsupportedOperationException(msg);
  -    }
  -}
  \ No newline at end of file
  +}
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Re: cvs commit: jakarta-commons-sandbox/math/src/java/org/apache/commons/math/stat AbstractStoreUnivariate.java StatUtils.java

Posted by "Mark R. Diggory" <md...@latte.harvard.edu>.
I got this by hand and I'm commiting it now. -Mark

Phil Steitz wrote:
> mdiggory@apache.org wrote:
> 
>> mdiggory    2003/06/20 14:05:47
>>
>>   Modified:    math/src/java/org/apache/commons/math/stat
>>                         AbstractStoreUnivariate.java StatUtils.java
>>   Log:
>>   PR: http://nagoya.apache.org/bugzilla/show_bug.cgi?id=20964
>>   Submitted by:    phil@steitz.com
>>     Revision  Changes    Path
>>   1.4       +0 -10     
>> jakarta-commons-sandbox/math/src/java/org/apache/commons/math/stat/AbstractStoreUnivariate.java 
>>
>>     Index: AbstractStoreUnivariate.java
>>   ===================================================================
>>   RCS file: 
>> /home/cvs/jakarta-commons-sandbox/math/src/java/org/apache/commons/math/stat/AbstractStoreUnivariate.java,v 
>>
>>   retrieving revision 1.3
>>   retrieving revision 1.4
>>   diff -u -r1.3 -r1.4
>>   --- AbstractStoreUnivariate.java    11 Jun 2003 14:50:30 -0000    1.3
>>   +++ AbstractStoreUnivariate.java    20 Jun 2003 21:05:47 -0000    1.4
>>   @@ -62,16 +62,6 @@
>>     */
>>    public abstract class AbstractStoreUnivariate implements 
>> StoreUnivariate {
>>      -    /**   -     * Returns the most frequently occuring value
>>   -     * @see org.apache.commons.math.stat.StoreUnivariate#getMode()
>>   -     */
>>   -    public double getMode() {
>>   -        // Mode depends on a refactor Freq class
>>   -        String msg = "getMode() is not yet implemented";
>>   -        throw new UnsupportedOperationException(msg);
>>   -    }
>>   -
>>        /**
>>         * Returns the skewness of this collection of values
>>         * @see org.apache.commons.math.stat.StoreUnivariate#getSkewness()
>>         1.8       +1 -23     
>> jakarta-commons-sandbox/math/src/java/org/apache/commons/math/stat/StatUtils.java 
>>
>>     Index: StatUtils.java
>>   ===================================================================
>>   RCS file: 
>> /home/cvs/jakarta-commons-sandbox/math/src/java/org/apache/commons/math/stat/StatUtils.java,v 
>>
>>   retrieving revision 1.7
>>   retrieving revision 1.8
>>   diff -u -r1.7 -r1.8
>>   --- StatUtils.java    18 Jun 2003 12:42:24 -0000    1.7
>>   +++ StatUtils.java    20 Jun 2003 21:05:47 -0000    1.8
>>   @@ -282,26 +282,4 @@
>>            }
>>            return min;
>>        }
>>   -
>>   -    /**   -     * Returns the mode of the values that have been 
>> added.  The mode is
>>   -     * the element which occurs with the most frequency
>>   -     * @return the mode
>>   -     */
>>   -    public static double mode() {
>>   -        // Mode depends on a refactor Freq class
>>   -        String msg = "mode() is not yet implemented";
>>   -        throw new UnsupportedOperationException(msg);
>>   -    }
>>   -
>>   -    /**   -     * Returns the mode of the values that have been 
>> added.  The mode is
>>   -     * the element which occurs with the most frequency
>>   -     * @return the mode
>>   -     */
>>   -    public static double median(double[] values) {
>>   -        // Mode depends on a refactor Freq class
>>   -        String msg = "median() is not yet implemented";
>>   -        throw new UnsupportedOperationException(msg);
>>   -    }
>>   -}
>>   \ No newline at end of file
>>   +}
>>      
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
>> For additional commands, e-mail: commons-dev-help@jakarta.apache.org
>>
>>
> 
> Looks like the last part of the patch, removing the method in 
> StoreUnivariate, somehow got chopped off.  The getMode() method needs to 
> be removed from StoreUnivariate.  This is included in the patch here:
> http://nagoya.apache.org/bugzilla/showattachment.cgi?attach_id=6921
> 
> 
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: commons-dev-help@jakarta.apache.org
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Re: cvs commit: jakarta-commons-sandbox/math/src/java/org/apache/commons/math/stat AbstractStoreUnivariate.java StatUtils.java

Posted by Phil Steitz <ph...@steitz.com>.
mdiggory@apache.org wrote:
> mdiggory    2003/06/20 14:05:47
> 
>   Modified:    math/src/java/org/apache/commons/math/stat
>                         AbstractStoreUnivariate.java StatUtils.java
>   Log:
>   PR: http://nagoya.apache.org/bugzilla/show_bug.cgi?id=20964
>   Submitted by:	phil@steitz.com
>   
>   Revision  Changes    Path
>   1.4       +0 -10     jakarta-commons-sandbox/math/src/java/org/apache/commons/math/stat/AbstractStoreUnivariate.java
>   
>   Index: AbstractStoreUnivariate.java
>   ===================================================================
>   RCS file: /home/cvs/jakarta-commons-sandbox/math/src/java/org/apache/commons/math/stat/AbstractStoreUnivariate.java,v
>   retrieving revision 1.3
>   retrieving revision 1.4
>   diff -u -r1.3 -r1.4
>   --- AbstractStoreUnivariate.java	11 Jun 2003 14:50:30 -0000	1.3
>   +++ AbstractStoreUnivariate.java	20 Jun 2003 21:05:47 -0000	1.4
>   @@ -62,16 +62,6 @@
>     */
>    public abstract class AbstractStoreUnivariate implements StoreUnivariate {
>    
>   -    /** 
>   -     * Returns the most frequently occuring value
>   -     * @see org.apache.commons.math.stat.StoreUnivariate#getMode()
>   -     */
>   -    public double getMode() {
>   -        // Mode depends on a refactor Freq class
>   -        String msg = "getMode() is not yet implemented";
>   -        throw new UnsupportedOperationException(msg);
>   -    }
>   -
>        /**
>         * Returns the skewness of this collection of values
>         * @see org.apache.commons.math.stat.StoreUnivariate#getSkewness()
>   
>   
>   
>   1.8       +1 -23     jakarta-commons-sandbox/math/src/java/org/apache/commons/math/stat/StatUtils.java
>   
>   Index: StatUtils.java
>   ===================================================================
>   RCS file: /home/cvs/jakarta-commons-sandbox/math/src/java/org/apache/commons/math/stat/StatUtils.java,v
>   retrieving revision 1.7
>   retrieving revision 1.8
>   diff -u -r1.7 -r1.8
>   --- StatUtils.java	18 Jun 2003 12:42:24 -0000	1.7
>   +++ StatUtils.java	20 Jun 2003 21:05:47 -0000	1.8
>   @@ -282,26 +282,4 @@
>            }
>            return min;
>        }
>   -
>   -    /** 
>   -     * Returns the mode of the values that have been added.  The mode is
>   -     * the element which occurs with the most frequency
>   -     * @return the mode
>   -     */
>   -    public static double mode() {
>   -        // Mode depends on a refactor Freq class
>   -        String msg = "mode() is not yet implemented";
>   -        throw new UnsupportedOperationException(msg);
>   -    }
>   -
>   -    /** 
>   -     * Returns the mode of the values that have been added.  The mode is
>   -     * the element which occurs with the most frequency
>   -     * @return the mode
>   -     */
>   -    public static double median(double[] values) {
>   -        // Mode depends on a refactor Freq class
>   -        String msg = "median() is not yet implemented";
>   -        throw new UnsupportedOperationException(msg);
>   -    }
>   -}
>   \ No newline at end of file
>   +}
>   
>   
>   
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: commons-dev-help@jakarta.apache.org
> 
> 

Looks like the last part of the patch, removing the method in 
StoreUnivariate, somehow got chopped off.  The getMode() method needs to 
be removed from StoreUnivariate.  This is included in the patch here:
http://nagoya.apache.org/bugzilla/showattachment.cgi?attach_id=6921




---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org