-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issue with upcoming ggplot2 #19
Comments
Thanks for the heads up, I've tracked down the error. It happens when I use
which returns the error
Interestingly, I don't get the error if I set both limits to NA, in which case the plot uses the Is this something that will be fixed before the next release or do I need to update daiquiri to work around it? (Note, I do need to be able to create plots that contain all NAs.) Thanks |
Thanks for tracking down the bug! This sounds like something ggplot2 should fix instead of daquiri. |
Considered fixed by tidyverse/ggplot2#5681 |
Hello there,
We have been preparing a new release of ggplot2 and during a reverse dependency check, it became apparent that the prospective ggplot2 3.5.0 would break daiquiri.
Unfortunately we weren't able to diagnose what is causing the error, but our check appears to point to the
plot_overview_combo_static()
function. It is entirely posible that this might be a false positive failure, but we'd like to report on it anyway to give an early heads up.To test the code changes with the release candidate, you can install it with the code below:
The release of ggplot2 3.5.0 is scheduled for the 12th of Februari. The progress of the release can be tracked in tidyverse/ggplot2#5588.
The text was updated successfully, but these errors were encountered: