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

ENH: include nvidia-smi across lectures #60

Merged
merged 3 commits into from
May 16, 2023
Merged

ENH: include nvidia-smi across lectures #60

merged 3 commits into from
May 16, 2023

Conversation

mmcky
Copy link
Contributor

@mmcky mmcky commented May 16, 2023

fixes #51

@netlify
Copy link

netlify bot commented May 16, 2023

Deploy Preview for incomparable-parfait-2417f8 ready!

Name Link
🔨 Latest commit 0a95930
🔍 Latest deploy log https://app.netlify.com/sites/incomparable-parfait-2417f8/deploys/64631b8a2d282e000700805e
😎 Deploy Preview https://deploy-preview-60--incomparable-parfait-2417f8.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions
Copy link

github-actions bot commented May 16, 2023

@github-actions github-actions bot temporarily deployed to pull request May 16, 2023 00:48 Inactive
@mmcky
Copy link
Contributor Author

mmcky commented May 16, 2023

The linkchecker is failing due to the anchor issue #53 so this looks ready.

@mmcky mmcky requested a review from jstac May 16, 2023 01:59
@mmcky
Copy link
Contributor Author

mmcky commented May 16, 2023

@jstac the only lecture I didn't add nvidia-smi to is

https://6462d2eada9c00149ee7641a--incomparable-parfait-2417f8.netlify.app/jax_intro.html

thinking it simplifies the lecture. But we can add it there as well if you like.

@mmcky mmcky added the ready label May 16, 2023
@chappiewuzefan
Copy link
Collaborator

Hi @mmcky Matt , I think the left broken link was fixed by @Smit-create in 53e0674

@@ -57,6 +57,12 @@ import jax
import jax.numpy as jnp
```

We can also check the `gpu` hardware we are running on using `nvidia-smi`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks @mmcky

How about "Let's check the GPU we are running"

I'm very keen on cutting words ATM. Either way gpu should be GPU.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jstac done.

@github-actions github-actions bot temporarily deployed to pull request May 16, 2023 06:23 Inactive
@jstac
Copy link
Contributor

jstac commented May 16, 2023

Thanks @mmcky !

@jstac jstac merged commit d501a58 into main May 16, 2023
@jstac jstac deleted the add-nvidia-smi branch May 16, 2023 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[hardware] Include nvidia-smi output in lectures
3 participants