-
Notifications
You must be signed in to change notification settings - Fork 356
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
Enhance stat vs FAR plots on the offline analysis summary page to include all event types and singles fit lines #5006
Comments
For whoever gets to this, the code on the Caltech LVK cluster at This was used to produce Figure 6 of https://arxiv.org/pdf/2203.08545, and the general codes / plots produced can be found here (LVK paywall) |
This paper plot script used function calls like |
As the plotting uses a mix of both fitted and extrapolated IFAR estimates, we needed to use both the If we do that, we will want to use If we only care about plotting the functions actually used for the IFAR, we can use get_far directly. But I think it makes sense to plot the n_louder in the case that we are using a trigger fit, simply so that we can check that the fit looks good. We may also want to include the IFAR limit information, as that is used by the statmap jobs - probably as a hline for each combination that uses a limit. |
FWIW I've just used two calls to |
I thought that shouldn't be needed - it should fall back to n_louder below the fit threshold if using trigger_fit |
I haven't looked at the singles far code in that much detail, am just trying to do the minimal adaptation of the paper script. Let's consider the details in a PR. |
At present the stat/FAR plots appear separately for each coinc or single event type, but that does not allow us to easily compare how the coinc backgrounds line up against each other, or against the singles fits / extrapolations. There is a nice multi-event-type FAR-stat plot in the methods paper https://arxiv.org/abs/2203.08545 which does so. Extend the SNRIFAR plot code to allow for this and make it part of the standard plotting workflow.
Specifically, this would allow us to diagnose possible cases where the FAR at high stat values might be dominated by singles fit extrapolation (which would hurt sensitivity at a given combined FAR for coincs) - or indeed unduly dominated by any event type.
The text was updated successfully, but these errors were encountered: