Alexander Pace authored 6 years ago and Kipp Cannon committed 6 years ago

Update lloidhandler.py: suppressing "float() argument must be a string or a number..." warnings in gstlal_inspiral

- this code dates from a time when noise alone was what we "expected", now that statement is not true.  this patch changes the wording in some plots from "expected" to "noise model" to be more precise

chad.hanna authored 6 years ago and Kipp Cannon committed 6 years ago

inspiral.py: under max over coinc in gracedb upload since clustering has been added post coincidence

chad.hanna authored 6 years ago and Kipp Cannon committed 6 years ago

lloidhandler.py: 3 changes. 1) output the 'full' coinc trigger information to the kafka queue. 2) add a clustering keyword to the hander.  3) Fix a typo in gracedb verbosity message that caused a crash

 - gstlal_inspiral_pipe and gstlal_svd_bank take as option
--sample-rate
 - the waveforms that are generated with a lower fhigh
are shifted forward such that all waveforms in a given
sub bank have a common amount of early warning
 - this has the effect that the actual fhigh of waveforms
is less than requested in a given sub bank, since the
waveforms are shifted according to the waveform that takes
longest to go from fhigh to ISCO

Signed-off-by: Surabhi Sachdev <surabhi.sachdev@ligo.org>

- the SNR segments are stored with float boundaries, so be careful to
  convert things to floats when testing to see if they are covered by the
  segments to avoid repeated type casts inside the internal bisection
  search comparison operation

- reduce the operation count in appsink_new_buffer()

- move more performance-critical methods into the C layer, this time the
  rich comparison methods

- recognizing that nothing uses the trigger class defined in
  snglinspiraltable except the trigger generator and its coincidence
  machinery, flatten the inheritence tree by one to improve attribute
  look-up performance

- move performance-critical methods to the C layer

- forgot a line from e3b38b0d

- use .template_id to check for template equality

- the coincidence code will now rely on the .template_id attribute to
  determine which template each trigger has come from.  this patch adds an
  assert to confirm that that is safe to do.

- a thinko was causing it to include one too many segments below the bound
  than required to satisfy the criterion.  it wouldn't lead to incorrect
  results, just a small slow-down.  ligo.segments now provides an
  implementation of this function, but that version of ligo.segments is not
  yet in production so we need to retain this implementation here for the
  time being.