Skip to content

Correct setup of margins for logarithmic plots #2263

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.

Sign up for GitHub

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

Closed
wants to merge 3 commits into from
Closed

Correct setup of margins for logarithmic plots #2263

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
fc086f5
auoscale correctly for log view
Jul 31, 2013
f835cdf
corected margin formula for log plots
Jul 31, 2013
ea430ee
removed prints for debug purposes
Jul 31, 2013
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 18 additions & 6 deletions lib/matplotlib/axes/_base.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1906,9 +1906,15 @@ def autoscale_view(self, tight=None, scalex=True, scaley=True):
x0, x1 = mtransforms.nonsingular(x0, x1, increasing=False,
expander=0.05)
if self._xmargin > 0:
delta = (x1 - x0) * self._xmargin
x0 -= delta
x1 += delta
xscale = self.get_xscale()
if xscale == 'log':
delta = (x1 / x0) ** self._xmargin
x0 /= delta
x1 *= delta
else:
delta = (x1 - x0) * self._xmargin
x0 -= delta
x1 += delta
if not _tight:
x0, x1 = xlocator.view_limits(x0, x1)
self.set_xbound(x0, x1)
Expand All @@ -1925,9 +1931,15 @@ def autoscale_view(self, tight=None, scalex=True, scaley=True):
y0, y1 = mtransforms.nonsingular(y0, y1, increasing=False,
expander=0.05)
if self._ymargin > 0:
delta = (y1 - y0) * self._ymargin
y0 -= delta
y1 += delta
yscale = self.get_yscale()
if yscale == 'log':
delta = (y1 / y0) ** self._ymargin
y0 /= delta
y1 *= delta
else:
delta = (y1 - y0) * self._ymargin
y0 -= delta
y1 += delta
if not _tight:
y0, y1 = ylocator.view_limits(y0, y1)
self.set_ybound(y0, y1)
Expand Down