Skip to content

Commit 82e8c49

Browse files
committed
revert encoder factory.
1 parent 140ad7c commit 82e8c49

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

src/main/java/org/apache/sysds/runtime/transform/encode/EncoderFactory.java

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -126,12 +126,9 @@ public static MultiColumnEncoder createEncoder(String spec, String[] colnames, i
126126
rcIDs = unionDistinct(rcIDs, except(except(dcIDs, binIDs), haIDs));
127127
// Error out if the first level encoders have overlaps
128128
if (intersect(rcIDs, binIDs, haIDs, weIDs, bowIDs))
129-
throw new DMLRuntimeException("More than one encoders (recode, binning, hashing, word_embedding) on one column is not allowed:\n" + spec);
129+
throw new DMLRuntimeException("More than one encoders (recode, binning, hashing, word_embedding, bag_of_words) on one column is not allowed:\n" + spec);
130130

131131
List<Integer> ptIDs = except(UtilFunctions.getSeqList(1, clen, 1), naryUnionDistinct(rcIDs, haIDs, binIDs, weIDs, bowIDs));
132-
133-
// List<Integer> ptIDs = except(except(except(UtilFunctions.getSeqList(1, clen, 1), unionDistinct(rcIDs, haIDs)), binIDs), weIDs);
134-
135132
List<Integer> oIDs = new ArrayList<>(Arrays.asList(ArrayUtils
136133
.toObject(TfMetaUtils.parseJsonIDList(jSpec, colnames, TfMethod.OMIT.toString(), minCol, maxCol))));
137134
List<Integer> mvIDs = new ArrayList<>(Arrays.asList(ArrayUtils.toObject(

0 commit comments

Comments
 (0)