Hi Gavin,
Thank you for the e-mail, I have already done so in:
https://github.com/talgalili/dendextend/blob/master/R/cutree.dendrogram.R
I wrote this e-mail since I understand that long term we would like to
avoid having packages override functions from base R packages. So since I
am extending cutree for dendrogram, it would have been nice if the current
function would have been S3, so that I wouldn't need to override it the way
I do.
Best,
Tal
----------------Contact
Details:-------------------------------------------------------
Contact me: tal.gal...@gmail.com |
Read me: www.talgalili.com (Hebrew) | www.biostatistics.co.il (Hebrew) |
www.r-statistics.com (English)
----------------------------------------------------------------------------------------------
On Mon, Jun 8, 2015 at 7:24 PM, Gavin Simpson <ucfa...@gmail.com> wrote:
> In lieu of that Tal, why not do as Writing R Extensions illustrates and
> add the following to your package
>
>
> On 6 June 2015 at 04:44, Tal Galili <tal.gal...@gmail.com> wrote:
>
>> Hello all,
>>
>> A question/suggestion:
>>
>> I was wondering if there is a chance of changing stats::cutree to be S3
>> and
>> use cutree.hclust?
>>
>> For example:
>>
>> cutree <- function(tree, k = NULL, h = NULL,...)
>> {
>> UseMethod("cutree")
>> }
>>
>> cutree.hclust <- stats::cutree
>> # This will obviously need the actual content of stats::cutree
>>
>>
>> This would be nicer for people like me to add new methods to cutree. For
>> example the cutree.dendrogram function
>> <
>> https://github.com/talgalili/dendextend/blob/master/R/cutree.dendrogram.R>
>> I
>> implemented in dendextend,
>>
>>
>>
>> With regards,
>> Tal
>>
>> [[alternative HTML version deleted]]
>>
>> ______________________________________________
>> R-devel@r-project.org mailing list
>> https://stat.ethz.ch/mailman/listinfo/r-devel
>>
>
>
>
> --
> Gavin Simpson, PhD
>
[[alternative HTML version deleted]]
______________________________________________
R-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-devel
