[scikit-learn] Silhouette example - performance issue
joel.nothman at gmail.com
Tue Oct 18 05:36:38 EDT 2016
And we can reduce any substantial performance issues by merging
https://github.com/scikit-learn/scikit-learn/pull/7177 ... :)
On 15 October 2016 at 00:55, Michael Eickenberg <
michael.eickenberg at gmail.com> wrote:
> Dear Anaël,
> if you wish, you could add a line to the example verifying this
> correspondence. E.g. by moving the print function from between the two
> silhouette evaluations to after and also evaluating that average and
> printing it in parentheses.
> Probably not necessary though. A comment would do also. Or nothing :)
> On Fri, Oct 14, 2016 at 3:38 PM, Raghav R V <ragvrv at gmail.com> wrote:
>> On Fri, Oct 14, 2016 at 3:27 PM, Anaël Bonneton <anael.bonneton at gmail.com
>> > wrote:
>>> In the silhouette example (http://scikit-learn.org/stabl
>>> the silhouette values of each sample is computed twice: once with *silhouette_score
>>> *and once with *silhouette_samples.* The call to *silhouette_score* can
>>> be easily avoided by computing the average of the result of*
>>> Do you think we should remove the call to *silhouette_score* to improve
>>> the performance ? Or it is better to keep the two functions to show how to
>>> use them ?
>> When I wrote it, I intended it to be demonstrative of the two methods.
>> Not sure if we should worry about performance issues there
>> Raghav RV
>> scikit-learn mailing list
>> scikit-learn at python.org
> scikit-learn mailing list
> scikit-learn at python.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the scikit-learn