Fix for issue#1002.

[rysktky]

subplot_titles position issue when shared_yaxes and/or shared_xaxes is True and start_cell is bottom-left.

Tested with this code.

from plotly import tools
import plotly.offline as py
import plotly.graph_objs as go

trace1 = go.Scatter( x=[1, 2], y=[10,10,10])
trace2 = go.Scatter( x=[1, 2], y=[20,20,20])
trace3 = go.Scatter( x=[1, 2, 3], y=[30,30,30])
trace4 = go.Scatter( x=[1, 2, 3], y=[40,40,40])
trace5 = go.Scatter( x=[1, 2, 3], y=[50,50,50])
trace6 = go.Scatter( x=[1, 2, 3], y=[60,60,60])
trace7 = go.Scatter( x=[1, 2, 3], y=[70,70,70])

fig = tools.make_subplots(rows=7, cols=1, 
                          shared_xaxes=True,
                          start_cell='bottom-left',
                          vertical_spacing=0.03,
                          subplot_titles=('trace1','trace2','trace3','trace4','trace5','trace6','trace7'))

fig.append_trace(trace1, 1, 1)
fig.append_trace(trace2, 2, 1)
fig.append_trace(trace3, 3, 1)
fig.append_trace(trace4, 4, 1)
fig.append_trace(trace5, 5, 1)
fig.append_trace(trace6, 6, 1)
fig.append_trace(trace7, 7, 1)

fig['layout'].update(showlegend=False, title='Test for issue 1002')
py.plot(fig, filename='Test4Issue1002.html')

Result with current code.

Result with the fix. subplot_titles are positioned and ordered correctly.

[jonmmease]

Hi @rysktky , thanks for the PR. If you're still interested, could you do the following:

1. Merge in master and then push again so that our new test suite runs.
2. Add a test or two to plotly/tests/test_core/test_tools/test_make_subplots.py.

Thanks!

[jonmmease]

This issue will be fixed in version 4 by #1528.

You can try it out with version 3.8+ using the v4_subplot future flag.

from _plotly_future_ import v4_subplots
from plotly import tools
import plotly.offline as py
import plotly.graph_objs as go

trace1 = go.Scatter( x=[1, 2], y=[10,10,10])
trace2 = go.Scatter( x=[1, 2], y=[20,20,20])
trace3 = go.Scatter( x=[1, 2, 3], y=[30,30,30])
trace4 = go.Scatter( x=[1, 2, 3], y=[40,40,40])
trace5 = go.Scatter( x=[1, 2, 3], y=[50,50,50])
trace6 = go.Scatter( x=[1, 2, 3], y=[60,60,60])
trace7 = go.Scatter( x=[1, 2, 3], y=[70,70,70])

fig = tools.make_subplots(rows=7, cols=1, 
                          shared_xaxes=True,
                          start_cell='bottom-left',
                          vertical_spacing=0.03,
                          subplot_titles=('trace1','trace2','trace3','trace4','trace5','trace6','trace7'))

fig.append_trace(trace1, 1, 1)
fig.append_trace(trace2, 2, 1)
fig.append_trace(trace3, 3, 1)
fig.append_trace(trace4, 4, 1)
fig.append_trace(trace5, 5, 1)
fig.append_trace(trace6, 6, 1)
fig.append_trace(trace7, 7, 1)

fig['layout'].update(showlegend=False, title='Test for issue 1002')
py.plot(fig, filename='Test4Issue1002.html')

Thanks for getting this started @rysktky!