Skip to content
Snippets Groups Projects
Commit 26d99771 authored by Kipp Cannon's avatar Kipp Cannon Committed by Kipp Cannon
Browse files

gstlal_inspiral: disable dist stats files for injection jobs

- this is a temporary hack to stop the collection of dist stats files for injection jobs.  this should put an end to the endless troubles injection jobs give us in this regard.
parent 3b9b7273
No related branches found
No related tags found
No related merge requests found
......@@ -780,7 +780,7 @@ for output_file_number, (svd_bank_url_dict, output_url, likelihood_url_namedtupl
rankingstat = rankingstat,
zerolag_rankingstatpdf_filename = zerolag_rankingstatpdf_filename,
rankingstatpdf_filename = options.marginalized_likelihood_file,
likelihood_url_namedtuple = likelihood_url_namedtuple,
likelihood_url_namedtuple = likelihood_url_namedtuple if not options.injections else None,
likelihood_snapshot_interval = options.likelihood_snapshot_interval, # seconds
thinca_interval = options.thinca_interval,
min_log_L = options.min_log_L,
......
......@@ -567,6 +567,22 @@ class Data(object):
# might be configured to disable ranking statistic output
# since they produce nonsense.
#
# FIXME: gstlal_inspiral now sets the likelihood_url to
# None when doing injections to disable the collection of
# ranking statistic information in injection jobs. this is
# now yet another way in which online analyses are broken
# following the ranking statistic rewrite and will need to
# be looked at. the reason for doing this is that because
# injection jobs skip intervals of SNR reconstruction they
# create ranking statistic data that appear to be
# inconsistent with the triggers that are being produced
# and the inconsistency triggers assertion failures
# throughout the new ranking statistic code. we have
# tried, in vain, to work around the problem but in the end
# we've had to simply stop injection jobs from even trying.
# the data they collect was never used anyway, exactly
# because of its inconsistencies.
#
if likelihood_url_namedtuple is not None:
self.reference_likelihood_url = likelihood_url_namedtuple.reference_likelihood_url
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment