Skip to content

Add a16w8 per-op test for var#19596

Merged
christine-long-meta merged 1 commit into
pytorch:mainfrom
christine-long-meta:export-D104532362
May 18, 2026
Merged

Add a16w8 per-op test for var#19596
christine-long-meta merged 1 commit into
pytorch:mainfrom
christine-long-meta:export-D104532362

Conversation

@christine-long-meta
Copy link
Copy Markdown
Contributor

@christine-long-meta christine-long-meta commented May 14, 2026

Summary:
Add int16 activation / int8 weight (a16w8) quantization tests for aten.var on Ethos-U55 and Ethos-U85.

Changes

  • Add test_var_a16w8_u55_INT and test_var_a16w8_u85_INT using EthosU55PipelineINT/EthosU85PipelineINT with a16w8_quantization=True, symmetric_io_quantization=True
  • Split Var.test_parameters into passing (a16w8_var_test_parameters, keepdim=True) and xfail (a16w8_var_test_parameters_xfails, keepdim=False) groups for the a16w8 tests
  • Mark keepdim=False cases (var_3d_no_keep_dim_0_correction, var_4d_no_keep_dim_0_5_correction) as pytest.mark.xfail since var a16w8 produces incorrect output for scalar/reduced-rank output
  • Register ops/test_var.py in fbcode/ and xplat/ targets.bzl

Differential Revision: D104532362

@pytorch-bot
Copy link
Copy Markdown

pytorch-bot Bot commented May 14, 2026

🔗 Helpful Links

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

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

❗ 2 Active SEVs

There are 2 currently active SEVs. If your PR is affected, please view them below:

❌ 2 New Failures, 1 Unrelated Failure

As of commit c7709e1 with merge base 824cbff (image):

NEW FAILURES - The following jobs have failed:

FLAKY - The following job failed but was likely due to flakiness present on trunk:

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

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 14, 2026
@github-actions github-actions Bot added ciflow/trunk module: arm Issues related to arm backend labels May 14, 2026
@pytorch-bot
Copy link
Copy Markdown

pytorch-bot Bot commented May 14, 2026

Workflows were awaiting approval. CI has now been triggered for the ciflow labels on this PR.

@github-actions
Copy link
Copy Markdown

This PR needs a release notes: label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

