Skip to content
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

Set a temperature dependent mag limit for imposters #118

Merged
merged 3 commits into from
Sep 29, 2018
Merged

Conversation

taldcroft
Copy link
Member

@taldcroft taldcroft commented Sep 26, 2018

Use ideas in imposter stars notebook to set a temperature and box size-dependent mag limit for imposters.

For a given box size, set the mag limit to the magnitude where one expects (statistically) N stars, where N = 2.5 for a 120" box at t_ccd=-10.0. The idea is that statistically on-board you always have imposters fainter than this limit impacting acquisition, and thus they are already accounted for in the acquisition probability model.

With this modification, the proseco probability is only modified if there happen to be brighter-than-normal outliers. This was the original concept of imposters.

This change does significantly impact the estimated p_acq for faint stars, bringing it typically close to the acq prob model.

To do:

  • Fix test fails due to catalog changes (which are all minor)

Closes #25

@taldcroft
Copy link
Member Author

Ran validate-2018 notebook with this branch and results are acceptable. The Dragonfly 44 observations still need a hand-edit, but fine-tuning #132 may help.

@taldcroft taldcroft merged commit f5fe3ac into master Sep 29, 2018
@jeanconn jeanconn deleted the imposters branch January 23, 2019 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mag limit on imposters
1 participant