Closed gwenchee closed 5 years ago
Hello @gwenchee! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
d3ploy/plotter.py
:Line 51:80: E501 line too long (80 > 79 characters) Line 93:80: E501 line too long (80 > 79 characters) Line 138:80: E501 line too long (84 > 79 characters) Line 151:80: E501 line too long (81 > 79 characters)
d3ploy/tester.py
:Line 308:80: E501 line too long (91 > 79 characters) Line 327:80: E501 line too long (84 > 79 characters)
looks good
In this PR: 1) Make calculated demand red dots see through for ease of understanding the plots 2) addresses https://github.com/arfc/d3ploy/issues/195, with fixes suggested in https://github.com/arfc/d3ploy/pull/214