[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/
- [Octave-bug-tracker] [bug #56492] [octave forge] (image) improvements to documentation of cp2tform,
Hartmut <=