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

feat: BV-BRC support #128

Closed
j23414 opened this issue Oct 14, 2022 · 2 comments
Closed

feat: BV-BRC support #128

j23414 opened this issue Oct 14, 2022 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@j23414
Copy link
Contributor

j23414 commented Oct 14, 2022

Context

ViPR is being replaced with BV-BRC with final transfer possibly occurring by the end of the year (2022-12-31) which might disrupt pathogen builds.

BV-BRC_screenshot

Description

For any pathogen builds depending on ViPR (such as zika), we should make sure we can do one (or a combination) of the following:

  • pull the equivalent GenomicFastaResults.fasta
  • modify any required scripts for BV-BRC datasets

Possible solution

So far the user interface seems pretty equivalent:

BV-BRC_zika

BV-BRC_zika_download

However we may need to modify the fasta headers:

Notice how the header field is different from ViPR and not modifiable like ViPR. (example from dengue)

head BVBRC_genome_sequence.fasta
>accn|KY829115   Dengue virus 1 isolate H.sapiens-wt/BLM/2016/MA-WGS16-006-SER, complete genome.   [Dengue virus 1 H.sapiens-wt/BLM/2016/MA-WGS16-006-SER strain Dengue virus 1/H.sapiens-wt/BLM/2016/MA-WGS16-006-SER | 11053.9479]
agttgttagtctacgtggaccgacaagaacagtttcgaatcggaagcttgcttaacgtag
ttctaacagttttttattagagagcagatctctgatgaacaaccaacggaaaaagacggg
tcgaccgtctttcaatatgctgaaacgcgcgagaaaccgcgtgtcaactggttcacagtt

Instead we may need to download and process the tabular data to match our current GenomicFastaResults.fasta headers:

example from Zika

less GenomicFastaResults.fasta
>KY241742|ZIKV_SG_072|NA|2016_08_28|Human|Singapore|Asian|Zika_virus
GAATCAGACTGCGACAGTTCGAGTTTGAAGCGAAAGCTAGCAACAGTATCAACAGGTTTTATTTTGGATT
TGGAAACGAGAGTTTCTGGTCATGAAAAACCCAAAAAAGAAATCCGGAGGATTCCGGATTGTCAATATGC
TAAAACGCGGAGTAGCCCGTGTGAGCCCCTTTGGGGGCTTGAAGAGGCTGCCAGCCGGACTTCTGCTGGG
TCATGGGCCCATCAGGATGGTCTTGGCGATTCTAGCCTTTTTGAGGTTCACGGCAATCAAGCCATCACTG

Alternatively, someone could do a deep dive into BV-BRC cli

@j23414 j23414 added the enhancement New feature or request label Oct 14, 2022
@j23414 j23414 self-assigned this Oct 14, 2022
@j23414
Copy link
Contributor Author

j23414 commented Feb 27, 2023

Just a note that ViPR website redirects to BV-BRC. Ergo any ViPR instructions may be out of date

@j23414 j23414 mentioned this issue Nov 21, 2023
1 task
@j23414
Copy link
Contributor Author

j23414 commented Jul 23, 2024

Closing this, since instead of adding BV-BRC support, we've gone in the direction of using Entrez or NCBI datasets in the pathogen repo-guide.

@j23414 j23414 closed this as completed Jul 23, 2024
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
No open projects
Status: In Progress
Development

No branches or pull requests

1 participant