Skip to content

Conversation

@bbeckca
Copy link
Contributor

@bbeckca bbeckca commented Sep 22, 2025

Summary:
What: Unskip the CUDA path for test_int8_weight_only_quant in test_torchinductor.py as the kernel was added by #159325.

Why: Confirm CUDA backend for _weight_int8pack_mm is registered.

Test Plan:

buck2 test 'fbcode//mode/opt' fbcode//caffe2/test/inductor:test_inductor_cuda

https://www.internalfb.com/intern/testinfra/testrun/2533275104869494

Differential Revision: D82926440

cc @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @ipiszy @chenyang78 @kadeng @muchulee8 @amjames @chauhang @aakhundov @coconutruben

@pytorch-bot
Copy link

pytorch-bot bot commented Sep 22, 2025

🔗 Helpful Links

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

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

✅ No Failures

As of commit e449d65 with merge base 9d0d98a (image):
💚 Looks good so far! There are no failures yet. 💚

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

@facebook-github-bot
Copy link
Contributor

@bbeckca has exported this pull request. If you are a Meta employee, you can view the originating diff in D82926440.

@thenumberouscode
Copy link
Contributor

@bbeckca It appears that a CUDA implementation of _weight_int8pack_mm exists. Would you be able to provide the associated pull request? Thank you.

@bbeckca
Copy link
Contributor Author

bbeckca commented Sep 22, 2025

@bbeckca It appears that a CUDA implementation of _weight_int8pack_mm exists. Would you be able to provide the associated pull request? Thank you.

Sounds good. Added reference to test plan.

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Sep 23, 2025
@facebook-github-bot
Copy link
Contributor

@bbeckca has exported this pull request. If you are a Meta employee, you can view the originating diff in D82926440.

pytorch-bot bot pushed a commit that referenced this pull request Sep 23, 2025
Summary:

What: Unskip the CUDA path for test_int8_weight_only_quant in test_torchinductor.py.

Why: Confirm CUDA backend for _weight_int8pack_mm is registered.

Test Plan:
```
buck2 test 'fbcode//mode/opt' fbcode//caffe2/test/inductor:test_inductor_cuda
```
https://www.internalfb.com/intern/testinfra/testrun/2533275104869494

Reviewed By: jerryzh168

Differential Revision: D82926440
pytorch-bot bot pushed a commit that referenced this pull request Sep 23, 2025
Summary:

What: Unskip the CUDA path for test_int8_weight_only_quant in test_torchinductor.py.

Why: Confirm CUDA backend for _weight_int8pack_mm is registered.

Test Plan:
```
buck2 test 'fbcode//mode/opt' fbcode//caffe2/test/inductor:test_inductor_cuda
```
https://www.internalfb.com/intern/testinfra/testrun/2533275104869494

Reviewed By: jerryzh168

Differential Revision: D82926440
@facebook-github-bot
Copy link
Contributor

@bbeckca has exported this pull request. If you are a Meta employee, you can view the originating diff in D82926440.

bbeckca added a commit to bbeckca/pytorch that referenced this pull request Sep 23, 2025
…#163461)

Summary:

What: Unskip the CUDA path for test_int8_weight_only_quant in test_torchinductor.py.

Why: Confirm CUDA backend for _weight_int8pack_mm is registered.

Test Plan:
```
buck2 test 'fbcode//mode/opt' fbcode//caffe2/test/inductor:test_inductor_cuda
```
https://www.internalfb.com/intern/testinfra/testrun/2533275104869494

Reviewed By: jerryzh168

Differential Revision: D82926440
@facebook-github-bot
Copy link
Contributor

@bbeckca has exported this pull request. If you are a Meta employee, you can view the originating diff in D82926440.

bbeckca added a commit to bbeckca/pytorch that referenced this pull request Sep 23, 2025
…#163461)

Summary:

What: Unskip the CUDA path for test_int8_weight_only_quant in test_torchinductor.py.

Why: Confirm CUDA backend for _weight_int8pack_mm is registered.

Test Plan:
```
buck2 test 'fbcode//mode/opt' fbcode//caffe2/test/inductor:test_inductor_cuda
```
https://www.internalfb.com/intern/testinfra/testrun/2533275104869494

Reviewed By: jerryzh168

Differential Revision: D82926440
bbeckca added a commit to bbeckca/pytorch that referenced this pull request Sep 23, 2025
…#163461)

Summary:

What: Unskip the CUDA path for test_int8_weight_only_quant in test_torchinductor.py.

Why: Confirm CUDA backend for _weight_int8pack_mm is registered.

Test Plan:
```
buck2 test 'fbcode//mode/opt' fbcode//caffe2/test/inductor:test_inductor_cuda
```
https://www.internalfb.com/intern/testinfra/testrun/2533275104869494

Reviewed By: jerryzh168

Differential Revision: D82926440
@facebook-github-bot
Copy link
Contributor

@bbeckca has exported this pull request. If you are a Meta employee, you can view the originating diff in D82926440.

bbeckca added a commit to bbeckca/pytorch that referenced this pull request Sep 24, 2025
…#163461)

Summary:

What: Unskip the CUDA path for test_int8_weight_only_quant in test_torchinductor.py.

Why: Confirm CUDA backend for _weight_int8pack_mm is registered.

Test Plan:
```
buck2 test 'fbcode//mode/opt' fbcode//caffe2/test/inductor:test_inductor_cuda
```
https://www.internalfb.com/intern/testinfra/testrun/2533275104869494

Reviewed By: jerryzh168

Differential Revision: D82926440
@facebook-github-bot
Copy link
Contributor

@bbeckca has exported this pull request. If you are a Meta employee, you can view the originating diff in D82926440.

…#163461)

Summary:

What: Unskip the CUDA path for test_int8_weight_only_quant in test_torchinductor.py.

Why: Confirm CUDA backend for _weight_int8pack_mm is registered.

Test Plan:
```
buck2 test 'fbcode//mode/opt' fbcode//caffe2/test/inductor:test_inductor_cuda
```
https://www.internalfb.com/intern/testinfra/testrun/2533275104869494

Reviewed By: jerryzh168

Differential Revision: D82926440
@facebook-github-bot
Copy link
Contributor

@bbeckca has exported this pull request. If you are a Meta employee, you can view the originating diff in D82926440.

@facebook-github-bot
Copy link
Contributor

@pytorchbot merge

(Initiating merge automatically since Phabricator Diff has merged)

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

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

jainapurva pushed a commit that referenced this pull request Sep 29, 2025
Summary:
What: Unskip the CUDA path for test_int8_weight_only_quant in test_torchinductor.py as the kernel was added by #159325.

Why: Confirm CUDA backend for _weight_int8pack_mm is registered.

Test Plan:
```
buck2 test 'fbcode//mode/opt' fbcode//caffe2/test/inductor:test_inductor_cuda
```
https://www.internalfb.com/intern/testinfra/testrun/2533275104869494

Differential Revision: D82926440

Pull Request resolved: #163461
Approved by: https://github.com/jerryzh168
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants