Skip to content

Commit cafa55b

Browse files
dpark01claude
andcommitted
Re-add --skipRealign as deprecated no-op for WDL backward compatibility
All 9 measles-usa Terra failures were caused by the WDL still passing --skipRealign to align_and_fix. Accept the argument but ignore it since GATK realignment has been removed. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 8c48956 commit cafa55b

1 file changed

Lines changed: 7 additions & 1 deletion

File tree

src/viral_ngs/read_utils.py

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1249,7 +1249,8 @@ def align_and_fix(
12491249
threads=None,
12501250
skip_mark_dupes=False,
12511251
dup_marker='sambamba',
1252-
novoalign_license_path=None
1252+
novoalign_license_path=None,
1253+
skip_realign=False,
12531254
):
12541255
''' Take reads, align to reference with Novoalign, minimap2, or BWA-MEM.
12551256
Optionally mark duplicates with Picard or sambamba,
@@ -1388,6 +1389,11 @@ def parser_align_and_fix(parser=argparse.ArgumentParser()):
13881389
dest="novoalign_license_path",
13891390
help='A path to the novoalign.lic file. This overrides the NOVOALIGN_LICENSE_PATH environment variable. (default: %(default)s)'
13901391
)
1392+
parser.add_argument('--skipRealign',
1393+
dest="skip_realign",
1394+
action='store_true',
1395+
default=False,
1396+
help='Deprecated no-op. GATK realignment has been removed.')
13911397

13921398
util_cmd.common_args(parser, (('loglevel', None), ('version', None), ('tmp_dir', None)))
13931399
util_cmd.attach_main(parser, align_and_fix, split_args=True)

0 commit comments

Comments
 (0)