Skip to content

Commit 33c7c2d

Browse files
committed
Fix axes vlines and hlines using wrong coordinates
1 parent 5937318 commit 33c7c2d

File tree

3 files changed

+48
-12
lines changed

3 files changed

+48
-12
lines changed

lib/matplotlib/axes/_axes.py

+34-12
Original file line numberDiff line numberDiff line change
@@ -1101,14 +1101,25 @@ def hlines(self, y, xmin, xmax, colors=None, linestyles='solid',
11011101
# Extreme values of xmin/xmax/y. Using masked_verts here handles
11021102
# the case of y being a masked *object* array (as can be generated
11031103
# e.g. by errorbar()), which would make nanmin/nanmax stumble.
1104-
minx = np.nanmin(masked_verts[..., 0])
1105-
maxx = np.nanmax(masked_verts[..., 0])
1106-
miny = np.nanmin(masked_verts[..., 1])
1107-
maxy = np.nanmax(masked_verts[..., 1])
1104+
updatex = True
1105+
updatey = True
1106+
if self.name == "rectilinear":
1107+
datalim = lines.get_datalim(self.transData)
1108+
t = lines.get_transform()
1109+
updatex, updatey = t.contains_branch_seperately(self.transData)
1110+
minx = np.nanmin(datalim.xmin)
1111+
maxx = np.nanmax(datalim.xmax)
1112+
miny = np.nanmin(datalim.ymin)
1113+
maxy = np.nanmax(datalim.ymax)
1114+
else:
1115+
minx = np.nanmin(masked_verts[..., 0])
1116+
maxx = np.nanmax(masked_verts[..., 0])
1117+
miny = np.nanmin(masked_verts[..., 1])
1118+
maxy = np.nanmax(masked_verts[..., 1])
1119+
11081120
corners = (minx, miny), (maxx, maxy)
1109-
self.update_datalim(corners)
1121+
self.update_datalim(corners, updatex, updatey)
11101122
self._request_autoscale_view()
1111-
11121123
return lines
11131124

11141125
@_preprocess_data(replace_names=["x", "ymin", "ymax", "colors"],
@@ -1181,14 +1192,25 @@ def vlines(self, x, ymin, ymax, colors=None, linestyles='solid',
11811192
# Extreme values of x/ymin/ymax. Using masked_verts here handles
11821193
# the case of x being a masked *object* array (as can be generated
11831194
# e.g. by errorbar()), which would make nanmin/nanmax stumble.
1184-
minx = np.nanmin(masked_verts[..., 0])
1185-
maxx = np.nanmax(masked_verts[..., 0])
1186-
miny = np.nanmin(masked_verts[..., 1])
1187-
maxy = np.nanmax(masked_verts[..., 1])
1195+
updatex = True
1196+
updatey = True
1197+
if self.name == "rectilinear":
1198+
datalim = lines.get_datalim(self.transData)
1199+
t = lines.get_transform()
1200+
updatex, updatey = t.contains_branch_seperately(self.transData)
1201+
minx = np.nanmin(datalim.xmin)
1202+
maxx = np.nanmax(datalim.xmax)
1203+
miny = np.nanmin(datalim.ymin)
1204+
maxy = np.nanmax(datalim.ymax)
1205+
else:
1206+
minx = np.nanmin(masked_verts[..., 0])
1207+
maxx = np.nanmax(masked_verts[..., 0])
1208+
miny = np.nanmin(masked_verts[..., 1])
1209+
maxy = np.nanmax(masked_verts[..., 1])
1210+
11881211
corners = (minx, miny), (maxx, maxy)
1189-
self.update_datalim(corners)
1212+
self.update_datalim(corners, updatex, updatey)
11901213
self._request_autoscale_view()
1191-
11921214
return lines
11931215

11941216
@_preprocess_data(replace_names=["positions", "lineoffsets",

lib/matplotlib/tests/test_axes.py

+14
Original file line numberDiff line numberDiff line change
@@ -4920,6 +4920,20 @@ def test_lines_with_colors(fig_test, fig_ref, data):
49204920
colors=expect_color, linewidth=5)
49214921

49224922

4923+
@image_comparison(['vlines_hlines_blended_transform'],
4924+
extensions=['png'], style='mpl20')
4925+
def test_vlines_hlines_blended_transform():
4926+
t = np.arange(5.0, 10.0, 0.1)
4927+
s = np.exp(-t) + np.sin(2 * np.pi * t) + 10
4928+
fig, (hax, vax) = plt.subplots(2, 1, figsize=(6, 6))
4929+
hax.plot(t, s, '^')
4930+
hax.hlines([10, 9], xmin=0, xmax=0.5,
4931+
transform=hax.get_yaxis_transform(), colors='r')
4932+
vax.plot(t, s, '^')
4933+
vax.vlines([6, 7], ymin=0, ymax=0.15, transform=vax.get_xaxis_transform(),
4934+
colors='r')
4935+
4936+
49234937
@image_comparison(['step_linestyle', 'step_linestyle'], remove_text=True,
49244938
tol=0.2)
49254939
def test_step_linestyle():

0 commit comments

Comments
 (0)