Arm backend: Added real calibration for NFRU model - #23182
Conversation
This patch contains the following changes: - Added real calibration for NFRU model - Sped up dataset generation for NSS & NFRU models - Reworked NFRU QAT testing to use use real calibration data - Parameterised QAT testing Signed-off-by: Michiel Olieslagers <michiel.olieslagers@arm.com> Change-Id: Ifd9d1b41bbba52b200be4eba079363d12c7fb2af
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/23182
Note: Links to docs will display an error until the docs builds have been completed. ❗ 1 Active SEVsThere are 1 currently active SEVs. If your PR is affected, please view them below: ❌ 5 New Failures, 8 Pending, 1 Unrelated FailureAs of commit 4e4799c with merge base f3626e6 ( 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. |
|
@Michiel-Olieslagers this is breaking CI (e.g., the pull / test-arm-backend-no-driver (test_pytest_models_tosa) / linux-job). Are you working on a forward fix? An agent says the following: The failing job ran the PR’s merge commit, 22b43b2, and reports: |
#23265 , potential fwd fix ? |
|
@metascroy and @psiddh I don't think anyone is working until tomorrow, I'm back from vacation tomorrow also (CET timeframe) and will see if there is anything I can do and/or check progress of the mentioend fix in one of the other PRs that @Michiel-Olieslagers mentioned. |
…23265) #23182 made the NSS and NFRU model tests call pipeline.set_quantization_calibration(), but never added the method, so the real-data TOSA INT tests fail with AttributeError. Add it to BasePipeline, configuring the quantize stage the same way the NSS test did directly before #23182. Authored with Claude Code. cc @digantdesai @freddan80 @per @zingo @oscarandersson8218 @mansnils @Sebastian-Larsson @robell @rascani
This patch contains the following changes:
Change-Id: Ifd9d1b41bbba52b200be4eba079363d12c7fb2af
cc @digantdesai @freddan80 @per @zingo @oscarandersson8218 @mansnils @Sebastian-Larsson @robell @rascani