Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Refactor phylogenetic such that a reasonable phylogenetic/build-configs/ci works #12

Closed
3 tasks done
j23414 opened this issue Sep 24, 2024 · 1 comment
Closed
3 tasks done
Assignees
Labels
enhancement New feature or request

Comments

@j23414
Copy link
Collaborator

j23414 commented Sep 24, 2024

Context

While adding
https://github.com/nextstrain/pathogen-repo-guide/tree/main/phylogenetic/build-configs/ci
realized phylogenetic was reading directly from ingest/results/ in

input_metadata: "../ingest/results/metadata_all.tsv"
input_sequences: "../ingest/results/sequences_all.fasta"

This may need to be refactored to work with our automation setup.

Description

  • Add example sequences
  • Instead of taking ingest/results/* as input, add a download or copy rule to prepare_sequences.smk
  • Add phylogenetic/build-configs/ci check

Examples

Possible solution

(Optional)

@j23414 j23414 added the enhancement New feature or request label Sep 24, 2024
@j23414 j23414 self-assigned this Sep 24, 2024
@j23414
Copy link
Collaborator Author

j23414 commented Oct 11, 2024

Closed since the last task was completed at ec5dd33

@j23414 j23414 closed this as completed Oct 11, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant