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

graphcast hub #977

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

graphcast hub #977

wants to merge 8 commits into from

Conversation

julienchastang
Copy link
Member

@julienchastang julienchastang force-pushed the tm24f branch 4 times, most recently from a95faef to b35747e Compare November 15, 2024 17:41
@@ -1,7 +1,7 @@
# Heavily borrowed from docker-stacks/minimal-notebook/
# https://github.com/jupyter/docker-stacks/blob/main/minimal-notebook/Dockerfile

ARG BASE_CONTAINER=jupyter/minimal-notebook
ARG BASE_CONTAINER=jupyter/tensorflow-notebook
Copy link
Contributor

Choose a reason for hiding this comment

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

Why the switch from minimal to the tf image?

Copy link
Member Author

Choose a reason for hiding this comment

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

It is the only way I can get the GPU to work.

Copy link
Contributor

Choose a reason for hiding this comment

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

What are you trying? The image that's live right now uses the minimal-notebook and thomas and I can access the GPU just fine.

Copy link
Member Author

Choose a reason for hiding this comment

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

How did you get that to work beyond the usual, i.e., the newly defined environment.yml and jupyterhub_gpu.yaml. Did you have to install anything special CUDA/GPU-wise?

Copy link
Member Author

Choose a reason for hiding this comment

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

My understanding is that the current running environment was modified "in-place" for faster experimentation so not everything may have been captured in the Dockerfile, environment.yml, etc. I guess that is what I am asking.

Copy link
Contributor

Choose a reason for hiding this comment

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

You can find the most recent Dockerfile/environment combo in the "ana" tmux session on the docker-gpu machine. Your most recent commit has the correct environment file, however using the minimal-notebook as the base still works.

To get the GPU to work there is nothing more needed past the "normal" things--i.e. requesting the additional resource via jupyterhub_gpu.yaml.

I've found that the important parts of getting these things to work are making sure the packages you install (via conda or pip) are compatible with that shown when doing an nvidia-smi on JS2.

For example, in this case I tell pip to look for torch and associated packages from the cuda 12.1 index with this line:
- --extra-index-url https://download.pytorch.org/whl/cu121

We can get together to discuss this in-person after the holidays?

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.

2 participants