[Top][All Lists]

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

[Octave-patch-tracker] [patch #9809] Image package, cp2tform documentati

From: Evangelos Rozos
Subject: [Octave-patch-tracker] [patch #9809] Image package, cp2tform documentation
Date: Mon, 27 May 2019 09:28:52 -0400 (EDT)
User-agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/74.0.3729.169 Safari/537.36


                 Summary: Image package, cp2tform documentation
                 Project: GNU Octave
            Submitted by: vrozos
            Submitted on: Mon 27 May 2019 01:28:50 PM UTC
                Category: Forge : other
                Priority: 5 - Normal
                  Status: None
                 Privacy: Public
             Assigned to: None
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any



The documentation of cp2tform
(https://octave.sourceforge.io/image/function/cp2tform.html) has some errors.

In the affine transformation the following lines

in_cp = [out_cp ones(rows (out_cp,1))] * Tinv

out_cp = [in_cp ones(rows (in_cp,1))] * T

have two errors: i) 'rows' function takes only one argument, and ii) 'ones'
function takes two arguments. Furthermore, it would be more clear to replace
Tinv and T with the full path. That is:

in_cp =  [out_cp ones(rows (out_cp) ,1)] * T.tdata.Tinv

out_cp=  [in_cp ones(rows (in_cp) ,1)] * T.tdata.T

The same problems exist in the projective transformation.

Also, Tinv should be replaced with the full path (i.e. T.tdata.Tinv) in the
polynomial transformation too.


Reply to this item at:


  Message sent via Savannah

reply via email to

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