Plume plot

[znicholls]

Pull request

Please confirm that this pull request has done the following:

• Tests added
• Documentation added (where applicable)
• Example added (either to an existing notebook or as a new notebook, where applicable)
• Description in CHANGELOG.rst added

Adding to CHANGELOG.rst

Please add a single line in the changelog notes similar to one of the following:

- (`#XX <https://github.com/openscm/scmdata/pull/XX>`_) Added feature which does something
- (`#XX <https://github.com/openscm/scmdata/pull/XX>`_) Fixed bug identified in (`#YY <https://github.com/openscm/scmdata/issues/YY>`_)

[review-notebook-app]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[codecov]

Codecov Report

Merging #139 (565dfd3) into master (6012b2b) will increase coverage by 0.01%.
The diff coverage is 94.11%.

@@            Coverage Diff             @@
##           master     #139      +/-   ##
==========================================
+ Coverage   93.77%   93.78%   +0.01%     
==========================================
  Files          15       15              
  Lines        1527     1610      +83     
  Branches      303      326      +23     
==========================================
+ Hits         1432     1510      +78     
- Misses         77       79       +2     
- Partials       18       21       +3     

Impacted Files	Coverage Δ
src/scmdata/plotting.py	94.56% <93.90%> (-5.44%)	⬇️
src/scmdata/filters.py	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6012b2b...565dfd3. Read the comment docs.

[lewisjared]

Adding a test which runs (but doesn't necessarily check the resulting plot) might be useful if the seaborn/matplotlib api's change

[lewisjared]

I was playing with this yesterday, but I had trouble getting it to work. It assumes that you have already calculated the quantiles, no? Can you add an example?

[znicholls]

It assumes that you have already calculated the quantiles, no

Yep

Can you add an example

On top of the notebook?

[lewisjared]

Great improvement. That fixed a few things that confused me when I had a play with plumeplot