Skip to content

Conversation

@per
Copy link
Collaborator

@per per commented Dec 16, 2024

Summary

Move more operators to use the QDQ folding pass

Test plan

Existing unit tests in the Arm backend.

per added 4 commits December 16, 2024 09:46
The node.args that comes from nn.Parameters in the state_dict sometimes
have the requires_grad property set to True. It's seems to stem already
from the export stage and .eval() doesn't change the parameter.

Address it here in the pass for now.

Signed-off-by: Per Åstrand <per.astrand@arm.com>
Change-Id: Ie7425764f1f1865de0fc66e1d020f804c7e936b1
Quantized models might have output without quantization parameters
attached to them. The assert for parameters not being None are removed
and handled in order to allow for that case.
numpy transpose is removed in favor of torch.permute to keep the type
of the output after the operation.

Signed-off-by: Per Åstrand <per.astrand@arm.com>
Change-Id: I0e404062154cefa39f18b5706d72d19cac0e6d73
Signed-off-by: Per Åstrand <per.astrand@arm.com>
Change-Id: I9201d8bafd543204b697c7276d6929ad3aa09f25
Signed-off-by: Per Åstrand <per.astrand@arm.com>
Change-Id: I39d11cff0ef78df08e67f216b8e0bb86af9fac26
@pytorch-bot
Copy link

pytorch-bot bot commented Dec 16, 2024

🔗 Helpful Links

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

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

❌ 1 Cancelled Job

As of commit 8e6570e with merge base 99d5b80 (image):

CANCELLED JOB - The following job was cancelled. Please retry:

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

@facebook-github-bot facebook-github-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 Dec 16, 2024
@per per added partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm ciflow/trunk topic: not user facing labels Dec 16, 2024
@per per requested a review from freddan80 December 16, 2024 13:26
@per
Copy link
Collaborator Author

per commented Dec 16, 2024

Cancelled Check Labels (gh) due to new higher prio requests beeing raised.

@freddan80 freddan80 merged commit 47c2f2e into pytorch:main Dec 16, 2024
107 of 109 checks passed
@per per deleted the convert_operators_to_qdq_folding branch December 16, 2024 16:19
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. partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm topic: not user facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants