IPP Software Navigation Tools IPP Links Communication Pan-STARRS Links

Opened 21 years ago

Closed 21 years ago

Last modified 20 years ago

#523 closed defect (fixed)

psPolynomial using nTerms not nOrder

Reported by: eugene Owned by: gusciora@…
Priority: high Milestone:
Component: math Version: 0.7.0
Severity: major Keywords:
Cc:

Description

psPolynomial functions are all implemented using the convention psPolynomial.n
is the number of terms, not the number of orders (ie, a line requires n = 2, not
n = 1). This should be implemented as 'number of orders' instead (ie, line
should be specified using n = 1). The SDRS v.15 was a bit ambiguous: the text
said orders, but the comments in the code blocks said 'terms'.

Change History (4)

comment:1 by robert.desonia@…, 21 years ago

Owner: changed from robert.desonia@… to george.gusciora@…

comment:2 by Paul Price, 21 years ago

Severity: normalmajor

Ditto with psPlaneTransform: it currently takes the number of terms, not the
polynomial orders. I changed the SDRS to make it clearer:

Each of \code{psPlaneTransform} and \code{psPlaneDistort} has an
appropriate allocator that takes the polynomial order in each
dimension. Both the \code{x} and \code{y} polynomials shall be have
the same dimensions.
\begin{prototype}
psPlaneTransform *psPlaneTransformAlloc(int order1, int order2);
psPlaneDistort *psPlaneDistortAlloc(int order1, int order2, int order3, int order4);
\end{prototype}

(Previously had "n1, n2" as the arguments.)

comment:3 by robert.desonia@…, 21 years ago

Owner: changed from george.gusciora@… to gusciora@…

comment:4 by gusciora@…, 21 years ago

Resolution: fixed
Status: newclosed

Done.

Note: See TracTickets for help on using tickets.