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

Estimate sensitivity scale #28

Open
JannisNe opened this issue Dec 28, 2020 · 1 comment
Open

Estimate sensitivity scale #28

JannisNe opened this issue Dec 28, 2020 · 1 comment
Assignees
Labels
enhancement New feature or request wish list Desireable but not urgent

Comments

@JannisNe
Copy link
Collaborator

I'm always frustrated when I have to guess the injection scale where I assume the sensitivity to be and run the injection steps to find out I was wrong.

This could be avoided by first estimating the scale using quick injections with a low number of trials.

That does however require the background TS distribution which would mean that the background TS calculation and the signal injections would have to be submitted in two subsequent steps and not, as it is now, at the same time.

sensitivity.pdf

@JannisNe JannisNe added enhancement New feature or request wish list Desireable but not urgent labels Dec 28, 2020
@JannisNe JannisNe self-assigned this Dec 28, 2020
JannisNe added a commit that referenced this issue Jan 4, 2021
…y and to the cluster, enabling estimation of the injection scale prior to submission, progress on #28
@JannisNe
Copy link
Collaborator Author

JannisNe commented Jan 8, 2021

The method implemented now does not require the background TS distribution!
It basically assumes a background TS distribution median of zero. For fit_weights that does not hold and gives an estimate that is too low.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request wish list Desireable but not urgent
Projects
None yet
Development

No branches or pull requests

1 participant