@christine-long-meta christine-long-meta force-pushed the export-D104532362 branch 2 times, most recently from 8989a2c to 6b40fe6 Compare May 14, 2026 16:47
@meta-codesync meta-codesync Bot changed the title Add a16w8 per-op test for var Add a16w8 per-op test for var (#19596) May 14, 2026
@christine-long-meta christine-long-meta force-pushed the export-D104532362 branch 4 times, most recently from d004872 to 578a5dd Compare May 14, 2026 16:49
@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync Bot commented May 14, 2026

@christine-long-meta has exported this pull request. If you are a Meta employee, you can view the originating Diff in D104532362.

@christine-long-meta christine-long-meta force-pushed the export-D104532362 branch 3 times, most recently from f1d1cf7 to 7aa22ac Compare May 14, 2026 16:53
christine-long-meta added a commit to christine-long-meta/executorch that referenced this pull request May 14, 2026
Summary:
Pull Request resolved: pytorch#19596

Add int16 activation / int8 weight (a16w8) quantization tests for `aten.var` on Ethos-U55 and Ethos-U85.

## Changes
- Add `a16w8_var_test_parameters` dict with 4 test configurations covering keepdim/no-keepdim and correction values 0, 0.5, and 1
- Add `test_var_a16w8_u55_INT` using `EthosU55PipelineINT` with `a16w8_quantization=True, symmetric_io_quantization=True, qtol=128, epsilon=2**-16`
- Add `test_var_a16w8_u85_INT` using `EthosU85PipelineINT` with same kwargs
- Register `ops/test_var.py` in `fbcode/` and `xplat/` `targets.bzl`

bypass-pytorch-oss-checks

Differential Revision: D104532362
@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync Bot commented May 14, 2026

@christine-long-meta has exported this pull request. If you are a Meta employee, you can view the originating Diff in D104532362.

3 similar comments
@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync Bot commented May 14, 2026

@christine-long-meta has exported this pull request. If you are a Meta employee, you can view the originating Diff in D104532362.

@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync Bot commented May 14, 2026

@christine-long-meta has exported this pull request. If you are a Meta employee, you can view the originating Diff in D104532362.

@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync Bot commented May 14, 2026

@christine-long-meta has exported this pull request. If you are a Meta employee, you can view the originating Diff in D104532362.

@meta-codesync meta-codesync Bot changed the title Add a16w8 per-op test for var (#19596) Add a16w8 per-op test for var May 14, 2026
christine-long-meta added a commit to christine-long-meta/executorch that referenced this pull request May 14, 2026
Summary:
Pull Request resolved: pytorch#19596

Add int16 activation / int8 weight (a16w8) quantization tests for `aten.var` on Ethos-U55 and Ethos-U85.

## Changes
- Add `a16w8_var_test_parameters` dict with 4 test configurations covering keepdim/no-keepdim and correction values 0, 0.5, and 1
- Add `test_var_a16w8_u55_INT` using `EthosU55PipelineINT` with `a16w8_quantization=True, symmetric_io_quantization=True, qtol=128, epsilon=2**-16`
- Add `test_var_a16w8_u85_INT` using `EthosU85PipelineINT` with same kwargs
- Register `ops/test_var.py` in `fbcode/` and `xplat/` `targets.bzl`

bypass-pytorch-oss-checks

Differential Revision: D104532362
@meta-codesync meta-codesync Bot changed the title Add a16w8 per-op test for var Add a16w8 per-op test for var (#19596) May 14, 2026
@meta-codesync meta-codesync Bot changed the title Add a16w8 per-op test for var (#19596) Add a16w8 per-op test for var May 15, 2026
christine-long-meta added a commit to christine-long-meta/executorch that referenced this pull request May 15, 2026
Summary:
Pull Request resolved: pytorch#19596

Add int16 activation / int8 weight (a16w8) quantization tests for `aten.var` on Ethos-U55 and Ethos-U85.

## Changes
- Add `test_parameters_ethosu` class attribute to `Var` with 2 test configurations (4D tensors with correction=0 and correction=1)
- Switch existing `test_var_dim_u55_INT_no_dim` and `test_var_dim_u85_INT_no_dim` from `Var.test_parameters` to `Var.test_parameters_ethosu` for Ethos-U compatible tensor shapes
- Add `test_var_a16w8_u55_INT` using `EthosU55PipelineINT` with `a16w8_quantization=True, symmetric_io_quantization=True`
- Add `test_var_a16w8_u85_INT` using `EthosU85PipelineINT` with same kwargs
- Register `ops/test_var.py` in `fbcode/` and `xplat/` `targets.bzl`

Differential Revision: D104532362
@meta-codesync meta-codesync Bot changed the title Add a16w8 per-op test for var Add a16w8 per-op test for var (#19596) May 15, 2026
@meta-codesync meta-codesync Bot changed the title Add a16w8 per-op test for var (#19596) Add a16w8 per-op test for var May 16, 2026
@meta-codesync meta-codesync Bot changed the title Add a16w8 per-op test for var Add a16w8 per-op test for var (#19596) May 16, 2026
christine-long-meta added a commit to christine-long-meta/executorch that referenced this pull request May 16, 2026
Summary:

Add int16 activation / int8 weight (a16w8) quantization tests for `aten.var` on Ethos-U55 and Ethos-U85.

## Changes
- Add `test_parameters_ethosu` class attribute to `Var` with 2 test configurations (4D tensors with correction=0 and correction=1)
- Switch existing `test_var_dim_u55_INT_no_dim` and `test_var_dim_u85_INT_no_dim` from `Var.test_parameters` to `Var.test_parameters_ethosu` for Ethos-U compatible tensor shapes
- Add `test_var_a16w8_u55_INT` using `EthosU55PipelineINT` with `a16w8_quantization=True, symmetric_io_quantization=True`
- Add `test_var_a16w8_u85_INT` using `EthosU85PipelineINT` with same kwargs
- Register `ops/test_var.py` in `fbcode/` and `xplat/` `targets.bzl`

Differential Revision: D104532362
christine-long-meta added a commit to christine-long-meta/executorch that referenced this pull request May 16, 2026
Summary:

Add int16 activation / int8 weight (a16w8) quantization tests for `aten.var` on Ethos-U55 and Ethos-U85.

## Changes
- Add `test_parameters_ethosu` class attribute to `Var` with 2 test configurations (4D tensors with correction=0 and correction=1)
- Switch existing `test_var_dim_u55_INT_no_dim` and `test_var_dim_u85_INT_no_dim` from `Var.test_parameters` to `Var.test_parameters_ethosu` for Ethos-U compatible tensor shapes
- Add `test_var_a16w8_u55_INT` using `EthosU55PipelineINT` with `a16w8_quantization=True, symmetric_io_quantization=True`
- Add `test_var_a16w8_u85_INT` using `EthosU85PipelineINT` with same kwargs
- Register `ops/test_var.py` in `fbcode/` and `xplat/` `targets.bzl`

Differential Revision: D104532362
christine-long-meta added a commit to christine-long-meta/executorch that referenced this pull request May 16, 2026
Summary:

Add int16 activation / int8 weight (a16w8) quantization tests for `aten.var` on Ethos-U55 and Ethos-U85.

## Changes
- Add `test_parameters_ethosu` class attribute to `Var` with 2 test configurations (4D tensors with correction=0 and correction=1)
- Switch existing `test_var_dim_u55_INT_no_dim` and `test_var_dim_u85_INT_no_dim` from `Var.test_parameters` to `Var.test_parameters_ethosu` for Ethos-U compatible tensor shapes
- Add `test_var_a16w8_u55_INT` using `EthosU55PipelineINT` with `a16w8_quantization=True, symmetric_io_quantization=True`
- Add `test_var_a16w8_u85_INT` using `EthosU85PipelineINT` with same kwargs
- Register `ops/test_var.py` in `fbcode/` and `xplat/` `targets.bzl`

Differential Revision: D104532362
@christine-long-meta christine-long-meta force-pushed the export-D104532362 branch 2 times, most recently from 8805beb to 3879789 Compare May 16, 2026 02:08
@meta-codesync meta-codesync Bot changed the title Add a16w8 per-op test for var (#19596) Add a16w8 per-op test for var May 17, 2026
@christine-long-meta christine-long-meta force-pushed the export-D104532362 branch 2 times, most recently from 74f3654 to 0c1b2b7 Compare May 17, 2026 16:06
@meta-codesync meta-codesync Bot changed the title Add a16w8 per-op test for var Add a16w8 per-op test for var (#19596) May 17, 2026
Summary:
Add int16 activation / int8 weight (a16w8) quantization tests for `aten.var` on Ethos-U55 and Ethos-U85.

## Changes
- Add `test_var_a16w8_u55_INT` and `test_var_a16w8_u85_INT` using `EthosU55PipelineINT`/`EthosU85PipelineINT` with `a16w8_quantization=True, symmetric_io_quantization=True`
- Split `Var.test_parameters` into passing (`a16w8_var_test_parameters`, keepdim=True) and xfail (`a16w8_var_test_parameters_xfails`, keepdim=False) groups for the a16w8 tests
- Mark `keepdim=False` cases (`var_3d_no_keep_dim_0_correction`, `var_4d_no_keep_dim_0_5_correction`) as `pytest.mark.xfail` since var a16w8 produces incorrect output for scalar/reduced-rank output
- Register `ops/test_var.py` in `fbcode/` and `xplat/` `targets.bzl`

Differential Revision: D104532362
@meta-codesync meta-codesync Bot changed the title Add a16w8 per-op test for var (#19596) Add a16w8 per-op test for var May 17, 2026
@zingo
Copy link
Copy Markdown
Collaborator

zingo commented May 18, 2026

You seem to got a strange error in test-arm-ootb-linux, It seems very unrelated, I rerun the tests maybe it was a network glitch and will go away.

Copy link
Copy Markdown
Collaborator

@zingo zingo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The test re-run passed, approving :) Thanks!

@christine-long-meta christine-long-meta merged commit a142873 into pytorch:main May 18, 2026
591 of 600 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported meta-exported module: arm Issues related to arm backend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants