X Tutup
Skip to content

Remove misspelled and unused variable in GeoAxes class#6373

Merged
WeatherGod merged 1 commit intomatplotlib:masterfrom
lpsinger:remove_unused_logitude_degrees
May 6, 2016
Merged

Remove misspelled and unused variable in GeoAxes class#6373
WeatherGod merged 1 commit intomatplotlib:masterfrom
lpsinger:remove_unused_logitude_degrees

Conversation

@lpsinger
Copy link
Copy Markdown
Contributor

@lpsinger lpsinger commented May 5, 2016

The variable is used nowhere else.

@tacaswell tacaswell added this to the 2.0.1 (next bug fix release) milestone May 5, 2016
@lpsinger
Copy link
Copy Markdown
Contributor Author

lpsinger commented May 5, 2016

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

@WeatherGod
Copy link
Copy Markdown
Member

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
@WeatherGod
Copy link
Copy Markdown
Member

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 lpsinger deleted the remove_unused_logitude_degrees branch May 6, 2016 14:03
lpsinger added a commit to lpsinger/matplotlib that referenced this pull request May 6, 2016
@QuLogic QuLogic modified the milestones: 2.1 (next point release), 2.0.1 (next bug fix release) Aug 8, 2016
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.

5 participants

X Tutup