bugGNU Octave - Bugs: bug #37539, eps(z) incorrect for matrix arg

 
 

bug #37539: eps(z) incorrect for matrix arg

Submitter:  Edward Meyer <eem2314>
Submitted:  Wed 10 Oct 2012 06:11:21 PM UTC
   
 
Category:  Interpreter Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  Fixed Assigned to:  cdf
Originator Name:  Ed Meyer Open/Closed:  * Closed
Release:  * dev Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Fri 12 Oct 2012 07:34:11 AM UTC, comment #6: 

I like your version, so I pushed it :
http://hg.savannah.gnu.org/hgweb/octave/rev/f0dfdc7faa71
I am closing this bug.

Carlo de Falco <cdf>
Group Member
Thu 11 Oct 2012 08:04:27 PM UTC, comment #5: 

Carlo, take a look at the attached changeset (eps.cset).  I followed your lead, but I believe I simplified things by not creating a second copy of the input matrix.  This routine isn't a high performance one so I also used normal indexing, x(i), rather than changing over to a Fortran vector, C pointers, and x[i] notation.

(file #26742)

Rik <rik5>
Group administrator
Thu 11 Oct 2012 05:22:04 PM UTC, comment #4: 

Rik, is it OK then if I push?

Carlo de Falco <cdf>
Group Member
Thu 11 Oct 2012 04:04:07 PM UTC, comment #3: 

Ignore the last comment.  I see that the Matlab behavior has already been verified on the maintainers list.

Rik <rik5>
Group administrator
Thu 11 Oct 2012 03:35:17 PM UTC, comment #2: 

Could someone with Matlab verify what they do?  The documentation doesn't seem to contemplate that eps can be called with a matrix as an argument.  We could, of course, choose to extend it in this logical way.

Rik <rik5>
Group administrator
Thu 11 Oct 2012 07:27:45 AM UTC, comment #1: 

The attached changeset should fix the problem.
c.




(file #26737)

Carlo de Falco <cdf>
Group Member
Wed 10 Oct 2012 06:11:21 PM UTC, original submission:  

eps(z) returns eps(z(1)) for matrix argument z;  it should return a matrix the size of z with the (i,j) element = eps(z(i,j))

Edward Meyer <eem2314>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #26742:  eps.cset added by rik5 (5KiB - application/octet-stream)
file #26737:  open_af6NJoOl.txt added by cdf (5KiB - text/plain)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by cdf (Updated the item)
  • -email is unavailable- added by eem2314 (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2012-10-12 cdf StatusPatch Submitted Fixed
        Open/ClosedOpen Closed
    2012-10-11 rik5 Attached File- Added eps.cset, #26742
    2012-10-11 cdf Attached File- Added open_af6NJoOl.txt, #26737
        StatusNone Patch Submitted
        Assigned toNone cdf
        Operating SystemGNU/Linux Any

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code