Skip to content

cex_label_category in emapplot gone #322

@Mitmischer

Description

@Mitmischer

I'm using enrichplot 1.28.2 (according to BiocManager::valid() ).

In earlier versions, one could set the label size in emapplot via cex_label_category.

This options seems to be gone now. I did not find a mention of "cex_label_category" in the changelog (https://github.com/YuLab-SMU/enrichplot/blob/devel/NEWS.md), however there were changes to emapplot.
As there was a breaking change, I would expect to find this information somewhere in the changelog.

Also, I can still find mention of the parameter online (https://rdrr.io/bioc/enrichplot/man/emapplot.html), but - rightfully so - not in my local installation. I don't know who maintains the online documentation, but the last change was on 2021 (according to the image at the bottom on the page). This documentation should be updated or removed.

Anyway - in the outdated documentation I found declarations of parameter renaming (node_label_size - size of node label, this parameter has been changed to cex_label_category). That shows that renaming has been well handled in the past.

Of course, the issue at hand is a minor issue. Ultimately and more generally, I want to make suggestions about clearer communication of breaking changes (in the changelog and in the function itself (for example, consider the way sklearn documents their changes: https://scikit-learn.org/stable/modules/generated/sklearn.discriminant_analysis.LinearDiscriminantAnalysis.html), finer commit messages with a more thorough description of changes and consistency of online documentation to make such cases easier to handle in the future.

Lastly, which parameter replaces cex_label_category? size_category?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions