Skip to content

Conversation

mcr229
Copy link
Contributor

@mcr229 mcr229 commented Aug 25, 2023

Summary:
For Quantized Mobilenetv3.

Right now decomposed hardswish and hardsigmoid gets partitioned with XNNPACKQuantizedPartitioner2, because they contain simple ops like add, mul, div, etc.

These ops are all in FP32 since xnnpack doesn't support the quantized variants. Since we don't currently have support for a unified partition, we want to block the from our Quantized Partitioner.

Reviewed By: digantdesai

Differential Revision: D48667680

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Aug 25, 2023
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48667680

1 similar comment
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48667680

mcr229 and others added 3 commits August 25, 2023 16:56
Differential Revision: D48667679

fbshipit-source-id: ca01aff1fdc7888e0abcb5a5437b0b6f2231dbd8
Differential Revision: D48667676

fbshipit-source-id: 19dc3ff786f6a57dd68102bf69acf2cb0e68363e
Summary:
Pull Request resolved: pytorch/executorch#139

For Quantized Mobilenetv3.

Right now decomposed hardswish and hardsigmoid gets partitioned with XNNPACKQuantizedPartitioner2, because they contain simple ops like add, mul, div, etc.

These ops are all in FP32 since xnnpack doesn't support the quantized variants. Since we don't currently have support for a unified partition, we want to block the from our Quantized Partitioner.

Reviewed By: digantdesai

Differential Revision: D48667680

fbshipit-source-id: d055d4b7d5d0e0f0d3633635c16cfca7e6c5f48a
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48667680

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 41125f9.

Gasoonjia pushed a commit that referenced this pull request Jul 30, 2024
jirioc pushed a commit to nxp-upstream/executorch that referenced this pull request May 14, 2025
…erter

Merge in AITEC/executorch from feature/nxg10272/EIEX-292-add-support-for-aten.sigmoid-operator to main-nxp

* commit 'a31f739ed1c851d6680c1b89a4b92c087e738d6f':
  Add implementation of Sigmoid operator converter
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants