Skip to content

ENH: plotting methods can unpack labeled data [MOVED TO #4829] #4787

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.

Sign up for GitHub

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

Closed
wants to merge 8 commits into from
Closed

ENH: plotting methods can unpack labeled data [MOVED TO #4829] #4787

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
1e21903
ENH: plotting methods can unpack labeled data
tacaswell Jul 25, 2015
1cf0f38
ENH: Make implicit x in plot pandas aware
tacaswell Jul 26, 2015
676eeb4
ENH: add white-list of args/kwargs to relpace
tacaswell Jul 26, 2015
9e6e7b4
MNT: remove unused rcparam
tacaswell Jul 26, 2015
5ace465
MNT: python 2.6 does not support set literals
tacaswell Jul 27, 2015
e90d859
ENH: pass at decorator which extracts a label
tacaswell Jul 28, 2015
6646545
FIX: fix typo, slightly rename variables
tacaswell Jul 29, 2015
4059b44
FIX: fix yet more typos
tacaswell Jul 29, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
FIX: fix yet more typos
  • Loading branch information
@tacaswell
tacaswell committed Jul 29, 2015
commit 4059b448666c63b6b532f93ccde3043f60fd8e3c
5 changes: 4 additions & 1 deletion lib/matplotlib/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1563,6 +1563,9 @@ def inner(ax, *args, **kwargs):
new_kwargs = dict(
(k, _replacer(data, v) if k in wl_kwargs else v)
for k, v in six.iteritems(kwargs))
else:
new_args, new_kwargs = args, kwargs

if (label_pos is not None and ('label' not in kwargs or
Copy link
Contributor

Choose a reason for hiding this comment

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

This doesn't check against the case that a user called the function with args alone (aka: the new label is in args). Not sure how frequent that is...

def plotter(ax, x, y, label=None):
    pass

plotter(None, 1,2,"label")

Copy link
Member Author

Choose a reason for hiding this comment

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

Good point, but I am not super worried about that in this case because in order for this code path to be hit they have to be using the data kwarg so we won't break un-touched user code.

Some of the methods take label as an explicit input, but many take it through a blind **kwargs. I suspect that we will have to inspect func on the way in.

kwargs['label'] is None)):
if len(args) > label_arg:
Expand All @@ -1572,7 +1575,7 @@ def inner(ax, *args, **kwargs):
pass
elif label_kwarg in kwargs:
try:
kwargs['label'] = kwargs[label_kwarg].name
new_kwargs['label'] = kwargs[label_kwarg].name
except AttributeError:
pass

Expand Down