simple-R-shiny icon indicating copy to clipboard operation
simple-R-shiny copied to clipboard

Pull from rocker?

Open ateucher opened this issue 8 years ago • 6 comments

Should we use the dockerfiles from rocker? Eg. shiny or at the very least r-base. Perhaps even better we should use a versioned base from rocker/r-ver for which it looks like they recently added shiny support.

ateucher avatar May 05 '17 16:05 ateucher

The only thing I would say is .. if it's not broken why fix it? Or, instead of changing this repo, make new one.

Chris

On Fri, May 5, 2017 at 9:29 AM Andy Teucher [email protected] wrote:

Should we use the dockerfiles from rocker? Eg. shiny https://hub.docker.com/r/rocker/shiny/ or at the very least r-base https://hub.docker.com/r/rocker/r-base/. Perhaps even better we should use a versioned base from rocker/r-ver https://hub.docker.com/r/rocker/r-ver/ for which it looks like they recently added shiny support https://github.com/rocker-org/rocker-versioned/tree/master/rstudio.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/bcgov/simple-R-shiny/issues/11, or mute the thread https://github.com/notifications/unsubscribe-auth/AAEtDITXpXy33vh6ck4N9WVoIkbqQ_4cks5r205dgaJpZM4NSI9j .

ccoldwell avatar May 05 '17 17:05 ccoldwell

Well it's a little bit broken ;) When the latest stable R version changes (as it recently did to 3.4) we have to change the version in the Dockerfile... if we pulled from r-base it would always be up to date. Using a versioned base Dockerfile would make it even more stable, as it will always pull a specific version of R as well as dependent R packages... though we could also expect people to keep their apps up date!

ateucher avatar May 05 '17 17:05 ateucher

Rocker essentially became a standard in using R with Docker and I think it makes sense to update the repo to reflect this. It would also drastically reduce the time to build the image.

sasha-ruby avatar Mar 20 '18 04:03 sasha-ruby

Thanks @sasha-ruby - I'm afraid this repo is a bit out of date. The repo here: https://github.com/bcgov/land-designations-shinyapp has an implementation that has worked well for us and uses the rocker r-base image. I haven't been able to get around to updating this one.

ateucher avatar Mar 20 '18 16:03 ateucher

@sasha-ruby see also this issue: https://github.com/bcgov/land-designations-shinyapp/issues/32

ateucher avatar Mar 20 '18 16:03 ateucher

Thanks @ateucher for the feedback, I will check them out.

sasha-ruby avatar Mar 20 '18 19:03 sasha-ruby