On 11/24/15 7:11 AM, sebb wrote:
> On 24 November 2015 at 13:52, Phil Steitz <phil.ste...@gmail.com> wrote:
>> I need the following methods to complete the fix for MATH-1246.  I
>> can add them as private methods to the KS class; but they seem
>> generally useful, so I propose adding them to MathArrays.  Any
>> objections?
>>
>> /**
>>   * Concatenates two arrays.
>>   *
>>   * @param x first array
>>   * @param y second array
>>   * @return a new array consisting of the entries of x followed by
>> the entries of y
>>   * @throws NullPointerException if either x or y is null
>>   */
>> public static double[] concat(double[] x, double[] y)
> Could perhaps take a list?

You mean what Gilles suggested? 

Phil
>
>> /**
>>   * Returns an array consisting of the unique values in {@code data}.
>>   * The return array is sorted in descending order.
>>   *
>>   * @param data
>>   * @return descending list of values included in the input array
>>   */
>> public static double[] values(double[] data)
> The name gives no hint that the output is unique.
> IMO it should be renamed to reflect that.
>
> [The sort order seems to be a byproduct of the algorithm used, so is
> not particularly relevant to the name]
>
>> /**
>>   * Adds random jitter to {@code data} using deviates sampled from
>> {@code dist}.
>>   * <p>
>>   * Note that jitter is applied in-place - i.e., the array
>>   * values are overwritten with the result of applying jitter.</p>
>>   *
>>   * @param data input/output data array
>>   * @param dist probability distribution to sample for jitter values
>>   * @throws NullPointerException if either of the parameters is null
>>   */
>>  public static void jitter(double[] data, RealDistribution dist)
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
>> For additional commands, e-mail: dev-h...@commons.apache.org
>>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>
>


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

Reply via email to