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

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:

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

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

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 //

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=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-17064306

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 >

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

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

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 thenstart the PR of

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 thenstart 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
Hello. [Message formatting is fine now. Thanks!] Le mer. 26 févr. 2020 à 15:20, chentao...@qq.com a écrit : > > Hi, > > >Hello. > > > >[Please try and set your mail client to send plain text messages.] > > > >Le mer. 26 févr. 2020 à 14:05, CT a écrit : > >> > >> Hi Gilles, > >>

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-26 Thread chentao...@qq.com
Hi, >Hello. > >[Please try and set your mail client to send plain text messages.] > >Le mer. 26 févr. 2020 à 14:05, CT a écrit : >> >> Hi Gilles, >> --Original-- >> From:"GillesSadowski"> Date:Wed, Feb 26, 2020 05:41 PM >> To:"Commons Developers List"> >>

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-26 Thread Gilles Sadowski
Hello. [Please try and set your mail client to send plain text messages.] Le mer. 26 févr. 2020 à 14:05, CT a écrit : > > Hi Gilles, > --Original-- > From:"GillesSadowski" Date:Wed, Feb 26, 2020 05:41 PM > To:"Commons Developers List" > Subject:Re: [math]

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
ld all "Kmeans" implementations inherit from a common base class? Best regards, Gilles > "org.apache.commons.math4.userguide.ClusterAlgorithmComparison": > I remain have one question below: > > -- Original -- > From

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 significant

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

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:

Re: [math] Discuss: New feature MiniBatchKMeansClusterer

2020-02-22 Thread CT
een 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.