Skip to content

[INTEL MKL] Code cleanup#21593

Merged
tensorflow-copybara merged 2 commits intotensorflow:masterfrom
Intel-tensorflow:agramesh/MKL_ML_update
Aug 31, 2018
Merged

[INTEL MKL] Code cleanup#21593
tensorflow-copybara merged 2 commits intotensorflow:masterfrom
Intel-tensorflow:agramesh/MKL_ML_update

Conversation

@agramesh1
Copy link
Copy Markdown
Contributor

@agramesh1 agramesh1 commented Aug 14, 2018

Renamed INTEL_MKL_ML uses that were missed by previous commit a8e78e2 . Also added deprecation warning for INTEL_MKL_ML_ONLY. The DNN APIs in MKL ML have not been updated in a while and all new optimizations and enhancements are done in MKL DNN open source.

@agramesh1 agramesh1 changed the title [INTEL MKL] Renamed INTEL_MKL_ML in a few files. [INTEL MKL] Code cleanup Aug 20, 2018
Copy link
Copy Markdown
Member

@tatianashp tatianashp left a comment

Choose a reason for hiding this comment

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

Thanks for the clean up!

@tatianashp tatianashp added the kokoro:force-run Tests on submitted change label Aug 28, 2018
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Aug 28, 2018
@tatianashp tatianashp added the kokoro:force-run Tests on submitted change label Aug 28, 2018
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Aug 28, 2018
@tatianashp tatianashp added the ready to pull PR ready for merge process label Aug 28, 2018
@aaroey aaroey self-assigned this Aug 30, 2018
@tensorflow-copybara tensorflow-copybara merged commit 8077ae1 into tensorflow:master Aug 31, 2018
tensorflow-copybara pushed a commit that referenced this pull request Aug 31, 2018
@nammbash nammbash deleted the agramesh/MKL_ML_update branch April 28, 2020 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes ready to pull PR ready for merge process

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants