Skip to content

More flexible showRatioDonut and showRatioPie#24

Open
jassler wants to merge 3 commits intocardiomoon:masterfrom
jassler:master
Open

More flexible showRatioDonut and showRatioPie#24
jassler wants to merge 3 commits intocardiomoon:masterfrom
jassler:master

Conversation

@jassler
Copy link
Copy Markdown

@jassler jassler commented Mar 26, 2024

Implementation of Issue #23. Slightly changed the case where showRatioDonut and showRatioPie is assigned a function. My previous suggestion was that this function was passed an absolute value; it might be easier, however, if the ratios are also passed, to avoid any awkward calculations.

The roxygen documentation for PieDonut includes the following example

PieDonut(acs,aes(Dx,smoking),ratioByGroup=FALSE,r0=0,showRatioPie=function(a,r) paste0(a, " [",round(r*100),"%]"))

Pie chart showing custom value formatting below labels

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants