Skip to content
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

Add an "R only" layer for mixing R with other runtimes #59

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kgutwin
Copy link

@kgutwin kgutwin commented May 29, 2020

My use case is primarily based on another language runtime (python) but would like to be able to also call R scripts. The build framework here only needs slight tweaks to deploy an "R only" layer that is suitable for combining with all other runtimes.

This PR currently only contains the changes to the build scripts to build and deploy the "R only" layer. Updates to the documentation and test framework can follow later if desired.

Also note that replacing the chown call with docker run --user $(id -u) should resolve issue #52.

Thanks!

@kgutwin
Copy link
Author

kgutwin commented May 29, 2020

I'm not sure why this test is failing, I doubt it has anything to do with my changes.

self.assertEqual(1, len(result))

When I ran python3 -m unittest on my system, it failed here as well, although that was due to bad AWS credentials. When I fixed my AWS credentials, this test passed. Is this test dependent on AWS API access somehow?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant