hsvab / odbr

Pacote que disponibiliza dados de pesquisas OD do Brasil
https://hsvab.github.io/odbr/
GNU General Public License v3.0
15 stars 2 forks source link

Check if the internet is connected before downloading #35

Closed beatrizmilz closed 11 months ago

beatrizmilz commented 1 year ago

Haydee, essa é uma primeira ideia para que a função de download não gere um erro quando estiver sem internet!

diraol commented 1 year ago

@beatrizmilz não seria melhor, ao invés de verificar se tem internet ou não, só usar um "trycatch"? Isso resolveria tanto problemas de conexão quanto eventuais indisponibilidades do próprio github.

Ref

rafapereirabr commented 1 year ago

Oi pessoal. O comportamento que o CRAN exige é que o pacote retorne NULL e uma mensagem informativa quando há algum problema de conexão com a internet. Por isso, é necessario usar colocar a função de download dentro de um try(silent = TRUE, download_fun(x....)) para a função não dar erro quando tiver problema de conexão.

Se vocês quiserem, eu posso dar uma revisada no código atual do pacote e abrir um PR com uma solução como a que eu uso no censobr, geobr e noutros pacotes

hsvab commented 1 year ago

Agradeço super a ajuda @rafapereirabr! Seria ótimo adotar a mesma solução (que sabemos que funciona) do geobr e censobr!.

beatrizmilz commented 11 months ago

Fechei esse pull request pois o Rafa fez em #37