-
Notifications
You must be signed in to change notification settings - Fork 24.9k
[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
Conversation
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]
🔗 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 FailuresAs of commit f721398 with merge base 86eb65f ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
@pytorchbot merge -f "Everything is green" |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
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
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
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