Skip to content
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

Rename metadata column from isolate_id to gisaid_epi_isl #179

Merged
merged 2 commits into from
Oct 10, 2024

Conversation

joverlee521
Copy link
Contributor

Description of proposed changes

The fields to export in acknowledgements is hardcoded in Auspice¹ so the
metadata field needs to be named gisaid_epi_isl to be included in
the acknowledgements for the seasonal flu builds. Renaming the column
in our upload config to avoid having to rename it as part of the
phylogenetic workflow.

¹ https://github.com/nextstrain/auspice/blob/dae78e74d5e06303d901387adf9fe7550a7d4aa9/src/components/download/helperFunctions.js#L326

Related issue(s)

First step for #178

Checklist

  • Checks pass

The fields to export in acknowledgements is hardcoded in Auspice¹ so the
metadata field needs to be named `gisaid_epi_isl` to be included in
the acknowledgements for the seasonal flu builds. Renaming the column
in our upload config to avoid having to rename it as part of the
phylogenetic workflow.

¹ <https://github.com/nextstrain/auspice/blob/dae78e74d5e06303d901387adf9fe7550a7d4aa9/src/components/download/helperFunctions.js#L326>
@joverlee521 joverlee521 mentioned this pull request Sep 10, 2024
1 task
@joverlee521
Copy link
Contributor Author

I don't think there's any rush to merge this, so I'll plan to merge after I'm back from vacation to monitor this and #178.

@joverlee521 joverlee521 merged commit b0ff43d into master Oct 10, 2024
3 checks passed
@joverlee521 joverlee521 deleted the upload-gisaid-epi-isl branch October 10, 2024 16:48
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.

2 participants