[Top][All Lists]

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

[Octave-bug-tracker] [bug #56492] [octave forge] (image) improvements to

From: Hartmut
Subject: [Octave-bug-tracker] [bug #56492] [octave forge] (image) improvements to documentation of cp2tform
Date: Thu, 13 Jun 2019 14:47:57 -0400 (EDT)
User-agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:67.0) Gecko/20100101 Firefox/67.0


                 Summary: [octave forge] (image) improvements to documentation
of cp2tform
                 Project: GNU Octave
            Submitted by: hardy
            Submitted on: Thu 13 Jun 2019 06:47:55 PM UTC
                Category: Octave Forge Package
                Severity: 3 - Normal
                Priority: 5 - Normal
              Item Group: Documentation
                  Status: None
             Assigned to: None
         Originator Name: 
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any
                 Release: dev
        Operating System: Any



There were some suggestions to improve the documentation of cp2tform on the
mailing list. I will copy this to here in order to not forget it:

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

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.


Evangelos Rozos 



Reply to this item at:


  Message sent via Savannah

reply via email to

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