Welcome to iraf.net Friday, May 17 2024 @ 08:11 AM GMT
Jason Quinn |
02/27/2009 03:02PM (Read 3585 times)
|
|
|
Status: offline
Registered: 04/07/2006
Posts: 175
|
I found some bugs in noao$digiphot/apphot/aputil/apnlfuncs.x. I'm not sure how important this library is to IRAF. I'll post the code below and you can use your favorite diff tool to find the changes.Change summary:
|
|
|
|
valdes |
02/27/2009 03:02PM
|
|
|
Status: offline
Registered: 11/11/2005
Posts: 728
|
Hi Jason,Thanks for identifying the problems. I have made the changes you suggested. I used the term variance instead of sigma-squared.As to how these routines are used we have to assume the applications know what parameters are needed.Yours,
Frank Valdes
|
|
|
|
Jason Quinn |
02/27/2009 03:02PM
|
|
|
Status: offline
Registered: 04/07/2006
Posts: 175
|
In the IRAF 2.15ALPHA release, one of the previous problems was incorrectly fixed.In CDGAUSS1D procedure (not to be confused with the CGAUSS1D procedure)... the code comment should read:# p[1]=amplitude, p[2]=center, p[3]=sigma p[4]=skyThe third argument really is a sigma, not a variance, so the word "variance" should not be used there.JasonPS The CGAUSS1D procedure squares some numbers using the exponential operator (**). The other procedures avoid exponentiation and use an extra variable and multiplication to square numbers. If this is for efficiency, then CGAUSS1D could easily be improved. I'm not sure how costly doing "x**2" vs "x*x" is for IRAF.
|
|
|
|
| |
|
Content generated in: 0.10 seconds |
|