wbportfolio 1.54.9__py2.py3-none-any.whl → 1.54.10__py2.py3-none-any.whl

wbportfolio/import_export/handlers/trade.py

@@ -88,6 +88,8 @@ class TradeImportHandler(ImportExportHandler):
                     1 / (math.pow(10, 4))
                 )  # we need that convertion mechanism otherwise there is floating point approximation error while casting to decimal and get_instance does not work as expected
                 data[field.name] = Decimal(value).quantize(Decimal(str(q)))
+        if (target_weight := data.pop("target_weight", None)) is not None:
+            data["_target_weight"] = target_weight
 
     def _create_instance(self, data: Dict[str, Any], **kwargs) -> models.Model:
         if "transaction_date" not in data:  # we might get only book date and not transaction date

wbportfolio/models/portfolio.py

@@ -54,6 +54,8 @@ logger = logging.getLogger("pms")
 if TYPE_CHECKING:
     from wbportfolio.models.transactions.trade_proposals import TradeProposal
 
+MARKET_HOLIDAY_MAX_DURATION = 15
+
 
 def get_returns(
     instrument_ids: list[int],
@@ -105,6 +107,7 @@ def get_returns(
 
     prices_df = df["close"]
     fx_rate_df = df["fx_rate"]
+
     returns = prices_to_returns(fx_rate_df * prices_df, drop_inceptions_nan=False, fill_nan=ffill_returns)
     return {ts.date(): row for ts, row in prices_df.replace(np.nan, None).to_dict("index").items()}, returns.replace(
         [np.inf, -np.inf, np.nan], 0
@@ -400,7 +403,11 @@ class Portfolio(DeleteToDisableMixin, WBModel):
             weights = self.get_weights(val_date)
         return_date = (val_date + BDay(1)).date()
         _, returns = get_returns(
-            list(weights.keys()), (val_date - BDay(2)).date(), return_date, to_currency=self.currency, **kwargs
+            list(weights.keys()),
+            (val_date - BDay(MARKET_HOLIDAY_MAX_DURATION)).date(),
+            return_date,
+            to_currency=self.currency,
+            **kwargs,
         )
         if pd.Timestamp(return_date) not in returns.index:
             raise InvalidAnalyticPortfolio()
@@ -822,7 +829,7 @@ class Portfolio(DeleteToDisableMixin, WBModel):
 
         prices, returns = get_returns(
             instrument_ids,
-            (start_date - BDay(3)).date(),
+            (start_date - BDay(MARKET_HOLIDAY_MAX_DURATION)).date(),
             end_date,
             to_currency=self.currency,
             ffill_returns=True,

wbportfolio/models/transactions/trade_proposals.py

@@ -397,7 +397,7 @@ class TradeProposal(CloneMixin, RiskCheckMixin, WBModel):
             if approve_automatically and self.portfolio.can_be_rebalanced:
                 self.approve(replay=False, broadcast_changes_at_date=broadcast_changes_at_date)
 
-    def replay(self, force_reset_trade: bool = False, broadcast_changes_at_date: bool = True):
+    def replay(self, broadcast_changes_at_date: bool = True):
         last_trade_proposal = self
         last_trade_proposal_created = False
         while last_trade_proposal and last_trade_proposal.status == TradeProposal.Status.APPROVED:
@@ -405,7 +405,7 @@ class TradeProposal(CloneMixin, RiskCheckMixin, WBModel):
                 logger.info(f"Replaying trade proposal {last_trade_proposal}")
                 last_trade_proposal.approve_workflow(
                     silent_exception=True,
-                    force_reset_trade=force_reset_trade,
+                    force_reset_trade=True,
                     broadcast_changes_at_date=broadcast_changes_at_date,
                 )
                 last_trade_proposal.save()
@@ -803,9 +803,8 @@ class TradeProposal(CloneMixin, RiskCheckMixin, WBModel):
         )  # we delete the existing portfolio as it has been reverted
         for trade in self.trades.all():
             trade.status = Trade.Status.DRAFT
-            trade.drift_factor = Decimal("1")
             trades.append(trade)
-        Trade.objects.bulk_update(trades, ["status", "drift_factor"])
+        Trade.objects.bulk_update(trades, ["status"])
 
     def can_revert(self):
         errors = dict()

wbportfolio/models/transactions/trades.py

@@ -439,7 +439,7 @@ class Trade(TransactionMixin, ImportMixin, OrderedModel, models.Model):
                 method=RequestType.PATCH,
                 identifiers=("wbportfolio:trade",),
                 icon=WBIcon.UNDO.icon,
-                key="reverte",
+                key="revert",
                 label="Revert",
                 action_label="revert",
                 # description_fields="<p>Start: {{start}}</p><p>End: {{end}}</p><p>Title: {{title}}</p>",
@@ -518,6 +518,11 @@ class Trade(TransactionMixin, ImportMixin, OrderedModel, models.Model):
             self, "target_weight", round(self._effective_weight + self.weighting, self.TRADE_WEIGHTING_PRECISION)
         )
 
+    @_target_weight.setter
+    def _target_weight(self, target_weight):
+        self.weighting = Decimal(target_weight) - self._effective_weight
+        self._set_type()
+
     @property
     @admin.display(description="Target Shares")
     def _target_shares(self) -> Decimal:
@@ -558,12 +563,6 @@ class Trade(TransactionMixin, ImportMixin, OrderedModel, models.Model):
         ]
         # notification_email_template = "portfolio/email/trade_notification.html"
 
-    def __init__(self, *args, target_weight: Decimal | None = None, **kwargs):
-        super().__init__(*args, **kwargs)
-        if target_weight is not None:  # if target weight is provided, we guess the corresponding weighting
-            self.weighting = Decimal(target_weight) - self._effective_weight
-            self._set_type()
-
     def save(self, *args, **kwargs):
         if abs(self.weighting) < 10e-6:
             self.weighting = Decimal("0")

wbportfolio/viewsets/transactions/trades.py

@@ -445,14 +445,20 @@ class TradeTradeProposalModelViewSet(
 
             agg = {
                 "effective_weight": {
-                    "Cash": format_number(cash_sum_effective_weight, decimal=6),
-                    "Non-Cash": format_number(noncash_sum_effective_weight, decimal=6),
-                    "Total": format_number(noncash_sum_effective_weight + cash_sum_effective_weight, decimal=6),
+                    "Cash": format_number(cash_sum_effective_weight, decimal=Trade.TRADE_WEIGHTING_PRECISION),
+                    "Non-Cash": format_number(noncash_sum_effective_weight, decimal=Trade.TRADE_WEIGHTING_PRECISION),
+                    "Total": format_number(
+                        noncash_sum_effective_weight + cash_sum_effective_weight,
+                        decimal=Trade.TRADE_WEIGHTING_PRECISION,
+                    ),
                 },
                 "target_weight": {
-                    "Cash": format_number(cash_sum_target_cash_weight, decimal=6),
-                    "Non-Cash": format_number(noncash_sum_target_weight, decimal=6),
-                    "Total": format_number(cash_sum_target_cash_weight + noncash_sum_target_weight, decimal=6),
+                    "Cash": format_number(cash_sum_target_cash_weight, decimal=Trade.TRADE_WEIGHTING_PRECISION),
+                    "Non-Cash": format_number(noncash_sum_target_weight, decimal=Trade.TRADE_WEIGHTING_PRECISION),
+                    "Total": format_number(
+                        cash_sum_target_cash_weight + noncash_sum_target_weight,
+                        decimal=Trade.TRADE_WEIGHTING_PRECISION,
+                    ),
                 },
                 "effective_total_value_fx_portfolio": {
                     "Cash": format_number(cash_sum_effective_total_value_fx_portfolio, decimal=6),
@@ -471,9 +477,12 @@ class TradeTradeProposalModelViewSet(
                     ),
                 },
                 "weighting": {
-                    "Cash Flow": format_number(cash_sum_target_cash_weight - cash_sum_effective_weight, decimal=6),
-                    "Buy": format_number(sum_buy_weight, decimal=6),
-                    "Sell": format_number(sum_sell_weight, decimal=6),
+                    "Cash Flow": format_number(
+                        cash_sum_target_cash_weight - cash_sum_effective_weight,
+                        decimal=Trade.TRADE_WEIGHTING_PRECISION,
+                    ),
+                    "Buy": format_number(sum_buy_weight, decimal=Trade.TRADE_WEIGHTING_PRECISION),
+                    "Sell": format_number(sum_sell_weight, decimal=Trade.TRADE_WEIGHTING_PRECISION),
                 },
                 "total_value_fx_portfolio": {
                     "Cash Flow": format_number(

{wbportfolio-1.54.9.dist-info → wbportfolio-1.54.10.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: wbportfolio
-Version: 1.54.9
+Version: 1.54.10
 Author-email: Christopher Wittlinger <c.wittlinger@stainly.com>
 License-File: LICENSE
 Requires-Dist: cryptography==3.4.*

{wbportfolio-1.54.9.dist-info → wbportfolio-1.54.10.dist-info}/RECORD

@@ -113,7 +113,7 @@ wbportfolio/import_export/handlers/dividend.py,sha256=F0oLfNt2B_QQAjHBCRpxa5HSkf
 wbportfolio/import_export/handlers/fees.py,sha256=BOFHAvSTlvVLaxnm6KD_fcza1TlPc02HOR9J0_jjswI,2495
 wbportfolio/import_export/handlers/portfolio_cash_flow.py,sha256=W7QPNqEvvsq0RS016EAFBp1ezvc6G9Rk-hviRZh8o6Y,2737
 wbportfolio/import_export/handlers/register.py,sha256=sYyXkE8b1DPZ5monxylZn0kjxLVdNYYZR-p61dwEoDM,2271
-wbportfolio/import_export/handlers/trade.py,sha256=oLYNEAFyDeInvHjuAbB4pe60q6VmHX0wGcyuEGofbKM,12700
+wbportfolio/import_export/handlers/trade.py,sha256=_-P1ImDX6jfObm1WKiLtzz7RXEnXjtARAw7cxoHReCM,12826
 wbportfolio/import_export/parsers/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 wbportfolio/import_export/parsers/default_mapping.py,sha256=KrO-X5CvQCeQoBYzFDxavoQGriyUSeI2QDx5ar_zo7A,1405
 wbportfolio/import_export/parsers/jpmorgan/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -257,7 +257,7 @@ wbportfolio/models/asset.py,sha256=b0vPt4LwNrxcMiK7UmBKViYnbNNlZzPTagvU5vFuyrc,4
 wbportfolio/models/custodians.py,sha256=owTiS2Vm5CRKzh9M_P9GOVg-s-ndQ9UvRmw3yZP7cw0,3815
 wbportfolio/models/exceptions.py,sha256=3ix0tWUO-O6jpz8f07XIwycw2x3JFRoWzjwil8FVA2Q,52
 wbportfolio/models/indexes.py,sha256=gvW4K9U9Bj8BmVCqFYdWiXvDWhjHINRON8XhNsZUiQY,639
-wbportfolio/models/portfolio.py,sha256=9gbT3tuo73SfhBn4_HIes71feyjTXnZnn9lX2KL_1pQ,57861
+wbportfolio/models/portfolio.py,sha256=2m35YgEsF1gkt_JN5O4UlOtNLAqSfKi8JUp0G-1HPmQ,57997
 wbportfolio/models/portfolio_cash_flow.py,sha256=uElG7IJUBY8qvtrXftOoskX6EA-dKgEG1JJdvHeWV7g,7336
 wbportfolio/models/portfolio_cash_targets.py,sha256=WmgG-etPisZsh2yaFQpz7EkpvAudKBEzqPsO715w52U,1498
 wbportfolio/models/portfolio_relationship.py,sha256=ZGECiPZiLdlk4uSamOrEfuzO0hduK6OMKJLUSnh5_kc,5190
@@ -284,8 +284,8 @@ wbportfolio/models/transactions/claim.py,sha256=SF2FlwG6SRVmA_hT0NbXah5-fYejccWK
 wbportfolio/models/transactions/dividends.py,sha256=mmOdGWR35yndUMoCuG24Y6BdtxDhSk2gMQ-8LVguqzg,1890
 wbportfolio/models/transactions/fees.py,sha256=wJtlzbBCAq1UHvv0wqWTE2BEjCF5RMtoaSDS3kODFRo,7112
 wbportfolio/models/transactions/rebalancing.py,sha256=rwePcmTZOYgfSWnBQcBrZ3DQHRJ3w17hdO_hgrRbbhI,7696
-wbportfolio/models/transactions/trade_proposals.py,sha256=2EGi03APBC9QWU99LVRA1e-_-o8AUtEW332Iuxvb-_E,38214
-wbportfolio/models/transactions/trades.py,sha256=vTCoSBAmd6FPFyDRm5fqTqbxwzcH3ApWn0Dz2tkCCR8,34132
+wbportfolio/models/transactions/trade_proposals.py,sha256=UYhV2pjQE4V-k0tVTBeZgPPK0C-tBeY345iroDzulfg,38106
+wbportfolio/models/transactions/trades.py,sha256=oAUWCPPdBcMf5bcYI6BwvnrFKK588ZLRHPyPr65h-0U,33966
 wbportfolio/models/transactions/transactions.py,sha256=XTcUeMUfkf5XTSZaR2UAyGqCVkOhQYk03_vzHLIgf8Q,3807
 wbportfolio/pms/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 wbportfolio/pms/typing.py,sha256=BV4dzazNHdfpfLV99bLVyYGcETmbQSnFV6ipc4fNKfg,8470
@@ -521,8 +521,8 @@ wbportfolio/viewsets/transactions/fees.py,sha256=WT2bWWfgozz4_rpyTKX7dgBBTXD-gu0
 wbportfolio/viewsets/transactions/mixins.py,sha256=WipvJoi5hylkpD0y9VATe30WAcwIHUIroVkK10FYw7k,636
 wbportfolio/viewsets/transactions/rebalancing.py,sha256=6rIrdK0rtKL1afJ-tYfAGdQVTN2MH1kG_yCeVkmyK8k,1263
 wbportfolio/viewsets/transactions/trade_proposals.py,sha256=kQCojTNKBEyn2NcenL3a9auzBH4sIgLEx8rLAYCGLGg,6161
-wbportfolio/viewsets/transactions/trades.py,sha256=GHOw5jtcqoaHiRrxxxL29c9405QiPisEn4coGELKDrE,22146
-wbportfolio-1.54.9.dist-info/METADATA,sha256=SL5iDKkrof3J3-X4ykL9jJslbUo8ltmEwJzni_emoXY,702
-wbportfolio-1.54.9.dist-info/WHEEL,sha256=tkmg4JIqwd9H8mL30xA7crRmoStyCtGp0VWshokd1Jc,105
-wbportfolio-1.54.9.dist-info/licenses/LICENSE,sha256=jvfVH0SY8_YMHlsJHKe_OajiscQDz4lpTlqT6x24sVw,172
-wbportfolio-1.54.9.dist-info/RECORD,,
+wbportfolio/viewsets/transactions/trades.py,sha256=Y8v2cM0vpspHysaAvu8qqhzt86dNtb2Q3puo4HCJsTI,22629
+wbportfolio-1.54.10.dist-info/METADATA,sha256=uKqFJps-zs7857cq_8X_JmogU7gB5LG3RaZORctTI-k,703
+wbportfolio-1.54.10.dist-info/WHEEL,sha256=tkmg4JIqwd9H8mL30xA7crRmoStyCtGp0VWshokd1Jc,105
+wbportfolio-1.54.10.dist-info/licenses/LICENSE,sha256=jvfVH0SY8_YMHlsJHKe_OajiscQDz4lpTlqT6x24sVw,172
+wbportfolio-1.54.10.dist-info/RECORD,,