Skip to content

[GPU] Change eltwise quantized data behavior from saturation to overflow#33769

Open
p-wysocki wants to merge 3 commits intoopenvinotoolkit:masterfrom
p-wysocki:overflow_int
Open

[GPU] Change eltwise quantized data behavior from saturation to overflow#33769
p-wysocki wants to merge 3 commits intoopenvinotoolkit:masterfrom
p-wysocki:overflow_int

Conversation

@p-wysocki
Copy link
Contributor

Details:

  • See task description
  • Reproducer script now returns correct results
  • Added test

Tickets:

  • 179285

@michal-miotk
Copy link
Contributor

please extend range of this PR to all kernels which use saturation

@p-wysocki
Copy link
Contributor Author

@michal-miotk I extended the PR to eltwise_blocked_opt.cl and added a new test. The only other places using saturation I see are quantization (where I assume it's expected) and reorders (ex. float -> int). If there's more please let me know, I haven't contributed to GPU much yet.

@p-durandin p-durandin added this to the 2026.1 milestone Jan 28, 2026
@@ -6,7 +6,6 @@

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

random spot) this change itself looks good. but I think it is necessary to check potential side-effect. Could you trigger onepunch accuracy test?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: GPU OpenVINO GPU plugin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants