-
Notifications
You must be signed in to change notification settings - Fork 6.8k
[BACKPORT][BUGFIX][FEATURE] Add oneDNN 1D and 3D deconvolution support and fix bias #20292
Conversation
Hey @PawelGlomski-Intel , Thanks for submitting the PR
CI supported jobs: [miscellaneous, edge, windows-cpu, unix-cpu, unix-gpu, website, windows-gpu, centos-cpu, clang, sanity, centos-gpu] Note: |
@mxnet-bot run ci [centos-gpu, unix-cpu, unix-gpu, windows-cpu] |
Jenkins CI successfully triggered : [unix-cpu, centos-gpu, unix-gpu, windows-cpu] |
@mxnet-bot run ci [unix-cpu] |
Jenkins CI successfully triggered : [unix-cpu] |
@mxnet-bot run ci [unix-cpu] |
Jenkins CI successfully triggered : [unix-cpu] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@szha can you preview review? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for the forward-port. Could you help make sure that the functionality is tested from the mx.npx module instead of from the legacy Symbol API?
fcfd77d
to
c1b6b04
Compare
@mxnet-bot run ci [miscellaneous, website] |
Jenkins CI successfully triggered : [website, miscellaneous] |
@szha Can you please review (excluding mkldnn files)? |
@PawelGlomski-Intel thank you! |
Description
Backport of #20107 and #20137
Checklist
Essentials
Changes