From b4110c12d0f9ee909d9419b6ccad44f72c1bafd8 Mon Sep 17 00:00:00 2001
From: Gregory Ashton <gregory.ashton@ligo.org>
Date: Wed, 2 Aug 2017 10:15:23 +0200
Subject: [PATCH] Adds twice the spin-modulation to the window size

Previously, the spin-modulation was neglected. This adds it back in and
doubles it to ensure the window covers all the frequencies
---
 pyfstat/injection_helper_functions.py | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/pyfstat/injection_helper_functions.py b/pyfstat/injection_helper_functions.py
index 3698518..29bf392 100644
--- a/pyfstat/injection_helper_functions.py
+++ b/pyfstat/injection_helper_functions.py
@@ -42,7 +42,11 @@ def _calcDopplerWings(
     horizontals = s_lon - e_longitudes
 
     dopplerShifts = s_freq * np.sin(horizontals) * np.cos(vertical) * v_over_c
-    return np.amin(dopplerShifts), np.amax(dopplerShifts)
+    F0min, F0max = np.amin(dopplerShifts), np.amax(dopplerShifts)
+
+    # Add twice the spin-modulation
+    SpinModulationMax = 2*np.pi*lal.REARTH_SI/lal.DAYSID_SI/lal.C_SI * s_freq
+    return F0min - 2*SpinModulationMax, F0max + 2*SpinModulationMax
 
 
 def _calcSpindownWings(freq, fdot, minStartTime, maxStartTime):
-- 
GitLab