csdaw / ggprism

ggplot2 extension inspired by GraphPad Prism
https://csdaw.github.io/ggprism/
170 stars 21 forks source link

Guide compatability with ggplot2 >3.4.2 #25

Closed teunbrand closed 7 months ago

teunbrand commented 1 year ago

Hi Charlotte,

Apologies for the cold PR without filing an issue first. The ggplot2 package has changed the implementation of the guide system, which means that the old S3 will no longer work in ggplot2, see the news file. Because this change isn't fully backwards compatible, guides, including those in ggprism, have broken.

This PR lifts over the old S3 methods for axis drawing so that ggprism can work with the development version of ggplot2. Tests mostly* run fine in either current CRAN or development version.

* see comments

Best, Teun

csdaw commented 1 year ago

Thank for this I'll take a look ASAP

teunbrand commented 9 months ago

Hi @csdaw, I've updated the PR for further compatibility with the upcoming ggplot2 version. To test the code changes with the release candidate, you can install it with the code below:

remotes::install_github("tidyverse/ggplot2", ref = remotes::github_pull("5592"))

The release of ggplot2 3.5.0 is scheduled for the 12th of February. The progress of the release can be tracked in https://github.com/tidyverse/ggplot2/issues/5588. I hope that this PR might help ggprism get out a fix if necessary.

teunbrand commented 9 months ago

This is just a kind reminder that the release is scheduled soon.

samuel-marsh commented 8 months ago

Hi @csdaw,

Just wanted to add quick ping that it would be great to merge this soon and push to CRAN. I import theme_prism in my package scCustomize and recently ran into check errors due to this upcoming changes in ggplot2 (see https://github.com/tidyverse/ggplot2/issues/5702)

For now I'm going to work in a temp fix so that I can push my release out sooner but would be great to be able to remove that in near future.

Thanks! Sam

csdaw commented 7 months ago

Sorry for the delay, life things got in the way.

Thanks heaps @teunbrand for all your work 🤩