-
-
Notifications
You must be signed in to change notification settings - Fork 8.3k
Open
Labels
API: consistencyMaintenancestatus: needs comment/discussionneeds consensus on next stepneeds consensus on next step
Description
Summary
Hey, not sure if there is in fact an internal reason for this choice, but personally I find it odd
that plt.hexbin() uses a upper-case C as kwarg to assign custom values for aggregation in each bin.
matplotlib/lib/matplotlib/axes/_axes.py
Line 5469 in 29b3564
| def hexbin(self, x, y, C=None, gridsize=100, bins=None, |
In contrast, plt.scatter uses a lower-case c to assign values to each datapoint.
matplotlib/lib/matplotlib/axes/_axes.py
Line 5177 in 29b3564
| def scatter(self, x, y, s=None, c=None, marker=None, cmap=None, norm=None, |
Proposed fix
I believe it would be good in the long run to switch from upper-case C to lower-case c in plt.hexbin()
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
API: consistencyMaintenancestatus: needs comment/discussionneeds consensus on next stepneeds consensus on next step