Skip to content
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.

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

Fix basic_units.py on Python 3. PEP8 and PyLint cleanup. #980

Merged
merged 2 commits into from Jul 17, 2012
Merged

Fix basic_units.py on Python 3. PEP8 and PyLint cleanup. #980

merged 2 commits into from Jul 17, 2012

Conversation

cgohlke
Copy link
Contributor

@cgohlke cgohlke commented Jun 30, 2012

No description provided.

@@ -235,21 +246,23 @@ def get_conversion_fn(self, unit):
return self.conversions[unit]

def convert_value_to(self, value, unit):
#print 'convert value to: value ="%s", unit="%s"'%(value, type(unit)), self.conversions
#print 'convert value to: value ="%s", unit="%s"'%(value, type(unit)),
#self.conversions
Copy link
Member

Choose a reason for hiding this comment

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

This changes the meaning if the print statement is uncommented. Could just remove the print statement...

@pelson
Copy link
Member

pelson commented Jun 30, 2012

+1

@pelson
Copy link
Member

pelson commented Jul 17, 2012

Thanks @cgohlke we share a common interest in being picky about whitespace/trying to make PyLint happy :-)

pelson added a commit that referenced this pull request Jul 17, 2012
Fix basic_units.py on Python 3. PEP8 and PyLint cleanup.
@pelson pelson merged commit 1418e4a into matplotlib:master Jul 17, 2012
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.

None yet

2 participants