-
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
Action to add calibrators from PanSTARRS #142
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use ApMag instead of PSFMag
Set condition on the number of observations of each star, ApMagNpt>=5
Set condition on the ApMagStd<0.01
If with those conditions, we cannot find stars, then relax in a second step iteration those for ApMagNpt and ApMagStd
@joteros what do you think of those constraints? I think they are a reasonable minimum. Then we just take the top 10 results, by number of observations. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. I usually use as minimum magnitude allowed 17 instead of 16, but this is a minor change.
At some point we should add a second iteration search in case the search does not find any star in the FoV, relaxing a bit the criteria (it happened to me a couple of times), increasing the Std threshold, lowering the minimum magnitude and the number of observations required, but this can be done afterwards.
I am hesitant to perform this automatic lowering of the quality constraints, since the user would not know about this and would have no choice, at least through this GUI (if we added an interactive tab for it, perhaps). Another option would be to add two different actions, with two different sets of constraints (e.g. "Automatically add field stars from PanSTARRS" and "Automatically add field stars from PanSTARRS (relaxed)"). |
Sounds like a good solution. In any case, as I said this can be implemented (and even refined) afterwards, and my expectation is that it won't happen too often |
This PR adds an action to the AstroSource admin for automatically adding calibrators from PanSTARRS.