Skip to content

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]
@pytorch-bot
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
@github-actions github-actions bot deleted the gh/malfet/477/head branch September 8, 2025 02:15
markc-614 pushed a commit to markc-614/pytorch that referenced this pull request Sep 17, 2025
It does not matter whether it started to fail after 13.1 or 13.3, fact
that it still fails on latest MacOS
Pull Request resolved: pytorch#160228
Approved by: https://github.com/dcci
markc-614 pushed a commit to markc-614/pytorch that referenced this pull request Sep 17, 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: pytorch#160234
Approved by: https://github.com/Skylion007
ghstack dependencies: pytorch#160228
markc-614 pushed a commit to markc-614/pytorch that referenced this pull request Sep 17, 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 pytorch#154901
Pull Request resolved: pytorch#160270
Approved by: https://github.com/Skylion007, https://github.com/dcci
ghstack dependencies: pytorch#160228, pytorch#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