Re: [ML] IGNITE-9282 Naive Bayes task split

2018-10-06 Thread Alexey Zinoviev
Dear Ravil, will help with review next week, currently a lot of work with release documentation review/writing. Of course, you could assign Standart Scaler task for yourself. пт, 5 окт. 2018 г. в 16:38, Ravil Galeyev : > Hi Team > > so, can somebody look at PR https://github.com/apache/ignite/p

Re: [ML] IGNITE-9282 Naive Bayes task split

2018-10-05 Thread Ravil Galeyev
Hi Team so, can somebody look at PR https://github.com/apache/ignite/pull/4869 with naive Bayes? @zaleslaw, @ybabak do you have time? if reviewing takes time, can I take some other task? I.e. I can implement IGNITE-9284 Standart scaler for a whi

Re: [ML] IGNITE-9282 Naive Bayes task split

2018-10-01 Thread Alexey Zinoviev
Great, I support this idea, will help with review too -- Sent from: http://apache-ignite-developers.2346864.n4.nabble.com/

Re: [ML] IGNITE-9282 Naive Bayes task split

2018-09-30 Thread Ravil Galeyev
Hi Yuriy, I created new tickets for other Bayes classifiers IGNITE-9745 Multinomial Naive Bayes IGNITE-9746 Complement Naive Bayes IGNITE-9747

Re: [ML] IGNITE-9282 Naive Bayes task split

2018-09-30 Thread Yuriy Babak
Hi Ravil, I think this is a good idea. I prefer to have several small single-feature tickets instead of a big one with several features. I will start reviewing 9282 on this week. Also, I looking forward to seeing those new tickets. Regards, Yuriy вс, 30 сент. 2018 г. в 3:58, Ravil Galeyev : >

[ML] IGNITE-9282 Naive Bayes task split

2018-09-29 Thread Ravil Galeyev
Hi Team, I work on implementing Naive Bayes classifiers. Withing IGNITE-9282 I implemented a Gaussian Bayes and created a PR https://github.com/apache/ignite/pull/4869 But there are already a lot of changes That's' why I'd like to create separa