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

[fix] Support nullop in transpose #15865

Merged
merged 11 commits into from
Sep 2, 2019

Conversation

kshitij12345
Copy link
Contributor

Description

Fixes #15709

Copy link
Contributor

@apeforest apeforest left a comment

Choose a reason for hiding this comment

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

LGTM. @kshitij12345 thank you for your contribution. Could you please re-trigger the CI again?

@kshitij12345
Copy link
Contributor Author

@apeforest The pending pipelines have actually completed successfully. I guess there was an issue in notifying the Github. Do I have to retrigger it? Or is it ok this way?

@TaoLv
Copy link
Member

TaoLv commented Aug 29, 2019

@apeforest The pending pipelines have actually completed successfully. I guess there was an issue in notifying the Github. Do I have to retrigger it? Or is it ok this way?

Please retrigger. We cannot merge a PR unless the CI passing is reflected to the github page. The merge button is disabled in this case.

@kshitij12345
Copy link
Contributor Author

Oh. Sure.

@TaoLv
Copy link
Member

TaoLv commented Sep 2, 2019

@kshitij12345 Thanks for your contribution. Merging now.

@TaoLv TaoLv merged commit 5699939 into apache:master Sep 2, 2019
@kshitij12345 kshitij12345 deleted the fix/support-nullop/transpose branch September 3, 2019 02:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Transpose does not support inplace
5 participants