octave-patch-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Octave-patch-tracker] [patch #8119] Allow variable tolerance and improv


From: Dan Sebald
Subject: [Octave-patch-tracker] [patch #8119] Allow variable tolerance and improve error messages for assert.m script
Date: Fri, 02 Aug 2013 18:54:31 +0000
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:18.0) Gecko/20100101 Firefox/18.0 SeaMonkey/2.15

Follow-up Comment #6, patch #8119 (project octave):

I wasn't aware of the plus format, but I'm not sure it is that informative. 
It's definitely condensed, but it is difficult to pick out exactly what entry
it might be in a matrix.

I like your idea of a tabular form of display.  I've been trying to think of a
way to reuse existing display features, but it is probably easier to just
write a loop and create a table of this sort.  What I like about it is that
there is only one position entry that tells right away what element is being
compared, rather than have to look back and forth between two "cell display"
lists to find the, say, [2,1] and [2,1].  The numbers that do not match are
right next to one another.  Also, the last column gives insight into what
might be wrong, especially if it were to include the degree of failure.  If I
saw just a tiny discrepancy, I might guess it is a machine precision issue.

I think the column names are good ("observed" rather than "got", perhaps) and
maybe just a little more condensed explanation.  I think there'd be sufficient
space in 80 characters.  For example:


Location  |  Expected  |  Observed  |  Reason
[1,2]           2            2.0       Exceeds abs tol 0 by 2.174e-25


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?8119>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]