Remove misspelled and unused variable in GeoAxes class #6373

Merged
merged 1 commit into from May 6, 2016

Conversation

Projects
None yet
5 participants
Contributor

lpsinger commented May 5, 2016

The variable is used nowhere else.

@lpsinger lpsinger Remove misspelled and unused variable in GeoAxes class
7d58c34

mdboom added the needs_review label May 5, 2016

tacaswell added this to the 2.0.1 (next bug fix release) milestone May 5, 2016

Contributor

lpsinger commented May 5, 2016

Note that Travis build failures appear to be unrelated to the patch.

Member

WeatherGod commented May 6, 2016

What? Didn't you know you can have a log scale of degrees? :-P

@WeatherGod WeatherGod merged commit 8118b9b into matplotlib:master May 6, 2016

0 of 3 checks passed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
continuous-integration/appveyor/pr AppVeyor build failed
Details
coverage/coveralls Coverage pending from Coveralls.io
Details

mdboom removed the needs_review label May 6, 2016

Member

WeatherGod commented May 6, 2016

Actually, along a similar vein, self._latitude_degrees is also unused.

I think these were leftovers from before the days of the ThetaFormatter. The original coordinate data needed to be kept around separately from the display data.

lpsinger deleted the lpsinger:remove_unused_logitude_degrees branch May 6, 2016

@lpsinger lpsinger added a commit to lpsinger/matplotlib that referenced this pull request May 6, 2016

@lpsinger lpsinger Remove unused variable in GeoAxes class
See also #6373.
99a5189
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment