check

Author: Baunsgaard

src/main/java/org/apache/sysds/runtime/transform/decode/DecoderBin.java

@@ -142,8 +142,8 @@ public void initMetaData(FrameBlock meta) {
 				}
 				else { //_colList[ix1] > _dcCols[ix2]
 					ColumnMetadata d =meta.getColumnMetadata()[_dcCols[ix2]-1];
-					String v = meta.getString( _dcCols[ix2]-1, 0);
-					if(v.charAt(0) == '¿'){
+					String v = meta.getString(0, _dcCols[ix2]-1);
+					if(v.length() > 1 && v.charAt(0) == '¿'){
 						off += UtilFunctions.parseToLong(v.substring(1));
 					}
 					else {

src/main/java/org/apache/sysds/runtime/transform/decode/DecoderFactory.java

@@ -109,6 +109,7 @@ public static Decoder createDecoder(String spec, String[] colnames, ValueType[]
 					ArrayUtils.toPrimitive(dcIDs.toArray(new Integer[0]))));
 			}
 			if( !rcIDs.isEmpty() ) {
+				// todo figure out if we need to handle rc columns with regards to dictionary offsets.
 				ldecoders.add(new DecoderRecode(schema, !dcIDs.isEmpty(),
 					ArrayUtils.toPrimitive(rcIDs.toArray(new Integer[0]))));
 			}

src/main/java/org/apache/sysds/runtime/transform/decode/DecoderPassThrough.java

@@ -113,7 +113,7 @@ public void initMetaData(FrameBlock meta) {
 				}
 				else { //_colList[ix1] > _dcCols[ix2]
 					ColumnMetadata d =meta.getColumnMetadata()[_dcCols[ix2]-1];
-					String v = meta.getString( _dcCols[ix2]-1, 0);
+					String v = meta.getString( 0,_dcCols[ix2]-1);
 					if(v.charAt(0) == '¿'){
 						off += UtilFunctions.parseToLong(v.substring(1));
 					}