[HOTFIX] Fix validation of scalar-scalar binary min/max operations

This recent introduction of nary min/max operations corrupted the
language validation path for scalar-scalar operations. This patch fixes
various issues related to (1) value type inference, (2) output
dimension/blocksize propagation, and (3) the handling of all scalar nary
min/max operations.

Author: mboehm7

src/main/java/org/apache/sysml/parser/BuiltinFunctionExpression.java

@@ -574,18 +574,19 @@ public void validateExpression(HashMap<String, DataIdentifier> ids, HashMap<Stri
 		case MIN:
 		case MAX:
 			//min(X), min(X,s), min(s,X), min(s,r), min(X,Y)
-			//unary
-			if (getSecondExpr() == null) {
+			if (getSecondExpr() == null) { //unary
 				checkNumParameters(1);
 				checkMatrixParam(getFirstExpr());
 				output.setDataType(DataType.SCALAR);
 				output.setValueType(id.getValueType());
 				output.setDimensions(0, 0);
 				output.setBlockDimensions (0, 0);
 			}
-			
-			//nary operation
-			else {
+			else if( getAllExpr().length == 2 ) { //binary
+				checkNumParameters(2);
+				setBinaryOutputProperties(output);
+			}
+			else { //nary
 				for( Expression e : getAllExpr() )
 					checkMatrixScalarParam(e);
 				setNaryOutputProperties(output);
@@ -1463,17 +1464,18 @@ private void setNaryOutputProperties(DataIdentifier output) {
 			e -> e.getOutput().getDataType().isScalar()) ? DataType.SCALAR : DataType.MATRIX;
 		Expression firstM = dt.isMatrix() ? Arrays.stream(getAllExpr()).filter(
 			e -> e.getOutput().getDataType().isMatrix()).findFirst().get() : null;
-		ValueType vt = dt.isMatrix() ? ValueType.DOUBLE : ValueType.BOOLEAN;
+		ValueType vt = dt.isMatrix() ? ValueType.DOUBLE : ValueType.INT;
 		for( Expression e : getAllExpr() ) {
 			vt = computeValueType(e, e.getOutput().getValueType(), vt, true);
 			if( e.getOutput().getDataType().isMatrix() )
 				checkMatchingDimensions(firstM, e, true);
 		}
 		output.setDataType(dt);
 		output.setValueType(vt);
-		output.setDimensions(firstM.getOutput().getDim1(), firstM.getOutput().getDim2());
-		output.setBlockDimensions (
-			firstM.getOutput().getRowsInBlock(), firstM.getOutput().getColumnsInBlock());
+		output.setDimensions(dt.isMatrix() ? firstM.getOutput().getDim1() : 0,
+			dt.isMatrix() ? firstM.getOutput().getDim2() : 0);
+		output.setBlockDimensions (dt.isMatrix() ? firstM.getOutput().getRowsInBlock() : 0,
+			dt.isMatrix() ? firstM.getOutput().getColumnsInBlock() : 0);
 	}
 
 	private void expandArguments() {