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

Add missing code in dynamic fc impl #28026

Merged

Conversation

ahnyoung-paul
Copy link
Contributor

@ahnyoung-paul ahnyoung-paul commented Dec 12, 2024

Details:

  • Add acc_tmp in general calc in fc funcion in common include file

Tickets:

  • 158460

@ahnyoung-paul ahnyoung-paul requested review from a team as code owners December 12, 2024 02:11
@ahnyoung-paul ahnyoung-paul added the category: GPU OpenVINO GPU plugin label Dec 12, 2024
@ahnyoung-paul ahnyoung-paul force-pushed the fix_fc_bf_titled_default_kernel branch 3 times, most recently from 288cfa5 to a129c5f Compare December 13, 2024 05:13
@vladimir-paramuzov vladimir-paramuzov added the pr: needs tests PR needs tests updating label Dec 13, 2024
* Add acc_tmp in general calc in fc funcion in common include file
* Set DECOMPRESSION_SCALE_POST_OP=ON for dynamic qunatization case
@ahnyoung-paul ahnyoung-paul force-pushed the fix_fc_bf_titled_default_kernel branch from a129c5f to 82bb841 Compare December 19, 2024 15:56
* add unit test
@ahnyoung-paul ahnyoung-paul force-pushed the fix_fc_bf_titled_default_kernel branch from 82bb841 to e009729 Compare December 19, 2024 17:16
@ahnyoung-paul ahnyoung-paul removed the pr: needs tests PR needs tests updating label Dec 19, 2024
@yeonbok yeonbok added this pull request to the merge queue Dec 23, 2024
Merged via the queue into openvinotoolkit:master with commit f93d051 Dec 23, 2024
164 checks passed
@yeonbok yeonbok deleted the fix_fc_bf_titled_default_kernel branch December 23, 2024 10:00
11happy pushed a commit to 11happy/openvino that referenced this pull request Dec 23, 2024
### Details:
 - *Add acc_tmp in general calc in fc funcion in common include file*

### Tickets:
 - *158460*
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.

3 participants