Fix memory estimates remove empty intermediates (underestimation)

Project: http://git-wip-us.apache.org/repos/asf/incubator-systemml/repo
Commit: 
http://git-wip-us.apache.org/repos/asf/incubator-systemml/commit/8c3f9727
Tree: http://git-wip-us.apache.org/repos/asf/incubator-systemml/tree/8c3f9727
Diff: http://git-wip-us.apache.org/repos/asf/incubator-systemml/diff/8c3f9727

Branch: refs/heads/master
Commit: 8c3f9727691e2f0441d11c69106b26c0958e33e1
Parents: 9464101
Author: Matthias Boehm <[email protected]>
Authored: Sun Dec 6 14:17:35 2015 -0800
Committer: Matthias Boehm <[email protected]>
Committed: Sun Dec 6 14:17:35 2015 -0800

----------------------------------------------------------------------
 src/main/java/org/apache/sysml/hops/ParameterizedBuiltinOp.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-systemml/blob/8c3f9727/src/main/java/org/apache/sysml/hops/ParameterizedBuiltinOp.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/sysml/hops/ParameterizedBuiltinOp.java 
b/src/main/java/org/apache/sysml/hops/ParameterizedBuiltinOp.java
index b527022..89f6959 100644
--- a/src/main/java/org/apache/sysml/hops/ParameterizedBuiltinOp.java
+++ b/src/main/java/org/apache/sysml/hops/ParameterizedBuiltinOp.java
@@ -727,7 +727,7 @@ public class ParameterizedBuiltinOp extends Hop
                        if( cols )
                        {
                                //selection vector: boolean array in the number 
of columns 
-                               ret += OptimizerUtils.BIT_SIZE * dim2;
+                               ret += OptimizerUtils.BOOLEAN_SIZE * dim2;
                                
                                //removeEmpty-cols has additional memory 
requirements for intermediate 
                                //data structures in order to make this a 
cache-friendly operation.
@@ -736,7 +736,7 @@ public class ParameterizedBuiltinOp extends Hop
                        else //rows
                        {
                                //selection vector: boolean array in the number 
of rows 
-                               ret += OptimizerUtils.BIT_SIZE * dim1;
+                               ret += OptimizerUtils.BOOLEAN_SIZE * dim1;
                        }
                }
                else if( _op == ParamBuiltinOp.REXPAND )

Reply via email to