Commit 49564bc1 authored by Brandon Piotrzkowski's avatar Brandon Piotrzkowski

Include trials when checking whether gw in coincidence alert is significant on its own

parent 5012c1ab
......@@ -262,8 +262,10 @@ def compose_raven(gracedb_id, authors=(),
combined_skymap_greedy_area=greedy_area)
kwargs.update(main_dict(gracedb_id, client=client))
if (group.lower() == 'cbc' and superevent_far < 1 / (60 * 86400)) or \
(group.lower() == 'burst' and superevent_far < 1 / (365 * 86400)):
# Including trials of 5 for CBC and 4 for bursts due to number of pipelines
far_threshold = {'cbc': 1 / (60 * 86400), 'burst': 1 / (365 * 86400)}
trials = {'cbc': 5, 'burst': 4}
if superevent_far * trials[group.lower()] < far_threshold[group.lower()]:
kwargs.update(change_significance_statement=False)
else:
kwargs.update(change_significance_statement=True)
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment