[Octave-bug-tracker] [bug #39644] delaunay matlab compatibility and input check problem, patch attached

[Andreas Weber]

URL:
  <http://savannah.gnu.org/bugs/?39644>

                 Summary: delaunay matlab compatibility and input check
problem, patch attached
                 Project: GNU Octave
            Submitted by: andy1978
            Submitted on: Do 01 Aug 2013 13:12:12 GMT
                Category: Octave Function
                Severity: 3 - Normal
                Priority: 5 - Normal
              Item Group: Regression
                  Status: None
             Assigned to: None
         Originator Name: 
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any
                 Release: dev
        Operating System: GNU/Linux

    _______________________________________________________

Details:

Since change 7c4a6197e020
(http://hg.savannah.gnu.org/hgweb/octave/rev/7c4a6197e020) 

+verbatim
a=[1 5 2; 5 6 7]; b=[5 7 8;1 2 3]; T=delaunay(a,b)


stopped working. Matlab returns

T =
5 1 3
1 2 4
3 4 6
3 1 4 


which is equivalent to what octave3.6 returned:

T =
3 5 1
4 2 1
4 3 1
4 3 6


Furthermore I think the input check for single matrix

  case 1
    if (! ismatrix (varargin{1}) && columns (varargin{1}) != 2)
        error ("delaunay: X must be a matrix with 2 columns");

should be
    if (! ismatrix (varargin{1}) || columns (varargin{1}) != 2)
        error ("delaunay: X must be a matrix with 2 columns");


Regards, Andy






    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?39644>

_______________________________________________
  Nachricht gesendet von/durch Savannah
  http://savannah.gnu.org/