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

Denis Magda commented on IGNITE-7504:
-------------------------------------

[~amalykh] , [~chief] :
 * look for TODOs in the doc and help to fix the unclear parts.
 * Why is the example class called {{MNISTExample}}? Let's rename it to 
DecisionTreesExample. Otherwise, people won't get what's inside.
 * The MNISTExampe has several problems itself:
 ** some of the lines' width takes more than 120 columns 
([https://stackoverflow.com/questions/24355976/how-to-change-line-width-in-intellij-from-120-character)]
 ** {{cfg.setEvictionPolicy(null);}} is deprecated
 ** 
{{cfg.setAtomicityMode(CacheAtomicityMode.ATOMIC);}}cfg.setCacheMode(CacheMode.PARTITIONED);
 cfg.setBackups(0); - these are already default parameters! Clean the code 
please.

> Decision tree documentation
> ---------------------------
>
>                 Key: IGNITE-7504
>                 URL: https://issues.apache.org/jira/browse/IGNITE-7504
>             Project: Ignite
>          Issue Type: Task
>          Components: documentation, ml
>            Reporter: Yury Babak
>            Assignee: Denis Magda
>            Priority: Major
>              Labels: documentation
>             Fix For: 2.4
>
>
> We want to add Decision tree documentation



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

Reply via email to