Skip to content

Commit

Permalink
Update CLI checks for duration bins
Browse files Browse the repository at this point in the history
  • Loading branch information
titodalcanton committed Sep 12, 2023
1 parent 24633c8 commit 745d9a3
Showing 1 changed file with 14 additions and 15 deletions.
29 changes: 14 additions & 15 deletions bin/live/pycbc_live_single_trigger_fits
Original file line number Diff line number Diff line change
Expand Up @@ -139,22 +139,21 @@ if any(prune_options) and not all(prune_options):
"--prune-window and --prune-stat-threshold")

# Check the bin options
if args.duration_bin_edges and (args.duration_bin_start or
args.duration_bin_end or
args.duration_from_bank or
args.num_duration_bins):
# duration bin edges specified as well as linear/logarithmic
parser.error("Cannot use --duration-bin-edges with "
"--duration-bin-start, --duration-bin-end, "
"--duration-from-bank or --num-duration-bins.")

if not args.duration_bin_edges and not (args.duration_bin_start and
args.duration_bin_end and
args.num_duration_bins):
parser.error("--duration-bin-start, --duration-bin-end and "
"--num-duration-bins must be set if not using "
if args.duration_bin_edges:
if (args.duration_bin_start or args.duration_bin_end or
args.duration_from_bank or args.num_duration_bins):
parser.error("Cannot use --duration-bin-edges with "
"--duration-bin-start, --duration-bin-end, "
"--duration-from-bank or --num-duration-bins.")
else:
if not args.num_duration_bins:
parser.error("--num-duration-bins must be set if not using "
"--duration-bin-edges.")
if not ((args.duration_bin_start and args.duration_bin_end) or
args.duration_from_bank):
parser.error("--duration-bin-start & --duration-bin-end or "
"--duration-from-bank must be set if not using "
"--duration-bin-edges.")

if args.duration_bin_end and \
args.duration_bin_end <= args.duration_bin_start:
parser.error("--duration-bin-end must be greater than "
Expand Down

0 comments on commit 745d9a3

Please sign in to comment.