-
Notifications
You must be signed in to change notification settings - Fork 2.1k
geom_hex no longer recognizes ..count.. #1747
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
Comments
@hadley - is there a reason for renaming |
No, just an oversight |
@hadley Should this be fixed in |
The default aes in |
Ah this is going to be awkward to fix because it's natural for Might just be better to leave the behaviour as is, and better document. |
It's not too big of an issue as it can be intercepted in |
But how would you know when to make the transformation? It seems like a decent amount of work for little gain. |
See PR #1765 |
Fixes #1747 * Rename value to count
Strongly related to #1608.
This used to work:
But now returns the error:
object 'count' not found
. #1608 reported the same behavior for..density..
but this has been resolved since. There is a perfectly viable workaround using..value..
instead, but..count..
seems more consistent with other stats and geoms, and old code may no longer work.Also see on SO.
The text was updated successfully, but these errors were encountered: