ocbe-uio / DIscBIO

A user-friendly R pipeline for biomarker discovery in single-cell transcriptomics
Other
12 stars 5 forks source link

Retry URLs on Networking and PPI functions #29

Closed wleoncio closed 3 years ago

wleoncio commented 3 years ago

Summary

The Networking() and PPI() functions use httr::GET() to parse URLs. They only try the URL one time, which sometimes fail due to network errors (a common problem among users with less-than-perfect internet connections). It would be great if these calls were wrapped around loops that try GET some (e.g. 3) times until they get a status_code() of 200, which means a successful URL retrieval.

To-dos

MRE

The code below lacks downloading steps for the scripts and data from the Binder.

library("DIscBIO")
source("DIscBIO-CTCs-Binder-Part1.r")
source("DIscBIO-CTCs-Binder-Part2.r")

Networking(data, FileName)
PPI(data, FileName)
wleoncio commented 3 years ago

Using code from DIscBIO-CTCs-Binder-Part1.ipynb and DIscBIO-CTCs-Binder-Part2.ipynb as bases, I'll try to come back to this with an MRE.

Update: Notebooks are bugged and apparently won't be fixed (see issue #30). Trying the full-blown notebook instead.

Update 2: MRE updated to OP.