Skip to content

Commit 27ed060

Browse files
committed
move missing AF/DP tags earlier in arg check
1 parent 890edd6 commit 27ed060

File tree

1 file changed

+17
-16
lines changed

1 file changed

+17
-16
lines changed

pcgr/arg_checker.py

Lines changed: 17 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,23 @@ def verify_args(arg_dict):
4949
err_msg = f"Tumor ploidy value ('--tumor_ploidy' = {arg_dict['tumor_ploidy']}) must be > 0"
5050
error_message(err_msg, logger)
5151

52+
## check that allelic support tags are set correctly when minimum/maximum depth/allelic fractions are set
53+
if arg_dict['tumor_dp_tag'] == "_NA_" and arg_dict['tumor_dp_min'] is not None:
54+
err_msg = f"Minimum sequencing depth tumor ('tumor_dp_min' = {arg_dict['tumor_dp_min']}) requires '--tumor_dp_tag' to be set"
55+
error_message(err_msg, logger)
56+
57+
if arg_dict['tumor_af_tag'] == "_NA_" and (arg_dict['tumor_af_min'] is not None or arg_dict['tumor_ad_min'] is not None):
58+
err_msg = f"Minimum AF/AD tumor ('tumor_af_min' = {arg_dict['tumor_af_min']}, 'tumor_ad_min' = {arg_dict['tumor_ad_min']}) requires '--tumor_af_tag' to be set"
59+
error_message(err_msg, logger)
60+
61+
if arg_dict['control_dp_tag'] == "_NA_" and arg_dict['control_dp_min'] is not None:
62+
err_msg = f"Minimum sequencing depth control ('control_dp_min' = {arg_dict['control_dp_min']}) requires '--control_dp_tag' to be set"
63+
error_message(err_msg, logger)
64+
65+
if arg_dict['control_af_tag'] == "_NA_" and (arg_dict['control_af_max'] is not None or arg_dict['control_ad_max'] is not None):
66+
err_msg = f"Maximum AF/AD control ('control_af_max' = {arg_dict['control_af_max']}, 'control_ad_max' = {arg_dict['control_ad_max']}) requires '--control_af_tag' to be set"
67+
error_message(err_msg, logger)
68+
5269
# check that minimum/maximum depth/allelic fractions are set correctly
5370
dp_tumor_set = False
5471
if arg_dict['tumor_dp_min'] is not None:
@@ -135,22 +152,6 @@ def verify_args(arg_dict):
135152
err_msg = f"Minimum AF (tumor) for TMB calculation ('tmb_af_min' = {str(arg_dict['tmb_af_min'])}) must be "
136153
err_msg += f"greater or equal to minimum AF tumor ({str(arg_dict['tumor_af_min'])}, i.e. global filter for variant inclusion in report)"
137154
error_message(err_msg, logger)
138-
139-
if arg_dict['tumor_dp_tag'] == "_NA_" and arg_dict['tumor_dp_min'] is not None:
140-
err_msg = f"Minimum sequencing depth tumor ('tumor_dp_min' = {arg_dict['tumor_dp_min']}) requires '--tumor_dp_tag' to be set"
141-
error_message(err_msg, logger)
142-
143-
if arg_dict['tumor_af_tag'] == "_NA_" and (arg_dict['tumor_af_min'] is not None or arg_dict['tumor_ad_min'] is not None):
144-
err_msg = f"Minimum AF/AD tumor ('tumor_af_min' = {arg_dict['tumor_af_min']}, 'tumor_ad_min' = {arg_dict['tumor_ad_min']}) requires '--tumor_af_tag' to be set"
145-
error_message(err_msg, logger)
146-
147-
if arg_dict['control_dp_tag'] == "_NA_" and arg_dict['control_dp_min'] is not None:
148-
err_msg = f"Minimum sequencing depth control ('control_dp_min' = {arg_dict['control_dp_min']}) requires '--control_dp_tag' to be set"
149-
error_message(err_msg, logger)
150-
151-
if arg_dict['control_af_tag'] == "_NA_" and (arg_dict['control_af_max'] is not None or arg_dict['control_ad_max'] is not None):
152-
err_msg = f"Maximum AF/AD control ('control_af_max' = {arg_dict['control_af_max']}, 'control_ad_max' = {arg_dict['control_ad_max']}) requires '--control_af_tag' to be set"
153-
error_message(err_msg, logger)
154155

155156
# Check that coding target size region of sequencing assay is set correctly
156157
if float(arg_dict['effective_target_size_mb']) < 0 or float(arg_dict['effective_target_size_mb']) > float(pcgr_vars.CODING_EXOME_SIZE_MB):

0 commit comments

Comments
 (0)