Compatibility with MPS backend
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 45/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Stale
- Domain
- backend, operating-systems, testing
Research direction
Start with prep_pos_from_pad_and_prev_lens in neuron_explainer/models/transformer.py and get_default_device in neuron_explainer/models/model_context.py. Search the repository for other cumsum uses and the existing workaround that skips MPS tests. Verify the device-selection behavior on macOS versions before and after 13.3, then run the affected MPS tests to confirm the compatibility change.
Written by the indexing model from the issue text.
Description
While running inference on my Mac with MacOS version 13.1, I received the following error:
RuntimeError: MPS does not support cumsum_out_mps op with int64 input. Support has been added in macOS 13.3
I received this error because of the use of cumsum in prep_pos_from_pad_and_prev_lens. cumsum is also used in other places in the repository.
The same error arises when running various tests on the MPS backend, as mentioned around the function get_default_device. I have checked that this error is also because of the inability of MacOS to compute cumsum with version < 13.3.
Should we modify the function get_default_device to return torch.device("mps", 0) only when MacOS version >= 13.3? We can remove the current workaround that avoids running pytests with this backend.
If this seems like a useful change, I will be happy to submit a pull request.
Thank you!
- Dominant language
- Python
- Stars
- 4.1k
- Forks
- 242
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from openai/transformer-debugger
-
Difficulty 3/5 1-2 days Newbie friendliness 25/100
-
Difficulty 3/5 1-2 days Newbie friendliness 35/100
-
Difficulty 3/5 1-2 days Newbie friendliness 25/100
openai/transformer-debugger#21 · 5 comments · 1 reaction ·
-
Difficulty 3/5 1-2 days Newbie friendliness 35/100
-
Difficulty 4/5 3-5 days Newbie friendliness 20/100
openai/transformer-debugger#19 · 1 comment ·
All issues in openai/transformer-debugger
Similar issues
-
Difficulty 1/5 Under an hour Newbie friendliness 80/100
521xueweihan/HelloGitHub#4063 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
Vanilla-OS/apx-gui#112 ·
-
Devstral and Voxtral: section 5 fails in us-east-1 with 400 "Model does not support audio modality"Open
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
aws-samples/sample-per-model-bedrock#409 ·
Maintainers usually reply within 1 day
-
DuckDB 2.0 breaks to_pyarrow_batchesPossibly taken @DG47 claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
ibis-project/ibis#12196 ·
Maintainers usually reply within 1 day