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

Making it easier for people to know they need Entrez Direct. #2

Merged
merged 1 commit into from
May 5, 2015

Conversation

adamnovak
Copy link
Contributor

I didn't have Entrez Direct installed, so I was getting a bunch of empty files and no error messages.

I fixed the script to stop if the Entrez Direct tools don't work (or anything else bad happens) and added a note about the dependency.

Also, I was only running the scripts in the first place because the Arvados dataset seems to itself be a collection of empty files today. I know that that wasn't the case before, since I peeked at the FASTAs there and they had data. Has the dataset expired or something?

nouyang-curoverse added a commit that referenced this pull request May 5, 2015
Making it easier for people to know they need Entrez Direct.
@nouyang-curoverse nouyang-curoverse merged commit 8f9c580 into nouyang-curoverse:master May 5, 2015
nouyang-curoverse added a commit that referenced this pull request May 5, 2015
@nouyang-curoverse
Copy link
Owner

@adamnovak Apologies for the delayed response, I hadn't set up my github notifications correctly. Thanks for the bug report & pull request (merged the latter) and sorry about all the issues you've run into.

I easily replicated the bug, so I created new URLs and updated my Readme.md, and also filed a bug report on Arvados:
https://arvados.org/issues/5915.

tl;dr I'm not sure why the link gives you 0 byte files, but try the new links :)

@adamnovak
Copy link
Contributor Author

The new links seem to work. Thanks!

On Tue, May 5, 2015 at 2:26 PM, nouyang-curoverse notifications@github.com
wrote:

@adamnovak https://github.com/adamnovak Apologies for the delay, I
hadn't set up my github notifications correctly. Thanks for the bug report
& pull request (merged the latter) and sorry about all the issues you've
run into.

I created new URLs and updated my Readme.md, and also filed a bug report
on Arvados:
https://arvados.org/issues/5915.

tl;dr I'm not sure why the link gives you 0 byte files, but trying the new
links


Reply to this email directly or view it on GitHub
#2 (comment)
.

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

Successfully merging this pull request may close these issues.

None yet

2 participants