This is an automated email from the ASF dual-hosted git repository.

baunsgaard pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/systemds.git


The following commit(s) were added to refs/heads/main by this push:
     new 4c90e89b08 [MINOR] Fix Aggregate Unary Row metadata
4c90e89b08 is described below

commit 4c90e89b08c9de97163640e729827ddb885a6de6
Author: composer <[email protected]>
AuthorDate: Wed May 3 17:18:35 2023 +0800

    [MINOR] Fix Aggregate Unary Row metadata
    
    This commit fixes a typo in the metadata infered on the
    outputcharacteristics when using a rowAggregate on a matrix.
    
    Closes #1829
---
 src/main/java/org/apache/sysds/hops/AggUnaryOp.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/java/org/apache/sysds/hops/AggUnaryOp.java 
b/src/main/java/org/apache/sysds/hops/AggUnaryOp.java
index f60ee6dd15..dfe0ffd61b 100644
--- a/src/main/java/org/apache/sysds/hops/AggUnaryOp.java
+++ b/src/main/java/org/apache/sysds/hops/AggUnaryOp.java
@@ -332,7 +332,7 @@ public class AggUnaryOp extends MultiThreadedHop
                if( _direction == Direction.Col && dc.colsKnown() )
                        ret = new MatrixCharacteristics(1, dc.getCols(), -1, 
-1);
                else if( _direction == Direction.Row && dc.rowsKnown() )
-                       ret = new MatrixCharacteristics(dc.getRows(), 1, -1 -1);
+                       ret = new MatrixCharacteristics(dc.getRows(), 1, -1, 
-1);
                return ret;
        }
        

Reply via email to