[ 
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)

Reply via email to