Skip to content

Conversation

@xuhdev
Copy link
Contributor

@xuhdev xuhdev commented Nov 13, 2019

Emancipated by pytorch/pytorch#29591

Copy link
Contributor

@ailzhang ailzhang left a comment

Choose a reason for hiding this comment

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

Thanks! would you mind rebase on top of master?

@xuhdev xuhdev force-pushed the remove-logical-xor branch from f5760e4 to 73015e1 Compare November 13, 2019 22:12
@xuhdev
Copy link
Contributor Author

xuhdev commented Nov 13, 2019

@ailzhang Done!

@ailzhang
Copy link
Contributor

ailzhang commented Nov 13, 2019

Aha sadly this test still fails at half type. (XLA currently doesn't support half). We'll happily accept this PR if it only change the comment.

@xuhdev
Copy link
Contributor Author

xuhdev commented Nov 14, 2019

Maybe we can wait until pytorch/pytorch#28162 is merged? There the test code has specifically skipped XLA for half type

@ailzhang
Copy link
Contributor

Maybe we can wait until pytorch/pytorch#28162 is merged? There the test code has specifically skipped XLA for half type

SG! Let me put a label on this so that we come back to it later :D

@xuhdev
Copy link
Contributor Author

xuhdev commented Dec 4, 2019

Closing now, as it should give way to #1428

@xuhdev xuhdev closed this Dec 4, 2019
@xuhdev xuhdev deleted the remove-logical-xor branch December 4, 2019 00:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants