octave-bug-tracker
[Top][All Lists]
Advanced

[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

URL:
  <https://savannah.gnu.org/bugs/?56492>

                 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

    _______________________________________________________

Details:

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
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.

Regards

Evangelos Rozos 


source:
http://octave.1599824.n4.nabble.com/Image-package-cp2tform-documentation-td4692963.html#a4692964




    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?56492>

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




reply via email to

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