Looks good to me.

On Fri, Sep 9, 2011 at 6:39 AM, Gilles Sadowski <
gil...@harfang.homelinux.org> wrote:

> On Thu, Sep 08, 2011 at 04:55:46PM -0700, Ted Dunning wrote:
> > OK.
> >
> > Replace that with the correct value.  I meant it to be an index.
> >
> > That doesn't change my other points.  There is an inherent problem with
> > "less than" comments when you have subtracted several other elements
> > previously and only now notices that the remainder is less than some
> other
> > adjusted value.  The user cannot relate either value back to anything
> that
> > they have and they may well look at their own matrix and see the opposite
> > condition.  It is better to give a slightly more abstract message such as
> > "Non-positive definiteness in input detected at diagonal element number
> xxx"
>
> What do you think of the following message?
>
> <element value> is smaller than, or equal to, the minimum (<threshold
> value>):
> not positive definite matrix: value <element value> at index <element
> index>
>
> where, in actual messages, the <...> will be the numeric values of course.
>
> "NonPositiveDefiniteMatrixException" would inherit from
> "NumberIsTooSmallException". The rationale being that it exactly matches
> the
> test and that it is meaningless to report an element of the original
> matrix,
> as you've explained.
>
> > [...]
>
> Gilles
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>
>

Reply via email to