Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Few fixes #66

Merged
merged 7 commits into from Nov 21, 2019
Merged

Few fixes #66

merged 7 commits into from Nov 21, 2019

Conversation

@raver119
Copy link

raver119 commented Nov 20, 2019

This PR adds few things:

  • adds bypass in case of empty inputs to legacy transform/scalar/pairwise/broadcast ops
  • adds MatchCondition as pairwise operation
raver119 added 3 commits Nov 20, 2019
Signed-off-by: raver119 <raver119@gmail.com>
…sible

- TrueBroadcastHelper now uses samediff::threads

Signed-off-by: raver119 <raver119@gmail.com>
Signed-off-by: raver119 <raver119@gmail.com>
@raver119 raver119 requested a review from AlexDBlack Nov 20, 2019
Copy link
Member

AlexDBlack left a comment

LGTM 👍

@raver119 raver119 changed the title [WIP] Few fixes Few fixes Nov 21, 2019
Signed-off-by: raver119 <raver119@gmail.com>
raver119 added 3 commits Nov 21, 2019
Signed-off-by: raver119 <raver119@gmail.com>
# Conflicts:
#	nd4j/nd4j-backends/nd4j-tests/src/test/java/org/nd4j/linalg/custom/CustomOpsTests.java
Signed-off-by: raver119 <raver119@gmail.com>
@raver119 raver119 merged commit 064a56c into master Nov 21, 2019
@raver119 raver119 deleted the r119_empty_bypass branch Nov 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.