Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[GPU] Fix hybrid quantization reg issue #27404

Conversation

hyunback
Copy link
Contributor

@hyunback hyunback commented Nov 5, 2024

Many daily int8 models have Perf regression.(incorrect conv data type and bias)
Fix kernel selection issue.

Tickets:

  • 156564, 155267, 156243

vladimir-paramuzov and others added 2 commits November 5, 2024 19:47
### Details:
- set LPT callbacks to handle compression and avoid constant folding for
it (taken from openvinotoolkit#20973)
 - Allow u8/i8 output data type for compressed onednn FC
- Disable Dequantize propagation through Transpose if it's a dependency
of SDPA to keep Transpose+SDPA fusion
Many daily int8 models have Perf regression.(incorrect conv data type and bias)
Fix kernel selection issue.

Signed-off-by: hyunback <[email protected]>
@hyunback hyunback added the category: GPU OpenVINO GPU plugin label Nov 5, 2024
@hyunback hyunback requested review from a team as code owners November 5, 2024 10:53
Convolution is expected int8 data type in int8 model, but with mixed weight compressed occurs, it run fp16.

Signed-off-by: hyunback <[email protected]>
Currently dynamic quantized int8 onednn convolution has the problem. Working with ref convolution.
So replace to run fp16 mode instead.

Signed-off-by: hyunback <[email protected]>
@hyunback hyunback added WIP work in progress and removed WIP work in progress labels Nov 13, 2024
@vladimir-paramuzov vladimir-paramuzov added this pull request to the merge queue Nov 19, 2024
Merged via the queue into openvinotoolkit:master with commit 40152d1 Nov 19, 2024
154 checks passed
@hyunback hyunback deleted the fix_hybrid_quantization_reg_issue branch November 20, 2024 00:58
@hyunback hyunback restored the fix_hybrid_quantization_reg_issue branch November 21, 2024 23:47
NishantPrabhuFujitsu pushed a commit to NishantPrabhuFujitsu/openvino that referenced this pull request Nov 26, 2024
Many daily int8 models have Perf regression.(incorrect conv data type
and bias)
Fix kernel selection issue.

### Tickets:
 - *156564, 155267, 156243*

---------

Signed-off-by: hyunback <[email protected]>
Co-authored-by: Vladimir Paramuzov <[email protected]>
github-merge-queue bot pushed a commit that referenced this pull request Nov 29, 2024
### Details:
- Backport of PR #27404

### Tickets:
 - *ticket-id*

---------

Signed-off-by: hyunback <[email protected]>
Co-authored-by: Vladimir Paramuzov <[email protected]>
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.

5 participants