Sankey5

[kdavies4]

This change turns out to be mostly cosmetic--better line-wrapping and plot margins in a demo. It does however fix one problem where the string labels were being appended on successful calls to add subdiagrams.

[dmcdougall]

@kdavies4 You need to rebase against current upstream/master, not origin/master. origin is the remote that points to your github fork. upstream points to the main codebase.

git fetch upstream
git checkout sankey5
git rebase upstream/master

[dmcdougall]

Most of these changes address PEP8 issues. Commit f1f949b is the only one the contains non-PEP8 changes.

[kdavies4]

I think I made a mess of my local copy when I was trying to rebase. Can you work with what is here?

[dmcdougall]

I feel #1574 supersedes this. As a result I'm closing this ticket. @kdavies4 If I am mistaken, please feel free to reopen.

[kdavies4]

Yes, it should be closed.