Skip to content

add exclude parameter to filter_subsample_sequences task and augur_from_msa workflow; improve handling of whitespace, empty lines, or absent last-line trailing newline in input include/exclude files#586

Open
tomkinsc wants to merge 2 commits intomasterfrom
ct-expose-augur_from_msa-exclude
Open

add exclude parameter to filter_subsample_sequences task and augur_from_msa workflow; improve handling of whitespace, empty lines, or absent last-line trailing newline in input include/exclude files#586
tomkinsc wants to merge 2 commits intomasterfrom
ct-expose-augur_from_msa-exclude

Conversation

@tomkinsc
Copy link
Copy Markdown
Contributor

@tomkinsc tomkinsc commented Mar 5, 2025

  • add an exclude_list parameter to the filter_subsample_sequences task for use in the augur_from_msa workflow
    • (exclusions are processed after considering IDs designated explicitly as those to keep)
  • add optional inputs to filter_subsample_sequences so include/exclude IDs can be specified in string inputs as space-delimited lists
    • (such IDs are combined with any coming from corresponding include/exclude files)
  • improve handling of whitespace, empty lines, or absent last-line trailing newline in input include/exclude files

This does not alter the prior behavior that input sequences are passed through to the output unaltered if no filtering is specified.

…om_msa workflow; improve handling of whitespace, empty lines, or absent last-line trailing newline in input include/exclude files

add an `exclude_list` parameter to the `filter_subsample_sequences` task for use in the `augur_from_msa` workflow (exclusions are processed after considering IDs designated explicitly as those to keep); add optional inputs to `filter_subsample_sequences` so include/exclude IDs can be specified in string inputs as space-delimited lists (such IDs are combined with any coming from corresponding include/exclude files); improve handling of whitespace, empty lines, or absent last-line trailing newline in input include/exclude files
@tomkinsc tomkinsc requested a review from dpark01 March 5, 2025 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant