Make list of plots -> subplots

[sriki18]

Use Plotly's add_trace instead of Plotly express to plot the timeseries while converting it from a list of plots to one figure with subplots. Plots now share X axis, so zooming in one will also zoom in the others. Fixes #30

[sriki18]

@keenanjohnson not sure if you had a look!

[keenanjohnson]

I did see it but haven’t gotten a chance to take a look! I’ll get to it soon! Thanks for the contribution!

On Sat, Oct 9, 2021 at 6:44 PM Srikiran Chandrasekaran < @.***> wrote:

@keenanjohnson https://github.com/keenanjohnson not sure if you had a look!

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/Ribbit-Network/ribbit-network-dashboard/pull/52#issuecomment-939389871, or unsubscribe https://github.com/notifications/unsubscribe-auth/AATQ3FUTAXQLPAKGIWXNROTUGDVWZANCNFSM5FURFKVA .

[spestana]

Thanks @sriki18 !

What do you think about adding a second y-axis to the temperature and pressure plots, so we can see the measurements in different units?

So we would retain the current "scientific/metric" units on the left, then add another axis with alternate units on the right (old-fashioned US/Imperial units).

• Temperature: degrees C on the left, and degrees F on the right
• Pressure: mbar on the left, and "inches of Hg" on the right

(@keenanjohnson I can help review this PR if I had the permissions to assign myself as reviewer)

[keenanjohnson]

Oh sure thing @spestana ! I can add you! Seems like this PR has gone a bit stale if you want to help push it along :)

[sriki18]

I can do that, should it be a branch once this branch has been merged? @keenanjohnson @spestana

[keenanjohnson]

@sriki18 can you see my comment in the review previously about the read-ability?

[sriki18]

@spestana done. Have a look!