Stack plot bug part 1 issue 266

[znicholls]

Please confirm that this PR has done the following:

• Tests Added
• Documentation Added (N/A)
• Description in RELEASE_NOTES.md Added

Adding to RELEASE_NOTES.md (remove section after adding to RELEASE_NOTES.md)

Please add a single line in the release notes similar to the following:

- (#XX)[http://link-to-pr.com] Added feature which does something

Description of PR

Addresses the first problem identified in #266

[znicholls]

This fixes the first part of #266. The wrong image can be seen here with the correction available here. Do we want this added to the tutorial too or not worry about it as this is a bug fix?

[coveralls]

Coverage decreased (-0.3%) to 87.873% when pulling f2260e4 on znicholls:stack-plot-bug-part-1-issue-266 into 1d0d5d2 on IAMconsortium:master.

[danielhuppmann]

thanks @znicholls! I don't think that this needs to go into the tutorial, as it just fixes unexpected behaviour...

Can you rebase, then I'll merge?

[znicholls]

Can you rebase, then I'll merge?

cool done thanks @danielhuppmann !

[danielhuppmann]

It seems that appveyor upgraded to a new version of matplotlib, causing this error. I think I solved a similar problem by downgrading.

@gidden, does it make sense to do that here as a quickfix, too, or is it worthwhile to make the underlying use of matplotlib-internals more robust?

[gidden]

Took a quick look at the logs, specifically conda list.

Last success:

...
markupsafe                1.1.1            py37he774522_0  
matplotlib                3.0.3            py37hc8f65d3_0  
mistune                   0.8.4            py37he774522_0  
...

This PR:

...
matplotlib                3.0.3            py37hc8f65d3_0  
matplotlib-base           3.1.1            py37h2852a4a_1    conda-forge
mistune                   0.8.4            py37he774522_0
...

Let me do a quick test in #274

[gidden]

ok, looks like #274 has a fix. mind incorporating it?

[znicholls]

mind incorporating it?

yep done

[danielhuppmann]

thanks @znicholls!