Skip to content

Commit 58db737

Browse files
KullyKully
authored
Merge pull request #623 from plotly/remove-scatterplotmatrix-labels
hide tick labels on boxplots in scatterplotmatrix
2 parents ca9dd35 + 21ac680 commit 58db737

File tree

2 files changed

+51
-40
lines changed

2 files changed

+51
-40
lines changed

‎plotly/tests/test_optional/test_figure_factory.py‎

Lines changed: 22 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -1141,29 +1141,23 @@ def test_scatter_plot_matrix(self):
11411141
columns=['Numbers', 'Fruit'])
11421142

11431143
test_scatter_plot_matrix = tls.FigureFactory.create_scatterplotmatrix(
1144-
df=df, diag='scatter', height=1000, width=1000, size=13,
1144+
df=df, diag='box', height=1000, width=1000, size=13,
11451145
title='Scatterplot Matrix'
11461146
)
11471147

11481148
exp_scatter_plot_matrix = {
1149-
'data': [{'marker': {'size': 13},
1150-
'mode': 'markers',
1149+
'data': [{'name': None,
11511150
'showlegend': False,
1152-
'type': 'scatter',
1153-
'x': [2, 6, -15, 5, -2, 0],
1151+
'type': 'box',
11541152
'xaxis': 'x1',
11551153
'y': [2, 6, -15, 5, -2, 0],
11561154
'yaxis': 'y1'},
11571155
{'marker': {'size': 13},
11581156
'mode': 'markers',
11591157
'showlegend': False,
11601158
'type': 'scatter',
1161-
'x': ['Apple',
1162-
'Pear',
1163-
'Apple',
1164-
'Pear',
1165-
'Apple',
1166-
'Apple'],
1159+
'x': ['Apple', 'Pear', 'Apple',
1160+
'Pear', 'Apple', 'Apple'],
11671161
'xaxis': 'x2',
11681162
'y': [2, 6, -15, 5, -2, 0],
11691163
'yaxis': 'y2'},
@@ -1173,44 +1167,40 @@ def test_scatter_plot_matrix(self):
11731167
'type': 'scatter',
11741168
'x': [2, 6, -15, 5, -2, 0],
11751169
'xaxis': 'x3',
1176-
'y': ['Apple',
1177-
'Pear',
1178-
'Apple',
1179-
'Pear',
1180-
'Apple',
1181-
'Apple'],
1170+
'y': ['Apple', 'Pear', 'Apple',
1171+
'Pear', 'Apple', 'Apple'],
11821172
'yaxis': 'y3'},
1183-
{'marker': {'size': 13},
1184-
'mode': 'markers',
1173+
{'name': None,
11851174
'showlegend': False,
1186-
'type': 'scatter',
1187-
'x': ['Apple',
1188-
'Pear',
1189-
'Apple',
1190-
'Pear',
1191-
'Apple',
1192-
'Apple'],
1175+
'type': 'box',
11931176
'xaxis': 'x4',
1194-
'y': ['Apple', 'Pear', 'Apple', 'Pear', 'Apple', 'Apple'],
1177+
'y': ['Apple', 'Pear', 'Apple',
1178+
'Pear', 'Apple', 'Apple'],
11951179
'yaxis': 'y4'}],
11961180
'layout': {'height': 1000,
11971181
'showlegend': True,
11981182
'title': 'Scatterplot Matrix',
11991183
'width': 1000,
12001184
'xaxis1': {'anchor': 'y1',
1201-
'domain': [0.0, 0.45]},
1185+
'domain': [0.0, 0.45],
1186+
'showticklabels': False},
12021187
'xaxis2': {'anchor': 'y2',
12031188
'domain': [0.55, 1.0]},
12041189
'xaxis3': {'anchor': 'y3',
1205-
'domain': [0.0, 0.45], 'title': 'Numbers'},
1190+
'domain': [0.0, 0.45],
1191+
'title': 'Numbers'},
12061192
'xaxis4': {'anchor': 'y4',
1207-
'domain': [0.55, 1.0], 'title': 'Fruit'},
1193+
'domain': [0.55, 1.0],
1194+
'showticklabels': False,
1195+
'title': 'Fruit'},
12081196
'yaxis1': {'anchor': 'x1',
1209-
'domain': [0.575, 1.0], 'title': 'Numbers'},
1197+
'domain': [0.575, 1.0],
1198+
'title': 'Numbers'},
12101199
'yaxis2': {'anchor': 'x2',
12111200
'domain': [0.575, 1.0]},
12121201
'yaxis3': {'anchor': 'x3',
1213-
'domain': [0.0, 0.425], 'title': 'Fruit'},
1202+
'domain': [0.0, 0.425],
1203+
'title': 'Fruit'},
12141204
'yaxis4': {'anchor': 'x4',
12151205
'domain': [0.0, 0.425]}}
12161206
}

