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 fc 4d for onednn #26479

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

steve-y
Copy link
Contributor

@steve-y steve-y commented Sep 9, 2024

Details:

  • Fixed for onednn to work with the original fc
  • Added unit test

Tickets:

  • 150433

@steve-y steve-y requested review from a team as code owners September 9, 2024 00:45
@github-actions github-actions bot added the category: GPU OpenVINO GPU plugin label Sep 9, 2024
@steve-y steve-y force-pushed the sy/fix_enable_fc_4d branch 3 times, most recently from 16215e2 to 2d91919 Compare September 13, 2024 03:14
}
} else {
if (desc->input_size > 5) {
input_layout.set_partial_shape(reshape_to_2d(input_pshape, feature));
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you please add the tc for this case too?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It already has tc for 6d such as static_6d, static_6d_input_cached, dynamic_6d, and dynamic_6d_cached. Did you mean the other cases?

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.

2 participants