-
Notifications
You must be signed in to change notification settings - Fork 1
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
Dockerfile.RBuildEnv issues with dependencies #33
Comments
But if I install the dependencies from the terminal in RStudio, all is good: terminal:
console:
|
The most up-to-date build environment is in the connect-to-real-data branch: https://github.com/seattleflu/incidence-mapper/blob/connect-to-real-data/Dockerfile.RBuildEnv |
I believe I've fixed some of these package issues on my PR #35. Someone should check that it also works for them. |
@devclinton I rebuilt the image on our machine from trs/simplify-db-connection. Seems mostly good (and better than before). One issue is this line doesn't work incidence-mapper/Dockerfile.RBuildEnv Line 46 in d090214
|
(I didn't notice that failing for me, but it likely did!) |
@devclinton I'll return this to your list, as I'm a little confused about copying in something that is at the same level as the dockerfile. Thanks! incidence-mapper/Dockerfile.RBuildEnv Line 56 in 9d31ed8
|
@tsibley For now I removed the running of install_local_packages.R on the building on the container. This is more because of how the future workflow will actually execute. At the moment, the container is being used as a development environment but later the workflow will be more along the lines you provided where we need to build each package. For now, let's make that a script the user can execute from within the R environment after they start the environment. Later, I envision a workflow like the following
|
geojsonio isn't properly installing in the docker image. Multiple packages dependency libraries aren't coming through, despite having added them to the build environment file.
Called interactively:
The text was updated successfully, but these errors were encountered: