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

Added mps #337

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Added mps #337

wants to merge 2 commits into from

Conversation

CGCooke
Copy link

@CGCooke CGCooke commented Mar 23, 2023

  • Added MPS as a device option.
  • Potential for improved performance on devices that support MPS. (Mac M1/M2)

1. Added [MPS](https://developer.apple.com/documentation/metalperformanceshaders) as a device option, to Interacting_with_CLIP.ipynb.
2. Results in a 30% increase in execution speed for the notebook on an Apple M2 Mac Mini.
1. Added MPS as a device to Prompt_Engineering_for_ImageNet.ipynb
@kulinseth
Copy link

Thanks @CGCooke for enabling the "mps" backend option.

@kevflynn
Copy link

kevflynn commented Jul 3, 2023

Is this dead? I'm running into so many mps problems with so many different models - all running on pytorch. But here with CLIP, detectron2 - all seem to be old/stale issues. Trying to understand expectations - is apple not super committed to silicon being a viable option for using widely available open-sourced ML tooling/models?

@kulinseth
Copy link

Is this dead? I'm running into so many mps problems with so many different models - all running on pytorch. But here with CLIP, detectron2 - all seem to be old/stale issues. Trying to understand expectations - is apple not super committed to silicon being a viable option for using widely available open-sourced ML tooling/models?

@kevflynn we are committed to provide mps backend support on Mac platforms. The issue which is linked has been long solved on PyTorch nightlies.

can you please provide all the issues which you are running into and we will priortize it

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.

3 participants