[ https://issues.apache.org/jira/browse/SYSTEMML-1570?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16022381#comment-16022381 ]
Matthias Boehm commented on SYSTEMML-1570: ------------------------------------------ [~nakul02] as it turns out, this task is kind of blocked because our GPU backend does not support binary max and min, specifically matrix-scalar max and min. Any plans to add this in the near future? > Remove fused sel+ operator > -------------------------- > > Key: SYSTEMML-1570 > URL: https://issues.apache.org/jira/browse/SYSTEMML-1570 > Project: SystemML > Issue Type: Task > Reporter: Matthias Boehm > Fix For: SystemML 1.0 > > > The fused operator sel+ (select positive values) is applied for patterns like > (X>0)*X and max(X,0) in order to eliminate unnecessary intermediates. It > stems from a time when max was sparse-unsafe and hence inefficient over > sparse data. However, meanwhile we mark scalar operators as conditionally > sparse-safe depending on the given scalar constant c, which applies for max > if c<=0. Hence, this sel+ operator is meanwhile completely useless and should > be removed. > Furthermore, we should also generalize the rewrites to rewrite the selection > of negative values (X<0)*X to min(X,0) -- This message was sent by Atlassian JIRA (v6.3.15#6346)