dif-c-mri-5-epoxi-hartley2-photom-v1.0 ====================================== - document/code/image_corrections/gap_correction/image_gap_correction.pro: Comment on line 26 should be: ";I. resample image, effectively compressing each center row to 5/6 pixel width" ------------------------------------------------------------------------------- On 5 Mar 2013, at 08:44 EST, Jade Williams wrote: Brian thanks so much for pointing that out- and you're entirely correct. My code does actually compress each center row to 5/6 a pixel (correcting the 1/3 gap in the image as a whole). So it's the comment in line 26 of the referenced routine should instead say: ;I. resample image, effectively compressing each center row to 5/6 pixel width Whew, well it's a typo- so no changes needed to the data or results! I attached this routine with the edit to line 26. In the various documentations where explanation of the gap correction is given, we should check also to ensure a similar typo isn't there also...I'll ask Stef about this. Thanks again Brian! I'll be sitting in the review meeting & hope to answer any other questions/concerns. -Jade ------------------------------------------------------------------------------- On Mar 5, 2013, at 6:40 AM, Brian Carcich wrote: Hi Stef, I have some other telecons and projects going on today so I'll only be dialing in for the CAL IR and the Mars and Earth data sets, but feel free to email or call the mobile (607-342-7595) if you need me for anything. I'd especially appreciate knowing how the schedule is going as the later review times may shift. I was noodling around the data sets and noticed this: http://pdssbn.astro.umd.edu/holdings/ dif-c-mri-5-epoxi-hartley2-photom-v1.0/document/ code/image_corrections/gap_correction/image_gap_correction.pro A comment in that routine states it compresses the center pixel rows to 2/3 of a pixel each and the results look okay so this may be a canard, but my memory is that it was a 1/3-pixel gap between the center rows, so each row was 5/6 of a pixel. I haven't groked what the code is doing yet, and I'll have to check some old emails ... Okay, I found the old emails. We had a couple of extensive threads about this, so I know Jade received the message that the center rows are only 5/6 of a nominal pixel on the CCD, so I exect the comment in the .PRO is just a typo. Or maybe the code only shifts one of the row's brightnesses by a third which would be equivalent to shifting both by 1/6. The only difference would be in the declared position of the centroid center, not in the aperture photometry results. -b -------------------------------------------------------------------------------