-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add ingest-to-phylogenetic GitHub Action... [#14] #34
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note the use of pathogen-repo-build
workflow requires adding the pathogen to our list of supported pathogens. See nextstrain/infra docs on how to add a new pathogen
👍 this is on my list/radar |
35d3cd4
to
79a5f9b
Compare
Fixed both sets of file name / bucket key name issues |
Done in nextstrain/infra#27 |
...to fully automate `ingest` and subsequent rebuild of `phylogenetic` when data has changed. Includes related configuration updates, as follows: * copy github action and associated build-configs from template * customize nextstrain-automation/config.yaml files for `ingest` and `phylogenetic` builds * add custom file loading to top-level Snakefiles in `ingest` and `phylogenetic` builds * update .gitignore to ignore newly-used `phylogenetic/data` * update phylogenetic/defaults/config.yaml to remove unused keys * update values of various `metadata` and `sequences` inputs/outputs in subsidiary 'phylogenetic/rules/*.smk' files
79a5f9b
to
f52855e
Compare
This was applied via a PR in the infra repo. Application:
|
...to fully automate
ingest
and subsequent rebuild ofphylogenetic
when data has changed. Includes related configuration updates, as follows:ingest
andphylogenetic
buildsingest
andphylogenetic
buildsphylogenetic/data
metadata
andsequences
inputs/outputs in subsidiary 'phylogenetic/rules/*.smk' filesDescription of proposed changes
Related issue(s)
Checklist