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

[Evangelos Rozos]

URL:
  <https://savannah.gnu.org/patch/?9809>

                 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

    _______________________________________________________

Details:

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:

  <https://savannah.gnu.org/patch/?9809>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/