Kipp Cannon authored 5 years ago and Patrick Godwin committed 5 years ago

- this mostly reverts the rest of 70c88cd6
  "use private fields in pkgconfig files"
- the requirements of each of the gstlal libraries are not private, it is
  through these dependencies that the correct -I options are added when
  compiling things that #include the header files.  NOTE:  the header files
  in, for example, gstlal #include header files from other
  projects, header files that are not, in general, located in the default
  search paths of compilers.  pkg-config is required to determine the
  complete set of -I options, it does so using the "requires" field.
- 70c88cd6 "use private fields in pkgconfig
  files" didn't break gstlal builds because the gstlal packages,
  themselves, tend to have the same internal requirements as the exported
  requirements of other gstlal packages.  for example, using gstlal's
  header files requires you to add the -I options necessary to find all the
  gstreamer headers, which everything in gstlal-inspiral needs to add
  anyway, for its own reasons, so breaking gstlal's .pc file didn't break
  the gstlal-inspiral build.  it *does* break builds of other stuff,
  outside of these projects, that might try to link to gstlal but isn't a
  gstreamer based project itself.
- what this patch does do, is conduct an audit of what things are really
  #include'd in installed header files and only lists those things in the
  requires field instead of all libraries the project links against.

Kipp Cannon authored 5 years ago and Patrick Godwin committed 5 years ago

- this fixes a maintainance problem introduced by 70c88cd6 "use private fields in pkgconfig files"

Chad Hanna authored 5 years ago and Patrick Godwin committed 5 years ago

gstlal_inspiral: make sure anything with a reference to pipeline is freed before setting NULL - NOTE check in on bottle route stuff. Still not sure it is freeing everything it should be in each loop

Daichi Tsuna authored 5 years ago and Patrick Godwin committed 5 years ago

especially for Virgo, as the PSD is significantly underestimated at <10Hz.
This ad-hoc high-pass filter might not be the optimal way, and we might have to come back later.

Daichi Tsuna authored 5 years ago and Patrick Godwin committed 5 years ago

as the PSD shape differs between H,L and Virgo

ChiWai Chan authored 5 years ago and Patrick Godwin committed 5 years ago

and update docs.

ChiWai Chan authored 5 years ago and Patrick Godwin committed 5 years ago

- save template autocorrelation
- change output naming

Daichi Tsuna authored 5 years ago and Patrick Godwin committed 5 years ago

I/O problems, problems when merging rankingstat objects from different jobs

Daichi Tsuna authored 5 years ago and Patrick Godwin committed 5 years ago

binning (chi^2 => chi^2/rho^2) & smaller Gaussian kernel

Daichi Tsuna authored 5 years ago and Patrick Godwin committed 5 years ago

fixed bug of triggergen dumping coincs into rankstat denominator when analyzing data that include injections

Chad Hanna authored 5 years ago and Patrick Godwin committed 5 years ago

(cherry picked from commit 0f1b47c4)

Chad Hanna authored 5 years ago and Patrick Godwin committed 5 years ago

(cherry picked from commit 80a2dd8e)

Chad Hanna authored 5 years ago and Patrick Godwin committed 5 years ago

inspiral.py: change the naming convention for gracedb files to remove redundent instrument information


(cherry picked from commit b7cf6c74)

ChiWai Chan authored 5 years ago and Patrick Godwin committed 5 years ago

change --sub-bank-id to --bank-number to avoid confusion with bank_id;
clarify --save options;
remove __epilog__ from --help output.

inspiral_extrinsics.py: make TimePhaseSNR a class property rather than an instance property for InspiralExtrinsics()

This reverts commit b95d82df.

gstlal_ll_inspiral_event_uploader: reword some logging messages, only fill in 'time_sent' entry once

gstlal_ll_inspiral_event_uploader: modify logging messages, fix issue causing a crash when multiple candidates per event are uploaded