c3-time-domain / SeeChange

A time-domain data reduction pipeline (e.g., for handling images->lightcurves) for surveys like DECam and LS4
BSD 3-Clause "New" or "Revised" License
0 stars 4 forks source link

Cutouts & Measurements have identical RA and Dec ; is this right? #259

Closed rknop closed 2 months ago

rknop commented 2 months ago

Is this intended?

What is the meaning in each case? The x and y of the cutouts is always .00, so it must be the center pixel of the cutout. I was guessing that the cutout RA and Dec would be the RA and Dec of that center pixel, whereas the measurement ra and dec would be the centroid position of the actual detection. However, right now the coordinates are identical. What do they mean in each case? If the meaning is the same, then we shouldn't have the fields in both tables.

If we're just going to store one, it should be the ra and dec of the centroid of the actual detection. In that case, the ra and dec should be in the measurements table, but not in the cutouts table.

guynir42 commented 2 months ago

They should have different coordinates. This is on my list...

guynir42 commented 2 months ago

Duplicate of #234