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

rm get_reference_fp and ontology download deprecated functionality #3374

Open
sjanssen2 opened this issue Feb 23, 2024 · 4 comments
Open

rm get_reference_fp and ontology download deprecated functionality #3374

sjanssen2 opened this issue Feb 23, 2024 · 4 comments

Comments

@sjanssen2
Copy link
Contributor

sjanssen2 commented Feb 23, 2024

I think the local filenames of the GG 13.8 reference, used in the qp-target-sequencing plugin, is out of sync between qiita environment manager here:

files = {'tree': (get_reference_fp('gg_13_8-97_otus.tree'),
'ftp://ftp.microbio.me/greengenes_release/'
'gg_13_8_otus/trees/97_otus.tree'),
'taxonomy': (get_reference_fp('gg_13_8-97_otu_taxonomy.txt'),
'ftp://ftp.microbio.me/greengenes_release/'
'gg_13_8_otus/taxonomy/97_otu_taxonomy.txt'),
'sequence': (get_reference_fp('gg_13_8-97_otus.fasta'),
'ftp://ftp.microbio.me/greengenes_release/'
'gg_13_8_otus/rep_set/97_otus.fasta')}

and in the plugin itself:

https://github.com/qiita-spots/qp-target-gene/blob/858436ee6d4a1fccd35faee0b156a12953b5774a/qp_target_gene/__init__.py#L148-L149

I figure the plugin is correct, see this screenshot from qiita.ucsd.edu
image

@sjanssen2
Copy link
Contributor Author

hm, is there maybe even a way to overwrite these filepaths, provided by the plugin, when registering into qiita? Thus, the filepaths could point to those files that have actually downloaded through qiita-env make --download-references?!

@antgonza
Copy link
Member

In the original (like 10 years ago) Qiita design, the idea was that everything was self contained; however, we then realized that using plugins was the way to go. The old download-references is really deprecated.

Maybe this is a good opportunity, if you have the chance, to remove those old functionalities and just rely on the newest code. BTW this issue reminded me of this other one: #3197.

@antgonza
Copy link
Member

FWIW I think your effort might be better focus if we had a quick call to see what you are trying to achieve and maybe summarize all your open issues into one. Anyway, you have my email ...

@antgonza
Copy link
Member

After #3403 was merged, we should be able to easily and (almost) cleanly remove the old and deprecated ontology and reference download and processing; thus, changing the name of this issue so it can be addressed.

@antgonza antgonza changed the title qp-target-sequencing reference filenames out of sync? rm get_reference_fp and ontology download deprecated functionality May 15, 2024
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

No branches or pull requests

2 participants