‎plotly/tools.py‎

Lines changed: 29 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3881,6 +3881,9 @@ def _scatterplot(dataframe, headers, diag, size,
38813881
title=title,
38823882
showlegend=True
38833883
)
3884+
3885+
FigureFactory._hide_tick_labels_from_box_subplots(fig)
3886+
38843887
return fig
38853888

38863889
@staticmethod
@@ -4028,6 +4031,8 @@ def _scatterplot_dict(dataframe, headers, diag, size,
40284031
yaxis_key = 'yaxis{}'.format(1 + (dim * j))
40294032
fig['layout'][yaxis_key].update(title=headers[j])
40304033

4034+
FigureFactory._hide_tick_labels_from_box_subplots(fig)
4035+
40314036
if diag == 'histogram':
40324037
fig['layout'].update(
40334038
height=height, width=width,
@@ -4036,13 +4041,6 @@ def _scatterplot_dict(dataframe, headers, diag, size,
40364041
barmode='stack')
40374042
return fig
40384043

4039-
elif diag == 'box':
4040-
fig['layout'].update(
4041-
height=height, width=width,
4042-
title=title,
4043-
showlegend=True)
4044-
return fig
4045-
40464044
else:
40474045
fig['layout'].update(
40484046
height=height, width=width,
@@ -4218,6 +4216,8 @@ def _scatterplot_theme(dataframe, headers, diag, size, height,
42184216
yaxis_key = 'yaxis{}'.format(1 + (dim * j))
42194217
fig['layout'][yaxis_key].update(title=headers[j])
42204218

4219+
FigureFactory._hide_tick_labels_from_box_subplots(fig)
4220+
42214221
if diag == 'histogram':
42224222
fig['layout'].update(
42234223
height=height, width=width,
@@ -4391,6 +4391,8 @@ def _scatterplot_theme(dataframe, headers, diag, size, height,
43914391
yaxis_key = 'yaxis{}'.format(1 + (dim * j))
43924392
fig['layout'][yaxis_key].update(title=headers[j])
43934393

4394+
FigureFactory._hide_tick_labels_from_box_subplots(fig)
4395+
43944396
if diag == 'histogram':
43954397
fig['layout'].update(
43964398
height=height, width=width,
@@ -4536,6 +4538,8 @@ def _scatterplot_theme(dataframe, headers, diag, size, height,
45364538
yaxis_key = 'yaxis{}'.format(1 + (dim * j))
45374539
fig['layout'][yaxis_key].update(title=headers[j])
45384540

4541+
FigureFactory._hide_tick_labels_from_box_subplots(fig)
4542+
45394543
if diag == 'histogram':
45404544
fig['layout'].update(
45414545
height=height, width=width,
@@ -4558,6 +4562,22 @@ def _scatterplot_theme(dataframe, headers, diag, size, height,
45584562
showlegend=True)
45594563
return fig
45604564

4565+
@staticmethod
4566+
def _hide_tick_labels_from_box_subplots(fig):
4567+
"""
4568+
Hides tick labels for box plots in scatterplotmatrix subplots.
4569+
"""
4570+
boxplot_xaxes = []
4571+
for trace in fig['data']:
4572+
if trace['type'] == 'box':
4573+
# stores the xaxes which correspond to boxplot subplots
4574+
# since we use xaxis1, xaxis2, etc, in plotly.py
4575+
boxplot_xaxes.append(
4576+
'xaxis{}'.format(trace['xaxis'][-1])
4577+
)
4578+
for xaxis in boxplot_xaxes:
4579+
fig['layout'][xaxis]['showticklabels'] = False
4580+
45614581
@staticmethod
45624582
def _validate_index(index_vals):
45634583
"""
@@ -4806,7 +4826,8 @@ def create_scatterplotmatrix(df, index=None, endpts=None, diag='scatter',
48064826
that defines intervals on the real line. They are used to group
48074827
the entries in an index of numbers into their corresponding
48084828
interval and therefore can be treated as categorical data
4809-
:param (str) diag: sets the chart type for the main diagonal plots
4829+
:param (str) diag: sets the chart type for the main diagonal plots.
4830+
The options are 'scatter', 'histogram' and 'box'.
48104831
:param (int|float) height: sets the height of the chart
48114832
:param (int|float) width: sets the width of the chart
48124833
:param (float) size: sets the marker size (in px)

0 commit comments

Comments
 (0)