Skip to content

Added option --sort-by in dag layers and template Makefiles

Stefano Schmidt requested to merge add_svd_sort_in_config into master

I added support to set the option --sort-by in gstlal_inspiral_bank_splitter from config file. Two changes were made:

  1. Edited template Makefiles (full dag and filtering dag)
  2. Edited the dag layer corresponding to gstlal_inspiral_bank_splitter

As no default for this option was set, this change affects the way the config file shall be built. It will become compulsory to specify the option sort-by: for every sub-banks included in the analysis.

This may not be wanted and we could consider setting a default value for the sort-by option in the Makefile template. However, a default is not specified for any other parameter, so this will be the first time we set such behaviour.

Once this merge request is approved, I should update accordingly the doc page.

Merge request reports