Skip to content

Input IFOs from config file in tisi creation line

Shomik Adhicary requested to merge online-inspiral-tempate-tisi-ifo-change into master

The line in the Makefile.online_inspiral_template used to create tisi.xml had all the IFOs explicitly written. This change makes it so that the IFOs specified in the config.yml file are only used in creation of tisi.xml.

Not having tisi.xml generated with only the IFOs specified resulted in gstlal_inspiral jobs failing with this error:

^[[34mINFO:   ^[[0m Environment variable SINGULARITY_BIND is set, but APPTAINER_BIND is preferred
^[[34mINFO:   ^[[0m Environment variable SINGULARITY_PWD is set, but APPTAINER_PWD is preferred
WARNING: Overriding HOME environment variable with APPTAINERENV_HOME is not permitted
/usr/bin/gstlal_inspiral:478: UserWarning: It is not advised to choose PSD lengths below 32s, since a loss in sensitivity may occur
  warnings.warn("It is not advised to choose PSD lengths below 32s, since a loss in sensitivity may occur")
/usr/lib64/python3.6/site-packages/gstlal/httpinterface.py:134: UserWarning: disabling service discovery, this web server won't be able to advertise the location of the services it provides.
  warnings.warn("disabling service discovery, this web server won't be able to advertise the location of the services it provides.")
Traceback (most recent call last):
  File "/usr/bin/gstlal_inspiral", line 947, in <module>
    raise ValueError("\"%s\" is for %s but we need %s" % (options.ranking_stat_input, ", ".join(sorted(rankingstat.instruments)), ", ".join(sorted(all_instruments))))
ValueError: "dist_stats/H1L1-0368_GSTLAL_DIST_STATS-0-0.xml.gz" is for H1, L1 but we need H1, L1, V1
~                                                                                                       

This was discovered while launching Renee & Esme.

It was tested here: /ligo/home/ligo.org/shomik.adhicary/small-online-analysis/run/small-bbh by changing the instruments in config_small_bbh.yml. The Makefile generated only had the instruments specified in the correct format.

This is conditional on tisi.xml doesn't affect anything else downstream

Edited by Shomik Adhicary

Merge request reports