INET-Complexity / isle

11 stars 20 forks source link

Margin of safety bug solved #42

Closed jsabuco closed 6 years ago

x0range commented 6 years ago

Looks good, but as a sidenote: margin_of_safety should accordingly be removed from the computation of necessary_liquidity (currently line 124 in riskmodel.py). Or, better yet, the entire expected_profits computation should be fixed or removed. This can be done in a future commit.