PEP8-compliance on axes.py (patch 3 / 4) #1522

Merged
merged 1 commit into from Dec 3, 2012

Conversation

Projects
None yet
3 participants
@NelleV
Contributor

NelleV commented Nov 21, 2012

Hello,

Here is the third patch on the axes.py module.

Cheers,
N

if 'transform' in kwargs:
lines_kw['transform'] = kwargs['transform']
# arrays fine here, they are booleans and hence not units
if not iterable(lolims):
- lolims = np.asarray([lolims]*len(x), bool)
- else: lolims = np.asarray(lolims, bool)
+ lolims = np.asarray([lolims] * len(x), bool)

This comment has been minimized.

Show comment Hide comment
@dmcdougall

dmcdougall Dec 3, 2012

Member

This variable name made me laugh.

@dmcdougall

dmcdougall Dec 3, 2012

Member

This variable name made me laugh.

This comment has been minimized.

Show comment Hide comment
@dmcdougall

dmcdougall Dec 3, 2012

Member

I propose roflims for the next one...

@dmcdougall

dmcdougall Dec 3, 2012

Member

I propose roflims for the next one...

@dmcdougall

This comment has been minimized.

Show comment Hide comment
@dmcdougall

dmcdougall Dec 3, 2012

Member

This looks fine to me. +1. Will merge tomorrow.

Member

dmcdougall commented Dec 3, 2012

This looks fine to me. +1. Will merge tomorrow.

@pelson

This comment has been minimized.

Show comment Hide comment
@pelson

pelson Dec 3, 2012

Member

Agreed. 👍

Member

pelson commented Dec 3, 2012

Agreed. 👍

dmcdougall added a commit that referenced this pull request Dec 3, 2012

Merge pull request #1522 from NelleV/pep8_axes_3
PEP8-compliance on axes.py (patch 3 / 4)

@dmcdougall dmcdougall merged commit 73ceafa into matplotlib:master Dec 3, 2012

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment