New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add ability to fill between lines #109
Conversation
pyam/plotting.py
Outdated
@@ -640,6 +646,9 @@ def line_plot(df, x='year', y='value', ax=None, legend=None, title=True, | |||
props = assign_style_props(df, color=color, marker=marker, | |||
linestyle=linestyle, cmap=cmap) | |||
|
|||
if fill_between and not 'color' in props: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
E713 test for membership should be 'not in'
pyam/plotting.py
Outdated
dictionary, those arguments will be provided instead of defaults. | ||
default: None | ||
final_ranges : boolean or dict, optional | ||
Add vertical line between minima/maxima of the 'color' argument in the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
W291 trailing whitespace
This is good to go pending CI, @danielhuppmann |
ok, now actually g2g @danielhuppmann |
hold on this, testing with the SR1.5 dataset |
Ok, found some other issues which are now fixed and tested. Finally ready, @danielhuppmann |
what can I say other than - amazing! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
found one print()
statement in a unit test
tests/test_plotting.py
Outdated
newdata = ['test_model1', 'test_scenario1', 'World', 'Primary Energy|Coal', | ||
'EJ/y', 2015, 3.50] | ||
df.data.loc[len(df.data) + 1] = newdata | ||
print(df.data) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please remove print
apology accepted - very glad that this doesn't only happens to me... |
Please confirm that this PR has done the following:
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:
Description of PR
Please describe the changes introduced by this PR.