X Tutup
Skip to content

mep12 on manual_axis.py#4884

Merged
jenshnielsen merged 2 commits intomatplotlib:masterfrom
ericmjl:manual_axis.py_mep12
Aug 11, 2015
Merged

mep12 on manual_axis.py#4884
jenshnielsen merged 2 commits intomatplotlib:masterfrom
ericmjl:manual_axis.py_mep12

Conversation

@ericmjl
Copy link
Copy Markdown
Contributor

@ericmjl ericmjl commented Aug 7, 2015

No description provided.

@dopplershift dopplershift added the MEP: MEP12 gallery and examples improvements label Aug 7, 2015
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why comment out the line and not just remove?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Whoops! Going to remove it.

@dopplershift
Copy link
Copy Markdown
Contributor

Given the comment at the top of the file, I wonder if we should just do away with this one....

@ericmjl
Copy link
Copy Markdown
Contributor Author

ericmjl commented Aug 7, 2015

Yeah, I was thinking so too... but I hesitated just to get others' input.

@WeatherGod
Copy link
Copy Markdown
Member

It belongs somewhere... but not here. It is way too advanced for a simple pylab/pyplot example.

@ericmjl
Copy link
Copy Markdown
Contributor Author

ericmjl commented Aug 7, 2015

I will post this as an issue.

@jenshnielsen
Copy link
Copy Markdown
Member

Since there now is an issue I will merge this

jenshnielsen added a commit that referenced this pull request Aug 11, 2015
@jenshnielsen jenshnielsen merged commit 9e905ce into matplotlib:master Aug 11, 2015
@ericmjl ericmjl deleted the manual_axis.py_mep12 branch August 11, 2015 19:29
@QuLogic QuLogic added this to the v1.5.0 milestone Nov 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

MEP: MEP12 gallery and examples improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

X Tutup