Skip to content

[MPS][BE] Combine all pre-MacOS14 xfail lists #160228

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

Closed
wants to merge 1 commit into from

Conversation

malfet
Copy link
Contributor

@malfet malfet commented Aug 8, 2025

Stack from ghstack (oldest at bottom):

It does not matter whether it started to fail after 13.1 or 13.3, fact
that it still fails on latest MacOS

It does not matter whether it started to fail after 13.1 or 13.3, fact
that it still fails on latest MacOS

[ghstack-poisoned]
Copy link

pytorch-bot bot commented Aug 8, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/160228

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit f721398 with merge base 86eb65f (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

malfet added a commit that referenced this pull request Aug 8, 2025
It does not matter whether it started to fail after 13.1 or 13.3, fact
that it still fails on latest MacOS

ghstack-source-id: b1c8090
Pull Request resolved: #160228
@malfet malfet added the ciflow/mps Run MPS tests (subset of trunk) label Aug 8, 2025
@malfet malfet added the topic: not user facing topic category label Aug 8, 2025
@malfet
Copy link
Contributor Author

malfet commented Aug 8, 2025

@pytorchbot merge -f "Everything is green"

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use -f as last resort and instead consider -i/--ignore-current to continue the merge ignoring current failures. This will allow currently pending tests to finish and report signal before the merge.

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

pytorchmergebot pushed a commit that referenced this pull request Aug 10, 2025
Right now alpha and betha are both less than zero, which makes them useless for all addmm samples for interal types
Pull Request resolved: #160234
Approved by: https://github.com/Skylion007
ghstack dependencies: #160228
pytorchmergebot pushed a commit that referenced this pull request Aug 11, 2025
By adding `addmm` kernel, which is a logical continuation  of `mm` one. The only tricking part are how alpha and beta constants are handled, which are passed as `optmath_t`, i.e. that it could be, int64, int32 or float

Unified all MM flavors instantiations thru `INSTANTIATE_MM_OPS` and tested that `addmm` metal kernel works as expected for floating types as well by testing it via
```
 PYTORCH_MPS_PREFER_METAL=1 python test/test_mps.py -v -k test_output_match_addmm_mps_
```

Fixes #154901
Pull Request resolved: #160270
Approved by: https://github.com/Skylion007, https://github.com/dcci
ghstack dependencies: #160228, #160234
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/mps Run MPS tests (subset of trunk) Merged topic: not user facing topic category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants