bugGNU Octave - Bugs: bug #38074, maketform : transform matrices...

 
 

bug #38074: maketform : transform matrices inverted

Submitter:  Pantxo Diribarne <pantxo>
Submitted:  Sun 13 Jan 2013 07:59:09 PM UTC
   
 
Category:  Octave Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  Fixed Assigned to:  carandraug
Originator Name:  Open/Closed:  * Closed
Release:  * dev Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 14 Jan 2013 04:33:57 PM UTC, comment #3: 

You're right. I have removed the superfluous call to tolower. I had also added one to maketform.

Carnë

Carnë Draug <carandraug>
Group Member
Mon 14 Jan 2013 04:23:22 PM UTC, comment #2: 

I have also seen this error when using npt>~30000 in projective transform test (the error was in svd function), and that is why I chose npt=10000 instead of 100000 as for the other tests. If the error you obtain is also in svd, you just have to put npt=1000 or so.

I wrote those tests on a fairly recent 64 bit linux box where the only one that occasionnally fails is the polynomial ord = 4.
Now on a 64bit MacBook Pro some other tests occasionnally fail. I will have to dig into this. Is mldivide supposed to behave differently from one platform to another?

About the "tolower" you added : it is unusefull as "ttype" is always lowercased at line 133. We should remove one of theim.

Pantxo Diribarne <pantxo>
Group Member
Mon 14 Jan 2013 02:55:33 PM UTC, comment #1: 

I have applied your patch. Thank you.

Also made a small changes so that the functions are not case-sensitive about the function name (use tolower around the option name to start the switch block).

I tried to run the tests you wrote for cp2tform, to make sure my changes did not break anything but I get a memory exausted error or requested size too large (my system is 32bit). Would it be possible to write some smaller tests? If not, would it be possible to have different tests depending on the system?

Carnë Draug <carandraug>
Group Member
Sun 13 Jan 2013 07:59:09 PM UTC, original submission:  

In the developement veresion of image package, the following code should return 1 :

A = [1, 2, 0; 3, 4, 0; 5, 6, 1];
T = maketform ('affine', A);
all (all (T.tdata.T == A))


maketform actually stores A in T.tdata.Tinv. I have attached a changeset that fixes this behaviours and makes subsequent changes in cp2tform.m

Pantxo Diribarne <pantxo>
Group Member

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #27252:  maketf_diff.patch added by pantxo (3KiB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by pantxo (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 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-10-26 carandraug Open/ClosedOpen Closed
    2013-01-14 carandraug StatusNone Fixed
        Assigned toNone carandraug
    2013-01-13 pantxo Attached File- Added maketf_diff.patch, #27252

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code