Mailing-List: contact dev-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Commons Developers List" <dev@commons.apache.org>
Received-SPF: pass (athena.apache.org: domain of phil.steitz@gmail.com
 designates 209.85.160.43 as permitted sender)
Message-ID: <5046E37F.2030206@gmail.com>
Date: Tue, 04 Sep 2012 22:30:39 -0700
From: Phil Steitz <phil.steitz@gmail.com>
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7;
 rv:14.0) Gecko/20120713 Thunderbird/14.0
MIME-Version: 1.0
To: Commons Developers List <dev@commons.apache.org>
Subject: [math] IllegalStateException thrown by Kurtosis increment and clear
 methods
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit

Slugging my way through the MATH-854 changes for the stats package,
I have found an anomaly that I would like to fix.  When a Kurtosis
instance is constructed using an external moment, increment and
clear methods do not behave as no-ops as they do for Variance,
SecondMoment and the other moment statistics that can be constructed
using embedded external moments.  Instead these methods throw
MathISE when called on an instance with an embedded moment. 
Changing the methods to behave like the other moment statistics
would technically not be an incompatible change because the
exception is unchecked.  Leaving it as is inconvenient for MATH-854
since we have to then advertise the exception and this is not
consistent with superclass declarations.   Anyone object to this change?

Phil

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