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

remove cuda-nvcc and document ptxas #398

Merged
merged 6 commits into from
Oct 18, 2022
Merged

Conversation

ngam
Copy link
Contributor

@ngam ngam commented Oct 18, 2022

@pangeo-bot
Copy link
Collaborator

/condalock
Automatically locking new conda environment, building, and testing images...

@github-actions
Copy link
Contributor

Binder 👈 Try on Mybinder.org!
Binder 👈 Try on Pangeo GCP Binder!
Binder 👈 Try on Pangeo AWS Binder!

@ngam ngam changed the title remove cuda-nvcc remove cuda-nvcc and document ptxas Oct 18, 2022
@ngam ngam marked this pull request as ready for review October 18, 2022 15:59
@dhruvbalwada
Copy link
Member

dhruvbalwada commented Oct 18, 2022

Should it be removed (user can install if needed) or just unpinned (user downgrades if needed)? Just wondering if unpinning is better, as some other things might be using it?

@ngam
Copy link
Contributor Author

ngam commented Oct 18, 2022

Should it be removed (user can install if needed) or just unpinned (user downgrades if needed)? Just wondering if unpinning is better, as some other things might be using it?

Nothing else is using it except JAX and if someone is forcing XLA stuff in TensorFlow. In either case, I think a removal is warranted. It's not a good idea to mix channels anyway, and it is obviously causing a lot of confusion. So it is better left to the user to make their own calls while keeping these overarching images as simple as possible.

README.md Outdated Show resolved Hide resolved
@yuvipanda
Copy link
Member

From a cluster perspective, I'm happy to keep drivers up to date with whatever it is the cloud providers support (which is out of our hands). If we feel this isn't going to affect non-jax users, and they have a well documented workaround, I think this seems fine to me.

Based on all this conversation, 2i2c now uses T4s as default GPUs for everyone 2i2c-org/infrastructure#1787. Thanks everyone!

@dhruvbalwada dhruvbalwada mentioned this pull request Oct 18, 2022
@scottyhq scottyhq merged commit 5f3ee72 into pangeo-data:master Oct 18, 2022
@scottyhq scottyhq mentioned this pull request Feb 3, 2023
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.

Document ML-image tag/GPU type/CUDA compatibility table
6 participants