[systemds] branch master updated: [MINOR] Fix parsing gpu binary aggregate instruction

[markd]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new d181ea5  [MINOR] Fix parsing gpu binary aggregate instruction
d181ea5 is described below

commit d181ea511a5e1735c844792c116ee9ecf19b2ff0
Author: Mark Dokter <m...@dokter.cc>
AuthorDate: Mon Sep 14 21:49:05 2020 +0200

    [MINOR] Fix parsing gpu binary aggregate instruction
    
    An additional integer for thread count was introduced in the string 
representation of bin_agg instructions which made the gpu version fail.
---
 .../runtime/instructions/gpu/AggregateBinaryGPUInstruction.java     | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/src/main/java/org/apache/sysds/runtime/instructions/gpu/AggregateBinaryGPUInstruction.java
 
b/src/main/java/org/apache/sysds/runtime/instructions/gpu/AggregateBinaryGPUInstruction.java
index 78b05a0..93cda1a 100644
--- 
a/src/main/java/org/apache/sysds/runtime/instructions/gpu/AggregateBinaryGPUInstruction.java
+++ 
b/src/main/java/org/apache/sysds/runtime/instructions/gpu/AggregateBinaryGPUInstruction.java
@@ -57,12 +57,12 @@ public class AggregateBinaryGPUInstruction extends 
GPUInstruction {
                String opcode = parts[0];
                if ( !opcode.equalsIgnoreCase("ba+*"))
                        throw new 
DMLRuntimeException("AggregateBinaryInstruction.parseInstruction():: Unknown 
opcode " + opcode);
-               InstructionUtils.checkNumFields( parts, 5 );
+               InstructionUtils.checkNumFields( parts, 6 );
                CPOperand in1 = new CPOperand(parts[1]);
                CPOperand in2 = new CPOperand(parts[2]);
                CPOperand out = new CPOperand(parts[3]);
-               boolean isLeftTransposed = Boolean.parseBoolean(parts[4]);
-               boolean isRightTransposed = Boolean.parseBoolean(parts[5]);
+               boolean isLeftTransposed = Boolean.parseBoolean(parts[5]);
+               boolean isRightTransposed = Boolean.parseBoolean(parts[6]);
                AggregateBinaryOperator aggbin = 
InstructionUtils.getMatMultOperator(1);
                return new AggregateBinaryGPUInstruction(aggbin, in1, in2, out, 
opcode, str, isLeftTransposed, isRightTransposed);      
        }