From e52cee6c9ae9a91235b122750adee38e626ddd3b Mon Sep 17 00:00:00 2001
From: Kipp Cannon <kipp.cannon@ligo.org>
Date: Tue, 19 Dec 2017 11:12:11 -0600
Subject: [PATCH] inspiral_lr: don't include V1 in coinc rates model
- V1 must be removed from the coinc rates model in the denominator because
all coincs will be made to be V1-less in the .__call__() method
---
gstlal-inspiral/python/stats/inspiral_lr.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/gstlal-inspiral/python/stats/inspiral_lr.py b/gstlal-inspiral/python/stats/inspiral_lr.py
index a534e1bacb..21f77b356c 100644
--- a/gstlal-inspiral/python/stats/inspiral_lr.py
+++ b/gstlal-inspiral/python/stats/inspiral_lr.py
@@ -561,8 +561,9 @@ class LnNoiseDensity(LnLRDensity):
# clusters this issue will go away (can use qhull's
# algebraic geometry code for the probability
# calculations).
+ # FIXME: remove V1 from rates model. delete after O2
self.coinc_rates = snglcoinc.CoincRates(
- instruments = self.instruments,
+ instruments = self.instruments - frozenset(("V1",)),
delta_t = self.delta_t,
min_instruments = self.min_instruments
)
--
GitLab