search

YeoLab / flotilla

Reproducible machine learning analysis of gene expression and alternative splicing data
http://yeolab.github.io/flotilla/docs
BSD 3-Clause "New" or "Revised" License
121 stars 26 forks source link

Change the way links to gene lists are downloaded #236

Open mlovci opened 9 years ago

mlovci commented 9 years ago

Remove curl requirement: https://github.com/YeoLab/flotilla/blob/master/flotilla/util.py#L296

olgabot commented 9 years ago

What do you suggest to use instead? curl is pretty universal, and wget is not available on all machines. I haven't tried it, but it also may be tricky to use the python urllib* libraries for dropbox links since it takes you to a page and not the actual content.

mlovci commented 9 years ago

I was hoping urllib*. ... Maybe not possible.

On Sun, Jan 11, 2015 at 2:29 PM, Olga Botvinnik notifications@github.com wrote:

What do you suggest to use instead? curl is pretty universal, and wget is not available on all machines. I haven't tried it, but it also may be tricky to use the python urllib* libraries for dropbox links since it takes you to a page and not the actual content.

— Reply to this email directly or view it on GitHub https://github.com/YeoLab/flotilla/issues/236#issuecomment-69515046.

olgabot commented 9 years ago

a PR with a fix would be welcomed :)

On Sun Jan 11 2015 at 3:38:08 PM Michael Lovci notifications@github.com wrote:

curl isn't default on ubuntu.

I was hoping urllib*. ... Maybe not possible.

On Sun, Jan 11, 2015 at 2:29 PM, Olga Botvinnik notifications@github.com

wrote:

What do you suggest to use instead? curl is pretty universal, and wget is not available on all machines. I haven't tried it, but it also may be tricky to use the python urllib* libraries for dropbox links since it takes you to a page and not the actual content.

— Reply to this email directly or view it on GitHub https://github.com/YeoLab/flotilla/issues/236#issuecomment-69515046.

— Reply to this email directly or view it on GitHub https://github.com/YeoLab/flotilla/issues/236#issuecomment-69517812.