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/demand_driven_deployment_inst.py
:Line 239:80: E501 line too long (89 > 79 characters) Line 242:80: E501 line too long (87 > 79 characters) Line 251:80: E501 line too long (108 > 79 characters) Line 262:80: E501 line too long (94 > 79 characters)
tests/integration_tests/installedcap_test.py
:Line 38:80: E501 line too long (106 > 79 characters) Line 39:80: E501 line too long (105 > 79 characters) Line 47:80: E501 line too long (94 > 79 characters) Line 52:80: E501 line too long (88 > 79 characters) Line 86:80: E501 line too long (85 > 79 characters) Line 94:80: E501 line too long (90 > 79 characters) Line 95:80: E501 line too long (86 > 79 characters) Line 109:80: E501 line too long (91 > 79 characters) Line 110:80: E501 line too long (88 > 79 characters)
I just noticed a bug, will close for now.
This PR addresses: https://github.com/arfc/d3ploy/issues/241. This PR gives d3ploy the capability to choose between using installed capacity or supply to deploy facilities.