-
Notifications
You must be signed in to change notification settings - Fork 174
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
Allow for fuzzy match of assembly summary header #210
Conversation
Hello, I got the same error as #209, is it fixed? |
@SolMiele Did you try installing this pull request?
Note: I haven't fully tested it but it did work with |
It works for me and downloaded 40,000 bacterial genomes from genbank and refseq |
Hello! I tried this : pip uninstall ncbi-genome-download pip install git+https://github.com/chasemc/ncbi-genome-download@4c5c24e27aa8123c7995890046b17b815da3571b and my command like this: ncbi-genome-download --assembly-accessions "GCA_001951075.2" bacteria,archaea -l "complete" --section genbank --formats fasta --flat-output -o ./ but it does not work. So , what should I do ? Can you hepl me ? The same error as #209, is it fixed? |
Hello, I tried the and then tried Seems to be working to some point but I also get an error message at the end. @ialbert Please see the following. Using
|
The proposed fix should work, so it's likely that it's an issue with installing the PR. Python is pretty notoriously difficult when dealing with package versions and I'm no expert pip's installation details but that's likely why it seems it's not working. Change the my_directory variable below to point to a path on your computer
Then |
@nush320 that is a different issue related to implementation (without the pipe to |
@chasemc as a temporary solution I used your changes to patch the Bioconda recipe |
Thanks for this, I ended up solving it slightly different in bd46071. Should be fixed in 0.3.2 |
fixes #209