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

Conversation

@TaoLv
Copy link
Member

@TaoLv TaoLv commented Dec 6, 2018

Reverts #13478

#13478 brought in some non-related code change including those changes in #13503 and would cause build issue of MXNet.

@srochel Feel free to submit new PR for your changes.

@TaoLv
Copy link
Member Author

TaoLv commented Dec 6, 2018

Please do not merge. Don't know why it reverts so many commits by just clicking the revert button in #13478.

@marcoabreu
Copy link
Contributor

Don't look at the commits but just the files changes. GitHub sometimes bugs out with the commit list for reverts. Alternatively, you can do a revert using the console.

Copy link
Contributor

@marcoabreu marcoabreu left a comment

Choose a reason for hiding this comment

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

On a first glance, it looks good to me. Please double check and then feel free to merge

@TaoLv
Copy link
Member Author

TaoLv commented Dec 6, 2018

Thank you @marcoabreu . A little scared by Github. The code change looks good to me. So I think it's safe to merge.

@TaoLv TaoLv merged commit 29885c5 into master Dec 6, 2018
@szha szha deleted the revert-13478-master branch December 18, 2018 01:18
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.

2 participants