Make treemapify compatible with plotly r package

[wilkox]

Make treemapify compatible with plotly r package

[wilkox]

@fernandovmacedo I'd be happy to help, but I'm not really familiar with plotly – what would be required at my end to make treemapify plotly-compatible?

[cpsievert]

Hi @fernandovmacedo, @moutikabdessabour is looking into a converter here plotly/plotly.R#2051

As I pointed out here plotly/plotly.R#2051 (comment), some of the implementation borrows heavily from {treemapify}'s code, and I'd like to avoid any potential GPL vs MIT licensing issues. Would you be willing to either release {treemapify} as MIT and/or sign a contributor agreement? We would, of course, also list you as an author in plotly's DESCRIPTION if you'd like.

[wilkox]

@cpsievert I'd be happy to sign a contributor agreement if that helps. Bob Rudis (@hrbrmstr) is also a listed author of treemapify and there are three other contributors to the package, so I assume at least Bob would have to sign too.

For what it's worth, the only section of the pull request that strikes me as being lifted from treemapify is lines 706–713 of R/layers2traces.R, which probably deserve to be refactored anyway.

[hrbrmstr]

I can def sign as well.

…

On Mon, Nov 15, 2021 at 01:27 David Wilkins ***@***.***> wrote: @cpsievert <https://github.com/cpsievert> I'd be happy to sign a contributor agreement if that helps. Bob Rudis ***@***.*** <https://github.com/hrbrmstr>) is also a listed author of treemapify and there are three other contributors to the package, so I assume at least Bob would have to sign too. For what it's worth, the only section of the pull request <https://github.com/plotly/plotly.R/pull/2051/files> that strikes me as being lifted from treemapify is lines 706–713 of R/layers2traces.R, which probably deserve to be refactored anyway. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#40 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAD4PNX4HN34JDSEC5GKGSDUMCR3ZANCNFSM4NO2CFCQ> .