Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Revert "[MXNET-367] update mkldnn to v0.14 and disable building test examples" #10808

Merged
merged 1 commit into from
May 4, 2018

Conversation

marcoabreu
Copy link
Contributor

@marcoabreu marcoabreu commented May 4, 2018

Reverts #10736

This PR breaks master

@pengzhao-intel @TaoLv @ashokei @zheng-da

@marcoabreu marcoabreu requested a review from szha as a code owner May 4, 2018 07:05
@TaoLv
Copy link
Member

TaoLv commented May 4, 2018

Cpp tests are still not enabled? I think that PR can be intercepted by mkldnn cpp tests.
Anyway, maybe we should ask author to rebase code if his last build is 1 day ago before merging. (for #10736, it's about 6 days ago.)

@marcoabreu
Copy link
Contributor Author

The problem here is that there were only 10 hours in between the merges of the two clashing PRs (see the email on dev@ for details). It's hard to avoid that without causing too much inconvenience.

@TaoLv
Copy link
Member

TaoLv commented May 4, 2018

Hah, I see. You are right, it's hard to avoid that. But we still can set a last build time constraint to reduce risk of that.

@marcoabreu marcoabreu merged commit 66365ef into master May 4, 2018
anirudh2290 pushed a commit to anirudh2290/mxnet that referenced this pull request May 7, 2018
jinhuang415 pushed a commit to jinhuang415/incubator-mxnet that referenced this pull request May 29, 2018
@marcoabreu marcoabreu deleted the revert-10736-mkldnn_build_version_flags branch May 31, 2018 14:43
rahul003 pushed a commit to rahul003/mxnet that referenced this pull request Jun 4, 2018
zheng-da pushed a commit to zheng-da/incubator-mxnet that referenced this pull request Jun 28, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants