From 9ef952f1b37e4c16f6d11a08afbb469641c700af Mon Sep 17 00:00:00 2001
From: "chad.hanna" <crh184@psu.edu>
Date: Thu, 14 Feb 2019 03:52:21 -0800
Subject: [PATCH] REVERT ME: lloidparts.py: raise the snr threshold to 4
 instead of 3.5 to make online running more stable until coincidence / other
 python stuff is faster

---
 gstlal-inspiral/python/lloidparts.py | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/gstlal-inspiral/python/lloidparts.py b/gstlal-inspiral/python/lloidparts.py
index 4df1ecf5c7..c0dc7e8c88 100644
--- a/gstlal-inspiral/python/lloidparts.py
+++ b/gstlal-inspiral/python/lloidparts.py
@@ -690,12 +690,12 @@ def mkLLOIDmulti(pipeline, detectors, banks, psd, psd_fft_length = 32, ht_gate_t
 		#snr = pipeparts.mktee(pipeline, snr)
 		if chisq_type == 'autochisq':
 			# FIXME don't hardcode
-			# peak finding window (n) in samples is 1 second at max rate, ie max(rates)
+			# peak finding window (n) in samples is 1/4 second at max rate, ie max(rates) / 4
+			# NOTE the snr min set in the diststats file is 3.5,
+			# but 4 is about the lowest we can do stably for
+			# coincidence online...
 			nsamps_window = max(max(bank.get_rates()) / 4, 256) # FIXME stupid hack
-			if instrument == 'H1' or instrument == 'L1':
-				head = pipeparts.mkitac(pipeline, snr, nsamps_window, bank.template_bank_filename, autocorrelation_matrix = bank.autocorrelation_bank, mask_matrix = bank.autocorrelation_mask, snr_thresh = 4, sigmasq = bank.sigmasq)
-			else:
-				head = pipeparts.mkitac(pipeline, snr, nsamps_window, bank.template_bank_filename, autocorrelation_matrix = bank.autocorrelation_bank, mask_matrix = bank.autocorrelation_mask, snr_thresh = bank.snr_threshold, sigmasq = bank.sigmasq)
+			head = pipeparts.mkitac(pipeline, snr, nsamps_window, bank.template_bank_filename, autocorrelation_matrix = bank.autocorrelation_bank, mask_matrix = bank.autocorrelation_mask, snr_thresh = 4.0, sigmasq = bank.sigmasq)
 			if verbose:
 				head = pipeparts.mkprogressreport(pipeline, head, "progress_xml_%s" % suffix)
 			triggersrcs[instrument].add(head)
-- 
GitLab