dstndstn / tractor

The Tractor: measuring astronomical sources via probabilistic inference
Other
86 stars 24 forks source link

PixelizedPsfEx keyword for XY [POLNAME1,POLNAME2] #109

Open marcogullieuszik opened 1 month ago

marcogullieuszik commented 1 month ago

I am just starting using tractor.

PixelizedPsfEx requires that PSFEX (and sextractor) use (X_IMAGE, Y_IMAGE) to define the source positions in the code line "assert(hdr['POLNAME1'].strip() == 'X_IMAGE')"

It would be extremely useful to add the option of using other sextractor positional parameters. For example, I usually use XWIN_IMAGE and YWIN_IMAGE as they provide more accurate centroid positions.

As a workaround I am now changing the "POLNAME1","POLNAME2" keywords in the PSFEX file

dstndstn commented 1 month ago

Good idea! If you want to send a Pull Request with that improvement, that would be great!