janosch314 / GWFish

Simulation of detector networks with Fisher-matrix PE
33 stars 20 forks source link

LISA sensitivity is overestimated #77

Closed jacopok closed 6 months ago

jacopok commented 6 months ago

The new sensitivity curves are from this paper.

They report some test cases: for example, the first row of Test Case 1 on page 31 is a BBH with source-frame individual masses of $10^5$, at $z=1$. The recovered SNR they find is something like 1100-1200.

The current implementation of the LISA sensitivity in GWFish overestimates this by roughly a factor 10:

from GWFish.modules.horizon import compute_SNR, Planck18
from GWFish.modules.detection import Detector

z = 1
params = {
    'luminosity_distance': Planck18.luminosity_distance(z).value,
    'mass_1': 1e5 * (1+z), 
    'mass_2': 1e5 * (1+z), 
    'theta_jn': 0,
    'ra': 1,
    'dec': -0.5,
    'psi': 0.,
    'phase': 0.,
    'geocent_time': 1187008882.4,
}

snr = compute_SNR(params, Detector('LISA'))
print(snr)

The result is 12009.

jacopok commented 6 months ago

@janosch314 I never looked into the projection_solarorbit function in detail, but it seems like it's only using the first component of LISA (I only see references to components[0]), could that be related to the problem?

janosch314 commented 6 months ago

After updating the LISA noise model, I introduced a bug by not modifying line 447. It currently reads proj[in_band_slice, :] = doppler_to_strain * AET(polarizations[in_band_slice, :], eij, theta, ra, psi, components[0].L, ff)

but it should read proj[in_band_slice, :] = AET(polarizations[in_band_slice, :], eij, theta, ra, psi, components[0].L, ff)