Skip to content

153 restrict sequences#174

Merged
nilsoberg merged 4 commits into
nextflow-testfrom
153-restrict-sequences
Apr 28, 2025
Merged

153 restrict sequences#174
nilsoberg merged 4 commits into
nextflow-testfrom
153-restrict-sequences

Conversation

@nilsoberg
Copy link
Copy Markdown
Collaborator

Add filter to restrict sequences in FASTA or Accession input files to a specific family(s).

- Add code to retrieve all attributes for all IDs in the input dataset,
  used to filter
- Add tests, one of which should fail
@nilsoberg nilsoberg self-assigned this Apr 24, 2025
@nilsoberg nilsoberg linked an issue Apr 24, 2025 that may be closed by this pull request
@nilsoberg nilsoberg requested a review from rbdavid April 24, 2025 16:37
Copy link
Copy Markdown
Contributor

@rbdavid rbdavid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread lib/EFI/Import/Filter/Family.pm Outdated
Comment thread tests/modules/04l_est_accession_filter_family_fail_empty.sh
@nilsoberg nilsoberg merged commit 0f0c1fc into nextflow-test Apr 28, 2025
@nilsoberg nilsoberg deleted the 153-restrict-sequences branch April 28, 2025 14:17
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.

Restrict sequences to specified family(s) (option C and D only)

2 participants