bugGNU Octave - Bugs: bug #41612, pinv with tol fix for Diagonal...

 
 

bug #41612: pinv with tol fix for Diagonal Matrices

Submitter:  None
Submitted:  Sun 16 Feb 2014 07:55:29 PM UTC
   
 
Category:  None Severity:  3 - Normal
Priority:  5 - Normal Item Group:  None
Status:  Duplicate Assigned to:  None
Originator Name:  Originator Email:  -email is unavailable-
Open/Closed:  * Closed Release:  * dev
Operating System:  * Any Fixed Release:  None
Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Sun 16 Feb 2014 10:30:36 PM UTC, comment #1: 

would someone pls mark this as a dupe of bug #41546

@Anonymous: could you pls repost your comment there; thanks.

Felipe G. Nievinski <fgnievinski>
Sun 16 Feb 2014 07:55:29 PM UTC, original submission:  

I just noticed a patch got sent to fix a bug I reported last week.

Awesome - thanks. However there's a small gotcha in the new code:

+      if (std::abs (elem (i, i)) < tol)
+        retval.elem (i, i) = 0.0;
+      else
         retval.elem (i, i) = 1.0 / elem (i, i);


When tol==0 and elem(i,i)==0, the code will divide by zero. Otherwise looks good - sorry I couldn't find the original bug report so I had to re-post.

Anonymous

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #30581:  bug_41612.diff added by eg123 (2KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by fgnievinski (Posted a comment)
  • -email is unavailable- added by eg123 (Updated 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 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2014-02-17 philipnienhuis StatusNone Duplicate
        Open/ClosedOpen Closed
    2014-02-16 eg123 Attached File- Added bug_41612.diff, #30581

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code