-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve the radius determination for the field mask #146
Comments
The radius-guess for the field mask should change before and after 2012 Dec 9, because the plate scale changed from 4.46 arcsec/pixel to 4.35 arcsec/pixel. The values should be stored in the epoch file. The code use a field radius guess to find the circle in the flats, then the values in the flat is used as guess for the science images. The old values used in the code for after 2012 was 297 but checking several random files, it seems the values of the field radius is always slightly larger, i.e. ~300 pixels, so I am changing this to a higher number The radius guess for the field stop should be:
|
Processing 20120517 with intermediate products has started. Results will appear on |
The intermediate products are in |
The new guesses for the occulter radius and field stop radius chosen for the epoch before 2012 12 09 work well. We need to improve the threshold for the first guess of the center. I hope this will fix the issue. I modified it to: threshold = 10*median(flat-dark) |
In 2012 (and likely in 2011) the field mask and occulter were not always co-centered.
Our centering routine does not find a reliable center for field mask and this introduced artifact in the data and must be improved before reprocessing 2011. We also need to reprocess some 2012 days when the center was incorrect.
This may be caused by the initial center guess that is based on a first simple estimate of the occulter center. I would start by increasing the range for the inflection point search for the file mask to see if finds the center better.
We can also try changing the fixed 250 threshold as described in issue #145
To further debug this and issue #132, I would like to run intermediate products for 2012 05 17.
Tasks
{fg,bkg}{plus, minus}
)The text was updated successfully, but these errors were encountered: