Use lal.GPSTimeNow to get current time instead of lalapps_tconvert

See merge request lscsoft/gstlal!9

instead of calling out to lalapps_tconvert

Makefile.gstlal_feature_extractor_offline: fix issue with generating segments from int8s conversion, add default safety info, clean up comments and unused variables

gstlal_feature_extractor_pipe: link feature combiner job to run after all feature extraction jobs are finished

gstlal_feature_extractor_pipe: fix some issues with changed imports, update params in offline makefile

- there was a time when streamthinca needed to define a special class to be
  used for triggers to allow the coincidence algorithm to work, and so it
  made sense for the definition to live in streamthinca where the
  requirements were defined.  today no special class is required, the
  underlying trigger class is simply used as-is, but code like lloidhandler
  is still importing the symbol from streamthinca instead of importing it
  themselves directly.  this patch untangles this now gratuituous
  interdependency.

update O3 optimized Makefiles: bump ligo-scald to 0.3.0 and build from tarball, remove urllib3 from build

	--move template making step to svd_bank_snr.FIR_SNR.make_template()
	--calculte the template duration instead of inputting template duration
	--correct snr's epoch by adding time offset due to conditioning of templates

ChiWai Chan authored 5 years ago and Kipp Cannon committed 5 years ago

- fix peakoffset determination (get from trigger, don't try to find it in
  the SNR time series, that doesn't work if there are other peaks)