Skip to content

snglcoinc.py: fix new bugs in coincidence code

Kipp Cannon requested to merge (removed):snglcoinc_work into master

Description

Recent bug fixes in snglcoinc.py introduced new bugs. This patch fixes this.

API Changes and Justification

Backwards Compatible Changes

  • This change introduces no API changes
  • This change adds new API calls

Backwards Incompatible Changes

  • This change modifies an existing API
  • This change removes an existing API

If any of the Backwards Incompatible check boxes are ticked please provide a justification why this change is necessary and why it needs to be done in a backwards incompatible way.

Review Status

This patch is currently being reviewed. Let's ask @daichi.tsuna and @hiroaki.ohta to confirm that it works for them. It passes local testing, but so did the previous patch (I am, separately, working on making better test suites for this code, which will eventually be added to lalsuite copying the way it was done for lalapps_bucluster)

Merge request reports