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

generic:sycl: Inner Product Backward #2360

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

Conversation

AD2605
Copy link
Contributor

@AD2605 AD2605 commented Jan 9, 2025

Description

Adds the backward_data and the backward_weight of the inner product for the generic backend

Fixes # (github issue)

Checklist

General

  • Do all unit and benchdnn tests (make test and make test_benchdnn_*) pass locally for each commit?
  • Have you formatted the code using clang-format?

@AD2605 AD2605 requested review from a team as code owners January 9, 2025 07:33
@github-actions github-actions bot added documentation A request to change/fix/improve the documentation. Codeowner: @oneapi-src/onednn-doc platform:gpu-nvidia Codeowner: @oneapi-src/onednn-gpu-nvidia platform:gpu-generic Codeowner: @oneapi-src/onednn-gpu-generic component:tests Codeowner: @oneapi-src/onednn-arch labels Jan 9, 2025
Copy link
Contributor

@ranukund ranukund left a comment

Choose a reason for hiding this comment

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

A couple of edits suggested. Please incorporate as you see fit, thanks!

* Supported post-ops: All the post operations as mentioned in the specification are supported.
* Supported data types: All possible data combinations as listed in the specification are supported.
* Supported post-ops: All the post-ops as mentioned in the specification are supported.
* The backward pass does not support post-ops. One should not use post-ops in the forward pass during training
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this sentence could be included in a note format as opposed to listing this as a bullet.

Suggested change
* The backward pass does not support post-ops. One should not use post-ops in the forward pass during training
Note: The backward pass does not support post-ops. You should not use post-ops in the forward pass during training.

* Supported data types: All possible data combinations listed in the oneDNN specification are supported.
* Supported post-ops: All the post operations as mentioned in the specification are supported.
* Supported data types: All possible data combinations as listed in the specification are supported.
* Supported post-ops: All the post-ops as mentioned in the specification are supported.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* Supported post-ops: All the post-ops as mentioned in the specification are supported.
* Supported post-ops: All post-ops mentioned in the specification are supported.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:tests Codeowner: @oneapi-src/onednn-arch documentation A request to change/fix/improve the documentation. Codeowner: @oneapi-src/onednn-doc platform:gpu-generic Codeowner: @oneapi-src/onednn-gpu-generic platform:gpu-nvidia Codeowner: @oneapi-src/onednn-gpu-nvidia
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants