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

MLST profile no more downloaded from pubmlst #117

Open
Hphilippon opened this issue Sep 7, 2020 · 1 comment
Open

MLST profile no more downloaded from pubmlst #117

Hphilippon opened this issue Sep 7, 2020 · 1 comment

Comments

@Hphilippon
Copy link

Hi,
I have tested MentaList on August 2020 and MLST profiles were correctly downloaded using the following command lines for exemple:
conda create -n mentalist
conda activate mentalist
conda install -c bioconda -c conda-forge mentalist
mentalist download_pubmlst -k 31 -o Ecoli_schemes -s 53 --db E_colie_1.db

But in September this does not work anymore because, I think, pubmlst.org has changed the fasta profile files names (all adk.tfa, fumC.tfa, gyrB.tfa, etc. are now named "sequence_fasta"). The downloaded files are then overwritten.

By modifying the line 96 of the "mlst_download_functions.jl" file from "filepath = _download_to_folder(locus_url, output_dir)" to "filepath = _download_to_folder(locus_url, output_dir, false, "$locus_name.tfa")" it works correctly again.
If this modification is correct, could you please update the mentalist code ?

Best regards,

HPhilippon

@slvrshot
Copy link

Hi,
I have tested MentaList on August 2020 and MLST profiles were correctly downloaded using the following command lines for exemple:
conda create -n mentalist
conda activate mentalist
conda install -c bioconda -c conda-forge mentalist
mentalist download_pubmlst -k 31 -o Ecoli_schemes -s 53 --db E_colie_1.db

But in September this does not work anymore because, I think, pubmlst.org has changed the fasta profile files names (all adk.tfa, fumC.tfa, gyrB.tfa, etc. are now named "sequence_fasta"). The downloaded files are then overwritten.

By modifying the line 96 of the "mlst_download_functions.jl" file from "filepath = _download_to_folder(locus_url, output_dir)" to "filepath = _download_to_folder(locus_url, output_dir, false, "$locus_name.tfa")" it works correctly again.
If this modification is correct, could you please update the mentalist code ?

Best regards,

HPhilippon

Do you have a fork of this for conda install? thx

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