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

Yury Babak commented on IGNITE-7660:
------------------------------------

[~borans], Hi.

Please take a look on those pages before you start: 
* https://ignite.apache.org/community/contribute.html#contribute
* https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute



> Refactor LSQR algorithm
> -----------------------
>
>                 Key: IGNITE-7660
>                 URL: https://issues.apache.org/jira/browse/IGNITE-7660
>             Project: Ignite
>          Issue Type: Improvement
>          Components: ml
>            Reporter: Anton Dmitriev
>            Priority: Minor
>
> This issues is the nest step of the IGNITE-7438 task.
> In the IGNITE-7438 the AbstractLSQR implementation has been copied from the 
> SciPy implementation which has been copies from another old implementation. 
> As result the code in the 
> [AbstractLSQR|https://github.com/apache/ignite/blob/master/modules/ml/src/main/java/org/apache/ignite/ml/math/isolve/lsqr/AbstractLSQR.java]
>  looks a bit weird. All variables have meaningless names and the whole 
> algorithm written as the one method.
> The goal of this task is to refactor the LSQR code and:
>  * Make variable names more meaningful.
>  * Add comments to the variables and result (see 
> [LSQRResult|https://github.com/apache/ignite/blob/master/modules/ml/src/main/java/org/apache/ignite/ml/math/isolve/lsqr/LSQRResult.java]).
>  * Move parts of the algorithm into separate methods where it's appropriate.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to