Skip to content
Snippets Groups Projects

Bug fixes and typo fixes

Merged Shio Sakon requested to merge tom_mass_model into master

There were a couple bugs in the previous merge request made to add the lvc rates and pop codes into gstlal-ugly. This is a merge request to fix those bugs, as well as typos found in the codes.

  • gstlal-ugly/bin/gstlal_lvc_rates_injection_dag: fixed the executable in the script from lvc_rates_injection_dag to gstlal_lvc_rates_injection_dag
  • gstlal-ugly/bin/gstlal_lvc_rates_injections: fixed the executable in the script from lvc_rates_injection_dag to gstlal_lvc_rates_injection_dag
  • gstlal-ugly/bin/gstlal_lvc_rates_injections: fix the description of option snr-threshold from sayng "Set the single detector snr threshold" and "default = 3.0" to "Set the network snr threshold" and "default = 5.0"
  • gstlal-ugly/python/injection_generation_dag.py: remove from gstlal.dags.util import pipeline_dot_py_append_opts_hack and adjust the script
  • gstlal-ugly/python/injection_utils.py: clean up for aesthetic reasons

Merge request reports

Pipeline #369375 passed with warnings

Pipeline passed with warnings for 8bdab589 on tom_mass_model

Approved by

Merged by Shio SakonShio Sakon 3 years ago (Mar 15, 2022 2:03am UTC)

Pipeline #369914 passed with warnings

Pipeline passed with warnings for 8bdab589 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading