Skip to content

Fix quantized Pad op dimension check when QDQ optimizer is disabled#582

Merged
saurabhkale17 merged 1 commit intoovep-developfrom
jatin_raise_pr_for_surender
Feb 18, 2025
Merged

Fix quantized Pad op dimension check when QDQ optimizer is disabled#582
saurabhkale17 merged 1 commit intoovep-developfrom
jatin_raise_pr_for_surender

Conversation

@jatinwadhwa921
Copy link

This change addresses an issue where a Pad op in quantized models fails due to an unsupported dimension input. The fix adds logic to detect if a Pad op is part of a quantized model by checking for a DequantizeLinear input. If found, the op is marked as quantized and the unsupported dimension check is bypassed, ensuring that the pad_value remains constant as required by the VPUX compiler.

Related-to: EISW-152222

This change addresses an issue where a Pad op in quantized models fails due to an unsupported dimension input. The fix adds logic to detect if a Pad op is part of a quantized model by checking for a DequantizeLinear input. If found, the op is marked as quantized and the unsupported dimension check is bypassed, ensuring that the pad_value remains constant as required by the VPUX compiler.

Related-to: EISW-152222
@saurabhkale17
Copy link

LGTM

@saurabhkale17 saurabhkale17 merged commit 4ca7075 into ovep-develop Feb 18, 2025
5 of 12 checks passed
jatinwadhwa921 added a commit that referenced this pull request Feb 18, 2025
…582)

This change addresses an issue where a Pad op in quantized models fails due to an unsupported dimension input. The fix adds logic to detect if a Pad op is part of a quantized model by checking for a DequantizeLinear input. If found, the op is marked as quantized and the unsupported dimension check is bypassed, ensuring that the pad_value remains constant as required by the VPUX compiler.

Related-to: EISW-152222

Co-authored-by: Surendar Rama Sitaraman <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants