Skip to content

Conversation

@xiangpin
Copy link
Member

Fix the issue of gheatmap with the ggplot2-4.0.0

related issues #667, #666

}

attr(p2, "mapping") <- mapping
attr(p2, "data_axis") <- data_axis

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this needed; is this even evaluated in ggplot? Can this line be just removed?

@TuomasBorman
Copy link

Hi @GuangchuangYu !

Can this PR be merged as soon as possible? Bioconductor release cycle is approaching and many packages depend on ggtree.

@jurresiegers
Copy link

Experiencing the same issue. To what version of R(studio) and/or ggtree and/or Bioconductor should we downgrade in the meantime?

@vzg100
Copy link

vzg100 commented Oct 14, 2025

My team has been having the same issue as well, advice on what versions to use would be appreciated ~

@GuangchuangYu GuangchuangYu merged commit ee77397 into YuLab-SMU:devel Oct 16, 2025
jlingford added a commit to jlingford/ggtree that referenced this pull request Oct 19, 2025
rename "mapping" to "data_axis" to avoid name collisions with new version of ggplot2. Similar issue to YuLab-SMU#666, YuLab-SMU#667, and PR YuLab-SMU#668. Using "data_axis" term to be consistent with commit b6dffa9.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

5 participants