-
Notifications
You must be signed in to change notification settings - Fork 689
Add int32 quant/dequant back #14269
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
Add int32 quant/dequant back #14269
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/14269
Note: Links to docs will display an error until the docs builds have been completed. ❌ 3 New Failures, 1 Unrelated FailureAs of commit 2bac712 with merge base a523306 ( NEW FAILURES - The following jobs have failed:
BROKEN TRUNK - The following job failed but were present on the merge base:👉 Rebase onto the `viable/strict` branch to avoid these failures
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
@mcremon-meta has exported this pull request. If you are a Meta employee, you can view the originating diff in D82282481. |
This PR needs a
|
@mcremon-meta has exported this pull request. If you are a Meta employee, you can view the originating diff in D82282481. |
Summary: Pull Request resolved: #14269 Now that the previous diff exists, we can add the int32 case back without adding to the code size of deployed models. Differential Revision: D82282481
c52612f
to
fd80b51
Compare
@mcremon-meta has exported this pull request. If you are a Meta employee, you can view the originating diff in D82282481. |
Summary: Pull Request resolved: #14269 Now that the previous diff exists, we can add the int32 case back without adding to the code size of deployed models. Reviewed By: hsharma35 Differential Revision: D82282481
fd80b51
to
a9bde68
Compare
@mcremon-meta has exported this pull request. If you are a Meta employee, you can view the originating diff in D82282481. |
Summary: Pull Request resolved: #14269 Now that the previous diff exists, we can add the int32 case back without adding to the code size of deployed models. Reviewed By: hsharma35 Differential Revision: D82282481
a9bde68
to
1997822
Compare
@mcremon-meta has exported this pull request. If you are a Meta employee, you can view the originating diff in D82282481. |
Summary: Pull Request resolved: #14269 Now that the previous diff exists, we can add the int32 case back without adding to the code size of deployed models. Reviewed By: hsharma35 Differential Revision: D82282481
1997822
to
f5cb955
Compare
@mcremon-meta has exported this pull request. If you are a Meta employee, you can view the originating diff in D82282481. |
Summary: Pull Request resolved: #14269 Now that the previous diff exists, we can add the int32 case back without adding to the code size of deployed models. Reviewed By: hsharma35 Differential Revision: D82282481
f5cb955
to
06be77c
Compare
@mcremon-meta has exported this pull request. If you are a Meta employee, you can view the originating diff in D82282481. |
06be77c
to
aa8473e
Compare
Summary: Pull Request resolved: #14269 Now that the previous diff exists, we can add the int32 case back without adding to the code size of deployed models. Reviewed By: hsharma35 Differential Revision: D82282481
@mcremon-meta has exported this pull request. If you are a Meta employee, you can view the originating diff in D82282481. |
Summary: Pull Request resolved: #14269 Now that the previous diff exists, we can add the int32 case back without adding to the code size of deployed models. Reviewed By: hsharma35 Differential Revision: D82282481
aa8473e
to
99f6957
Compare
@mcremon-meta has exported this pull request. If you are a Meta employee, you can view the originating diff in D82282481. |
Summary: Pull Request resolved: #14269 Now that the previous diff exists, we can add the int32 case back without adding to the code size of deployed models. Reviewed By: hsharma35 Differential Revision: D82282481
99f6957
to
bdf0c1e
Compare
@mcremon-meta has exported this pull request. If you are a Meta employee, you can view the originating diff in D82282481. |
Summary: Pull Request resolved: #14269 Now that the previous diff exists, we can add the int32 case back without adding to the code size of deployed models. Reviewed By: hsharma35 Differential Revision: D82282481
bdf0c1e
to
64b8710
Compare
@mcremon-meta has exported this pull request. If you are a Meta employee, you can view the originating diff in D82282481. |
Summary: Pull Request resolved: #14269 Now that the previous diff exists, we can add the int32 case back without adding to the code size of deployed models. Reviewed By: hsharma35 Differential Revision: D82282481
64b8710
to
24cfba2
Compare
@mcremon-meta has exported this pull request. If you are a Meta employee, you can view the originating diff in D82282481. |
Summary: Pull Request resolved: #14269 Now that the previous diff exists, we can add the int32 case back without adding to the code size of deployed models. Reviewed By: hsharma35 Differential Revision: D82282481
24cfba2
to
0abe197
Compare
Summary: Now that the previous diff exists, we can add the int32 case back without adding to the code size of deployed models. Reviewed By: hsharma35 Differential Revision: D82282481
0abe197
to
2bac712
Compare
@mcremon-meta has exported this pull request. If you are a Meta employee, you can view the originating diff in D82282481. |
Differential Revision: D82282481 Pull Request resolved: pytorch#14269
Summary: Now that the previous diff exists, we can add the int32 case back without adding to the code size of deployed models.
Differential Revision: D82282481