Harmonize ingest with pathogen repo guide - #35
Conversation
As part of conforming to the Pathogen Repo guide https://github.com/nextstrain/pathogen-repo-guide
As part of conforming to the Pathogen Repo guide https://github.com/nextstrain/pathogen-repo-guide
As part of conforming to the Pathogen Repo guide https://github.com/nextstrain/pathogen-repo-guide
As part of conforming to the Pathogen Repo guide https://github.com/nextstrain/pathogen-repo-guide
As part of conforming to the Pathogen Repo guide https://github.com/nextstrain/pathogen-repo-guide
As part of conforming to the Pathogen Repo guide https://github.com/nextstrain/pathogen-repo-guide
Match the upload rules in the pathogen-repo-guide https://github.com/nextstrain/pathogen-repo-guide/tree/main/ingest/build-configs/nextstrain-automation
Match pathogen-repo-guide nextstrain/pathogen-repo-guide@d9751bb
j23414
changed the title
joverlee521
left a comment
joverlee521
left a comment
There was a problem hiding this comment.
Changes look good to me!
I left comments on some additional clean up that I missed in nextstrain/zika@cee62cf and was also missed in #34
|
|
||
| configfile: "defaults/config.yaml" | ||
|
|
||
| send_slack_notifications = config.get("send_slack_notifications", False) |
There was a problem hiding this comment.
Missed this when I was reviewing nextstrain/zika@cee62cf, but this line can be removed since we are no longer sending Slack notifications.
There was a problem hiding this comment.
Good catch! Dropped slack related code and docs in 2e14f00
| @@ -57,12 +54,13 @@ rule all: | |||
| _get_all_targets, | |||
There was a problem hiding this comment.
Ditto missed this when I was reviewing nextstrain/zika@cee62cf, but I think the all rule can be simplified to the default targets and the entire _get_all_targets function can be removed.
rule all:
input:
expand(["results/sequences_{serotype}.fasta", "results/metadata_{serotype}.tsv"], serotype=serotypes)
There was a problem hiding this comment.
Ohh, I like the simplified default targets. Fixed in ed1bf1f
Should have been part of #34 Fixed here.
ed1bf1f to
2ffef24
Compare
2 participants
Description of proposed changes
Harmonize the ingest workflow to match the Pathogen Repo Guide. This may simplify reviewing the E_gene PR and Nextclade rules PR.
Related issue(s)
Checklist