Skip to content

[LPT] Skip PadTransformation for non-finite CONSTANT pad values - #37411

Open
pj3iL wants to merge 1 commit into
openvinotoolkit:masterfrom
pj3iL:pad-inf
Open

[LPT] Skip PadTransformation for non-finite CONSTANT pad values#37411
pj3iL wants to merge 1 commit into
openvinotoolkit:masterfrom
pj3iL:pad-inf

Conversation

@pj3iL

@pj3iL pj3iL commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Guard canBeTransformed() with checkConstantNotInf when pad mode is CONSTANT, preventing an invalid quantized constant from being created during the dequantization-move rewrite. Replace the magic input index 3 with a named constant for readability. Add a regression test covering ±inf on a unique pad dimension.

Details:

  • src/common/low_precision_transformations/src/pad.cpp
  • src/common/low_precision_transformations/tests/pad_transformation.cpp

Tickets:

AI Assistance:

  • AI assistance used: yes
  • Used to understand how int8 transformation works, creating the changes quickly for ±inf, followed by tests. Manual checks are done for correctness and validation.

@pj3iL
pj3iL requested a review from a team as a code owner August 13, 2026 05:33
@github-actions github-actions Bot added the category: LP transformations OpenVINO Low Precision transformations label Aug 13, 2026
@sys-openvino-ci sys-openvino-ci added the ExternalPR External contributor label Aug 13, 2026
@pj3iL
pj3iL marked this pull request as draft August 13, 2026 05:51
@pj3iL
pj3iL marked this pull request as ready for review August 13, 2026 07:24
Guard canBeTransformed() with checkConstantNotInf when pad mode is
CONSTANT, preventing an invalid quantized constant from being created
during the dequantization-move rewrite. Replace the magic input index
3 with a named constant for readability. Add a regression test covering
±inf on a unique pad dimension.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: LP transformations OpenVINO Low Precision transformations ExternalPR External contributor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants