Fix x axis - Githubissues

timandrews335 commented 5 years ago

Updated pull request to add small multiple plot option in the top row, correcting x-axis issue.

tcassou commented 5 years ago

Hi @timandrews335

Sorry it took me a while to get back to you. What I had in mind about sharing the same x axis was to stack all subplots vertically, which

allows to share the same x axis and vertical dotted line (date of intervention)
scales a bit better to an arbitrary number of input columns (all graphs have the same dimensions).

I pushed the following commit: 2d615db30be9ceb7f47099c77addb5f897f080f0

Visually, in case of a split chart this gives something like:

I'll release the change in a new version shortly, thanks again for your suggestion!

tcassou commented 5 years ago

I have just released 1.2.1 with the option to split charts, simply call

ci.plot(split=True)

Let me know if you observe any unexpected behaviours or have further ideas for improvements, for now I'll close this PR. Thanks again!

tcassou / causal_impact