55 convert mutable default to field with default factory (#56)

* remove test coverage display * remove test coverage display --------- Co-authored-by: Thomas Schmelzer <[email protected]>
cvxgrp · May 7, 2023 · fbf8466 · fbf8466
1 parent d46d90b
commit fbf8466
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 7 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -11,7 +11,7 @@ jobs:
     strategy:
       matrix:
         os: [ ubuntu-latest, windows-latest, macos-latest ]
-        python-version: [ '3.10' ]
+        python-version: [ '3.10', '3.11' ]
 
     # Steps represent a sequence of tasks that will be executed as part of the job
     steps:

diff --git a/book/docs/reports.md b/book/docs/reports.md
@@ -4,10 +4,6 @@
 
 {{ '[Lines of code]({url}/artifacts/lines/lines.txt)'.format(url=book_url) }}
 
-## Coverage
-
-{{ '[Test coverage]({url}/artifacts/tests/html-coverage/index.html)'.format(url=book_url) }}
-
 ## Test report
 
 {{ '[Report]({url}/artifacts/tests/html-report/report.html)'.format(url=book_url) }}
diff --git a/cvx/simulator/portfolio.py b/cvx/simulator/portfolio.py
@@ -1,4 +1,4 @@
-from dataclasses import dataclass
+from dataclasses import dataclass, field
 import pandas as pd
 
 from cvx.simulator.trading_costs import LinearCostModel, TradingCostModel
@@ -59,7 +59,7 @@ class _EquityPortfolio:
     stocks: pd.DataFrame
     trading_cost_model: TradingCostModel
     initial_cash: float = 1e6
-    _state: _State = _State()
+    _state: _State = field(default_factory=_State)
 
     def __post_init__(self):
         self._state.position = self.stocks.loc[self.index[0]]