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.

Sign up for GitHub

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

Jump to bottom

[Op] Remove _MklFusedMatMul op's redundant ops. #34

Open
changqi1 opened this issue Mar 2, 2022 · 0 comments
Open

[Op] Remove _MklFusedMatMul op's redundant ops. #34

changqi1 opened this issue Mar 2, 2022 · 0 comments
Assignees
@marvin-Yu

Comments

@changqi1
Copy link
Owner

changqi1 commented Mar 2, 2022 •
edited
Loading

Like the following timeline from WDL training, we could find 2 const ops and 1 _MklToTf op.

Please remove those redundant ops.

image
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@marvin-Yu marvin-Yu

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[OP] Change fused matmul layout type and number thread for small size inputs. changqi1/DeepRec
2 participants
@marvin-Yu @changqi1