Re: Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-03-25 Thread Gilles Sadowski
Hello. > > [...] > > I have started 2 PRs to solve the problem you metioned. > > About the "CentroidInitializer" I have a new idea: > Move CentroidInitializers as inner classes of "KMeansPlusPlusCluster", > and add a construct parameter and a property "useKMea

Re: Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-03-25 Thread chentao...@qq.com
Hi, >2020-03-25 4:54 UTC+01:00, chentao...@qq.com : >> Hi, >> >>>Hello. >>> >>>Le mar. 24 mars 2020 à 06:39, chentao...@qq.com a écrit >>> : Hi, I have started 2 PRs to solve the problem you metioned. About the "CentroidInitializer" I have a new idea: Mo

Re: Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-03-25 Thread Gilles Sadowski
2020-03-25 4:54 UTC+01:00, chentao...@qq.com : > Hi, > >>Hello. >> >>Le mar. 24 mars 2020 à 06:39, chentao...@qq.com a écrit >> : >>> >>> Hi, >>> >>> I have started 2 PRs to solve the problem you metioned. >>> >>> About the "CentroidInitializer" I have a new idea: >>> Move CentroidInitiali

Re: Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-03-24 Thread chentao...@qq.com
Hi, >Hello. > >Le mar. 24 mars 2020 à 06:39, chentao...@qq.com a écrit : >> >> Hi, >> >> I have started 2 PRs to solve the problem you metioned. >> >> About the "CentroidInitializer" I have a new idea: >> Move CentroidInitializers as inner classes of "KMeansPlusPlusCluster", >> and add a

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-03-24 Thread Gilles Sadowski
Hello. Le mar. 24 mars 2020 à 06:39, chentao...@qq.com a écrit : > > Hi, > > I have started 2 PRs to solve the problem you metioned. > > About the "CentroidInitializer" I have a new idea: > Move CentroidInitializers as inner classes of "KMeansPlusPlusCluster", > and add a construct parame

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-03-23 Thread chentao...@qq.com
Hi,     I have started 2 PRs to solve the problem you metioned.     About the "CentroidInitializer" I have a new idea: Move CentroidInitializers as inner classes of "KMeansPlusPlusCluster", and add a construct parameter and a property "useKMeansPlusPlus" to  "KMeansPlusPlusCluster": ```java // A

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-03-22 Thread Gilles Sadowski
Hi Tao. I've merged PR #128 but please see my comment on the JIRA page.[1] Thanks for your interest in improving the library, Gilles [1] https://issues.apache.org/jira/browse/MATH-1509?focusedCommentId=17064306&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-1706

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-03-05 Thread Gilles Sadowski
Hello. Le jeu. 5 mars 2020 à 13:23, chentao...@qq.com a écrit : > > [...] > It is a API change, how do we start it? I'd suggest putting the new API in a (temporary) package org.apache.math4.ml.clustering2 Regards, Gilles > [...] --

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-03-05 Thread chentao...@qq.com
Hi, >Hello. > >2020-03-05 4:50 UTC+01:00, chentao...@qq.com : >> Hi, >> >>>Hi. >>> >>>Le ven. 28 févr. 2020 à 05:04, chentao...@qq.com a >>> écrit : -- chentao...@qq.com >Hi. > >Le jeu. 27 févr. 2020 à 06:17, chentao...@qq.com a > écr

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-03-05 Thread Gilles Sadowski
Hello. 2020-03-05 4:50 UTC+01:00, chentao...@qq.com : > Hi, > >>Hi. >> >>Le ven. 28 févr. 2020 à 05:04, chentao...@qq.com a >> écrit : >>> >>> >>> >>> >>> -- >>> chentao...@qq.com >>> >Hi. >>> > >>> >Le jeu. 27 févr. 2020 à 06:17, chentao...@qq.com a >>> > écrit : >>> >> >>> >> Hi, >

Re: Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-03-04 Thread chentao...@qq.com
Hi, >Hi. > >Le ven. 28 févr. 2020 à 05:04, chentao...@qq.com a écrit : >> >> >> >> >> -- >> chentao...@qq.com >> >Hi. >> > >> >Le jeu. 27 févr. 2020 à 06:17, chentao...@qq.com a >> >écrit : >> >> >> >> Hi, >> >> >> >> > [...] >> >> >> >> >> >> >> >> Do you mean I should fire a JIRA

Re: Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-28 Thread Gilles Sadowski
Hi. Le ven. 28 févr. 2020 à 05:04, chentao...@qq.com a écrit : > > > > > -- > chentao...@qq.com > >Hi. > > > >Le jeu. 27 févr. 2020 à 06:17, chentao...@qq.com a écrit > >: > >> > >> Hi, > >> > >> > [...] > >> >> >> > >> >> >> Do you mean I should fire a JIRA issue about reuse "centr

Re: Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-27 Thread chentao...@qq.com
-- chentao...@qq.com >Hi. > >Le jeu. 27 févr. 2020 à 06:17, chentao...@qq.com a écrit : >> >> Hi, >> >> > [...] >> >> >> >> >> >> Do you mean I should fire a JIRA issue about reuse "centroidOf" >> >> >> and "chooseInitialCenters", >> >> >> then start a PR and a disscuss about "Clus

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-27 Thread Gilles Sadowski
Hi. Le jeu. 27 févr. 2020 à 06:17, chentao...@qq.com a écrit : > > Hi, > > > [...] > >> >> > >> >> Do you mean I should fire a JIRA issue about reuse "centroidOf" > >> >> and "chooseInitialCenters", > >> >> then start a PR and a disscuss about "ClusterUtils"? > >> >> And then start the PR of "Mi

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-26 Thread chentao...@qq.com
Hi, > [...] >> >> >> >> Do you mean I should fire a JIRA issue about reuse "centroidOf" and >> >> "chooseInitialCenters", >> >> then start a PR and a disscuss about "ClusterUtils"? >> >> And then start the PR of "MiniBatchKMeansClusterer" after all done? >> > >> >I cannot guarantee that the whole

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-26 Thread Gilles Sadowski
4:05, CT a écrit : > >> > >> Hi Gilles, > >> -- Original -- > >> From: "GillesSadowski" >> Date: Wed, Feb 26, 2020 05:41 PM > >> To: "Commons Developers List" >> > >> Subject: Re: [math] Discus

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-26 Thread chentao...@qq.com
eb 26, 2020 05:41 PM >> To: "Commons Developers List"> >> Subject: Re: [math] Discuss: New feature MiniBatchKMeansClusterer >> >> >> > [...] >> >> Do you mean I should fire a JIRA issue about reuse "centroidOf" and >> "chooseIn

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-26 Thread Gilles Sadowski
ers List" > Subject: Re: [math] Discuss: New feature MiniBatchKMeansClusterer > > > > >[...] > > >> Do you mean this: > >>  * For JIRA issue #MATH-1509 Start a PR with > "MiniBatchKMeansClusterer", but without

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-26 Thread CT
Hi Gilles, -- Original -- From: "GillesSadowski"

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-26 Thread Gilles Sadowski
Hello. Le mer. 26 févr. 2020 à 03:49, CT a écrit : > > Hi Gilles, > > > -- Original -- > From: "GillesSadowski" Date: Wed, Feb 26, 2020 00:32 AM > To: "Commons Developers List" > Subject: Re: [math] Discuss: New fea

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-25 Thread CT
Hi Gilles, -- Original -- From: "GillesSadowski"https://github.com/apache/commons-math/pull/120 > >Sorry for seemingly nit-picking but the global issue is the same >as with PR #118: It contains too many unrelated changes. >There should be *one* PR for each batch of

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-25 Thread CT
Hi, -- Original -- From: "GillesSadowski"https://github.com/apache/commons-math/pull/120 > >Sorry for seemingly nit-picking but the global issue is the same >as with PR #118: It contains too many unrelated changes. >There should be *one* PR for each batch of signif

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-25 Thread Gilles Sadowski
-- > From: "Gilles Sadowski"; > Date: Mon, Feb 24, 2020 09:52 PM > To: "dev"; > Subject: Re: [math] Discuss: New feature MiniBatchKMeansClusterer > > >Hi. > > > >Le sam. 22 févr. 2020 à 14:37, CT a écrit : > >> > >> Hi

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-25 Thread CT
Hi Gilles:   Sorry for my unfamiliar in contribution. I started a new PR for most of your suggestion: https://github.com/apache/commons-math/pull/120 I remain have one question below: -- Original -- From: "Gilles Sadowski"https://github.com/apache/commons-math/pul

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-24 Thread Gilles Sadowski
Hi. Le sam. 22 févr. 2020 à 14:37, CT a écrit : > > Hi Gilles: > I really appricate for your patient to help me to solve the mail sending > problem, I try to set the only setting about charset "Use Unicode" for this > mail. The contents seems fine. However, there is something wrong: replyin

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-22 Thread CT
KMeans and MiniBatchKMeans by using the "org.apache.commons.math4.userguide.ClusterAlgorithmComparison": -- Original -- From: "Gilles Sadowski";; Date: Feb 19, 2020 To: "dev@commons.apache.org"; Subject: Re: [math] Discuss: New feature

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-19 Thread Gilles Sadowski
[Re-sending post so that it is attached to the original thread.] Hello. Le mar. 18 févr. 2020 à 04:49, 陈 涛 a écrit : > > Hi Gilles: > >I really do not know if anyone received my last mail, no one replay me for > a long time so I send it again and copy to you with another email system. Sorr

Fwd: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-18 Thread Gilles Sadowski
Hello. Le mar. 18 févr. 2020 à 04:49, 陈 涛 a écrit : > > Hi Gilles: > >I really do not know if anyone received my last mail, no one replay me for > a long time so I send it again and copy to you with another email system. Sorry for the delay. :-} > > > Some remarks: > > > * I didn't get why