Closed jdebacker closed 3 years ago
Merging #350 (75b27f4) into master (b39cfeb) will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## master #350 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 11 11
Lines 1104 1109 +5
=========================================
+ Hits 1104 1109 +5
Flag | Coverage Δ | |
---|---|---|
unittests | 100.00% <100.00%> (ø) |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
ccc/calcfunctions.py | 100.00% <100.00%> (ø) |
A cost of capital that is non-positive does not make sense (the result would be infinite profits through infinite investment). This PR adds a
RuntimeError
that is raised if the parameters used result in a non-positive cost of capital.cc @hdoupe -- this addresses a recent error seen in Compute Studio.