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

Move url and accession column generation to ingest #77

Closed
j23414 opened this issue Dec 9, 2024 · 1 comment
Closed

Move url and accession column generation to ingest #77

j23414 opened this issue Dec 9, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@j23414
Copy link
Contributor

j23414 commented Dec 9, 2024

Description

Spinning this out as a separate issue from #73

While merging in USVI data with GenBank data, we have been creating new columns url and accession to ensure the link in the node callout works for both data sources. Currently, this column generation occurs during the phylogenetic workflow. We propose moving it to the ingest workflow for simplicity and better align with:

@j23414 j23414 added the enhancement New feature or request label Dec 9, 2024
@j23414
Copy link
Contributor Author

j23414 commented Jan 31, 2025

I think this was already completed in the merged PRs. I'm not finding remaining work that goes with this issue, but feel free to correct me.

@j23414 j23414 closed this as completed Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant