[ 
https://issues.apache.org/jira/browse/FLINK-3330?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15131745#comment-15131745
 ] 

Chiwan Park commented on FLINK-3330:
------------------------------------

I think that adding {{SparseVector}} support requires that {{size}}, 
{{indices}}, {{data}} are to be mutable. But mutable variable is not good 
especially for public variable. So I would like to change {{size}}, 
{{indices}}, {{data}} to package-private mutable.

Does this make sense? [~till.rohrmann], [~tvas]

> Add SparseVector support to BLAS library in FlinkML
> ---------------------------------------------------
>
>                 Key: FLINK-3330
>                 URL: https://issues.apache.org/jira/browse/FLINK-3330
>             Project: Flink
>          Issue Type: Improvement
>          Components: Machine Learning Library
>    Affects Versions: 1.0.0
>            Reporter: Chiwan Park
>            Assignee: Chiwan Park
>
> A user reported the problem using {{GradientDescent}} algorithm with 
> {{SparseVector}}. 
> (http://mail-archives.apache.org/mod_mbox/flink-user/201602.mbox/%3CCAMJxVsiNRy_B349tuRpC%2BY%2BfyW7j2SHcyVfhqnz3BGOwEHXHpg%40mail.gmail.com%3E)
> It seems lack of SparseVector support in {{BLAS.axpy}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to