Repository: systemml Updated Branches: refs/heads/master fe4779cef -> d0c5c5d29
[SYSTEMML-1668] Fix rbind size estimates in BinaryOp In BinaryOp.inferOutputCharacteristics, RBIND is not checked and CBIND is checked twice. Fix the second case to refer to RBIND. Closes #533. Project: http://git-wip-us.apache.org/repos/asf/systemml/repo Commit: http://git-wip-us.apache.org/repos/asf/systemml/commit/d0c5c5d2 Tree: http://git-wip-us.apache.org/repos/asf/systemml/tree/d0c5c5d2 Diff: http://git-wip-us.apache.org/repos/asf/systemml/diff/d0c5c5d2 Branch: refs/heads/master Commit: d0c5c5d29d4cc299649573fed3a2e112e828412d Parents: fe4779c Author: Dylan Hutchison <[email protected]> Authored: Thu Jun 8 16:22:28 2017 -0700 Committer: Matthias Boehm <[email protected]> Committed: Thu Jun 8 16:22:28 2017 -0700 ---------------------------------------------------------------------- src/main/java/org/apache/sysml/hops/BinaryOp.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/systemml/blob/d0c5c5d2/src/main/java/org/apache/sysml/hops/BinaryOp.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/apache/sysml/hops/BinaryOp.java b/src/main/java/org/apache/sysml/hops/BinaryOp.java index 17a099f..ccf73c4 100644 --- a/src/main/java/org/apache/sysml/hops/BinaryOp.java +++ b/src/main/java/org/apache/sysml/hops/BinaryOp.java @@ -844,7 +844,7 @@ public class BinaryOp extends Hop if( ldim1 > 0 || ldim2 > 0 || lnnz >= 0 ) return new long[]{ldim1, ldim2, lnnz}; } - else if( op== OpOp2.CBIND ) { + else if( op == OpOp2.RBIND ) { long ldim1 = -1, ldim2 = -1, lnnz = -1; if( mc[0].colsKnown() || mc[1].colsKnown() )
