Skip to content

Commit

Permalink
Fix Existing Cohort Parser Sequencing Type Param (#679)
Browse files Browse the repository at this point in the history
* Fix click overwriting param to None when not specified

* Fix whitespace after : for the linter
  • Loading branch information
vivbak authored Feb 12, 2024
1 parent bea77e3 commit 81644b0
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion scripts/parse_existing_cohort.py
Original file line number Diff line number Diff line change
Expand Up @@ -216,6 +216,7 @@ def get_existing_external_sequence_ids(self, participant_map: dict[str, dict]):
'--sequencing-type',
type=click.Choice(['genome', 'exome']),
help='Sequencing type: genome or exome',
default='genome',
)
@click.option('--search-location', 'search_locations', multiple=True)
@click.option(
Expand All @@ -239,11 +240,11 @@ async def main(
project: str,
search_locations: List[str],
batch_number: Optional[str],
sequencing_type: str,
confirm=True,
dry_run=False,
include_participant_column=False,
allow_missing_files=False,
sequencing_type: str = 'genome',
):
"""Run script from CLI arguments"""

Expand Down

0 comments on commit 81644b0

Please sign in to comment.