Skip to content

Conversation

@lucymcnatt
Copy link
Contributor

@lucymcnatt lucymcnatt commented Dec 1, 2025

Adds a new base image terra-base with minimal pre-installed packages. It is GPU-enabled and allows sudo access on the docker container for users.

Paired with this Leo PR: DataBiosphere/leonardo#4888

@lucymcnatt lucymcnatt requested a review from LizBaldo December 1, 2025 16:29
Copy link
Collaborator

@LizBaldo LizBaldo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The dockerfile is really clear, and the use of UV is so great! Thank you so much for pushing through this effort :) I have a few questions / comments, but should be good to merge whenever you are ready for it.

@@ -0,0 +1,29 @@
[project]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So beautiful!

# Latest gpu-enabled base image on Ubuntu 22, 132 MB compressed
FROM --platform=linux/amd64 nvidia/cuda:13.0.1-base-ubuntu22.04

LABEL maintainer="DSP Analysis Team <dsp-analysis@broadinstitute.org>"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice-to-have: probably copy-pasted but would be nice to update

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah I considered changing this, but we don't have an equivalent analysis team email at manifold so I thought I'd just leave this for now

##############################
# Setup UV Environment for Jupyter
##############################
# Using UV (Universal Virtualenv) to create a virtual environment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TIL

@lucymcnatt lucymcnatt merged commit bf5a7d5 into master Jan 7, 2026
4 checks passed
@lucymcnatt lucymcnatt deleted the CTM-142-terra-base branch January 7, 2026 16:29
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.

4 participants