Skip to content

change handling of legend labels which are None#1611

Merged
mdboom merged 2 commits intomatplotlib:masterfrom
segasai:master
Jan 16, 2013
Merged

change handling of legend labels which are None#1611
mdboom merged 2 commits intomatplotlib:masterfrom
segasai:master

Conversation

@segasai
Copy link
Copy Markdown
Contributor

@segasai segasai commented Dec 19, 2012

Fixes #1610

@mdboom
Copy link
Copy Markdown
Member

mdboom commented Dec 19, 2012

+1

@WeatherGod
Copy link
Copy Markdown
Member

Travis build failure is a dud (timeout error while downloading PIL). It only occurred for the py2.7 test.

Note that this PR is against master branch only. Shall we cherry-pick to v1.2.x branch?

@segasai
Copy link
Copy Markdown
Contributor Author

segasai commented Dec 19, 2012

I think, it is worth a patch for 1.2.x branch. (I won't do this cherry-picking myself, because I'm not sure what's the best way to do it on github).

@dmcdougall
Copy link
Copy Markdown
Member

The preferred method is to branch off of v1.2.x and patch the maintenance branch. Then when v1.2.x is merged into master the new changes will apply to master.

@segasai Are you comfortable doing this?

@ghost ghost assigned dmcdougall Dec 30, 2012
@mdboom
Copy link
Copy Markdown
Member

mdboom commented Jan 16, 2013

No worries. I'll merge this and cherry-pick it back to 1.2.x myself.

mdboom added a commit that referenced this pull request Jan 16, 2013
change handling of legend labels which are None
@mdboom mdboom merged commit a9dcdfd into matplotlib:master Jan 16, 2013
mdboom added a commit that referenced this pull request Jan 16, 2013
change handling of legend labels which are None
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

plotting legends none

4 participants