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 Dockerfile #31

Merged
merged 2 commits into from Sep 23, 2022
Merged

Add Dockerfile #31

merged 2 commits into from Sep 23, 2022

Conversation

timodonnell
Copy link
Contributor

I created a Dockerfile to run RFdesign (including af2_metrics.py) using CUDA. Getting all the versions of packages right was surprisingly tricky, so I thought I would contribute it back in case you want to add it the repository. I verified that inpainting, hallucination, and af2_metrics.py works on a V100. This does not include pyrosetta however due to the licensing constraints. Thanks for all your work on RFDesign.

Adds a Dockerfile to run RFDesign using CUDA. Also supports alphafold so af2_metrics.py can run.
@jueseph jueseph merged commit a796bcb into RosettaCommons:main Sep 23, 2022
@jueseph
Copy link
Collaborator

jueseph commented Sep 23, 2022

Thanks very much, Tim. I merged this.

@multimeric
Copy link

I wonder if we could make another Dockerfile that does install the licensed components such as pyrosetta, but it does something like mounting the license into the container so that the user still has to provide it?

@roccomoretti
Copy link
Member

@multimeric Licensing for PyRosetta functions at the access stage -- Licensing provides you with access to the installer. There isn't a separate license file which controls access to/authorizes an otherwise installed PyRosetta instance.

@multimeric
Copy link

In that case, perhaps a second Dockerfile could be made that will install everything but requires the Python wheel to be in the same directory?

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.

None yet

4 participants