wbportfolio 1.54.16__py2.py3-none-any.whl → 1.54.17__py2.py3-none-any.whl

wbportfolio/models/orders/order_proposals.py

@@ -394,35 +394,34 @@ class OrderProposal(CloneMixin, RiskCheckMixin, WBModel):
                 orders = service.trades_batch.trades_map.values()
             for order_dto in orders:
                 instrument = Instrument.objects.get(id=order_dto.underlying_instrument)
-                if not instrument.is_cash:  # we do not save order that includes cash component
-                    currency_fx_rate = instrument.currency.convert(
-                        self.value_date, self.portfolio.currency, exact_lookup=True
+                currency_fx_rate = instrument.currency.convert(
+                    self.value_date, self.portfolio.currency, exact_lookup=True
+                )
+                # we cannot do a bulk-create because Order is a multi table inheritance
+                weighting = round(order_dto.delta_weight, Order.ORDER_WEIGHTING_PRECISION)
+                daily_return = order_dto.daily_return
+                try:
+                    order = self.orders.get(underlying_instrument=instrument)
+                    order.weighting = weighting
+                    order.currency_fx_rate = currency_fx_rate
+                    order.daily_return = daily_return
+                except Order.DoesNotExist:
+                    order = Order(
+                        underlying_instrument=instrument,
+                        order_proposal=self,
+                        value_date=self.trade_date,
+                        weighting=weighting,
+                        daily_return=daily_return,
+                        currency_fx_rate=currency_fx_rate,
                     )
-                    # we cannot do a bulk-create because Order is a multi table inheritance
-                    weighting = round(order_dto.delta_weight, Order.ORDER_WEIGHTING_PRECISION)
-                    daily_return = order_dto.daily_return
-                    try:
-                        order = self.orders.get(underlying_instrument=instrument)
-                        order.weighting = weighting
-                        order.currency_fx_rate = currency_fx_rate
-                        order.daily_return = daily_return
-                    except Order.DoesNotExist:
-                        order = Order(
-                            underlying_instrument=instrument,
-                            order_proposal=self,
-                            value_date=self.trade_date,
-                            weighting=weighting,
-                            daily_return=daily_return,
-                            currency_fx_rate=currency_fx_rate,
-                        )
-                    order.price = order.get_price()
-                    order.order_type = Order.get_type(weighting, order_dto.previous_weight, order_dto.target_weight)
-                    # if we cannot automatically find a price, we consider the stock is invalid and we sell it
-                    if not order.price:
-                        order.price = Decimal("0.0")
-                        order.weighting = -order_dto.effective_weight
-
-                    order.save()
+                order.price = order.get_price()
+                order.order_type = Order.get_type(weighting, order_dto.previous_weight, order_dto.target_weight)
+                # if we cannot automatically find a price, we consider the stock is invalid and we sell it
+                if not order.price:
+                    order.price = Decimal("0.0")
+                    order.weighting = -order_dto.effective_weight
+
+                order.save()
         # final sanity check to make sure invalid order with effective and target weight of 0 are automatically removed:
         self.get_orders().filter(target_weight=0, effective_weight=0).delete()
 
@@ -579,7 +578,9 @@ class OrderProposal(CloneMixin, RiskCheckMixin, WBModel):
         currency = self.portfolio.currency
 
         # Calculate the total target weight of all orders
-        total_target_weight = orders.aggregate(s=models.Sum("target_weight"))["s"] or Decimal(0)
+        total_target_weight = orders.exclude(underlying_instrument__is_cash=True).aggregate(
+            s=models.Sum("target_weight")
+        )["s"] or Decimal(0)
 
         if target_cash_weight is None:
             target_cash_weight = Decimal("1") - total_target_weight

wbportfolio/tests/models/orders/test_order_proposals.py

@@ -248,7 +248,9 @@ class TestOrderProposal:
         assert t3.weighting == normalized_t3_weight
 
     # Test resetting orders
-    def test_reset_orders(self, order_proposal, instrument_factory, instrument_price_factory, asset_position_factory):
+    def test_reset_orders(
+        self, order_proposal, instrument_factory, cash, instrument_price_factory, asset_position_factory
+    ):
         """
         Verify orders are correctly reset based on effective and target portfolios.
         """
@@ -334,6 +336,31 @@ class TestOrderProposal:
         assert t2.weighting == Decimal("0")
         assert t3.weighting == Decimal("0.5")
 
+        # assert cash position creates a proper order
+        # build the target portfolio
+        target_portfolio_with_cash = PortfolioDTO(
+            [
+                Position(
+                    underlying_instrument=i1.id,
+                    date=order_proposal.trade_date,
+                    weighting=Decimal("0.5"),
+                    price=float(p1.net_value),
+                ),
+                Position(
+                    underlying_instrument=cash.id,
+                    date=order_proposal.trade_date,
+                    weighting=Decimal("0.5"),
+                    price=1.0,
+                ),
+            ]
+        )
+        order_proposal.reset_orders(target_portfolio=target_portfolio_with_cash)
+
+        # Assert existing trade weights are correctly updated
+        assert order_proposal.orders.get(underlying_instrument=i1).weighting == Decimal("-0.2")
+        assert order_proposal.orders.get(underlying_instrument=i2).weighting == Decimal("-0.3")
+        assert order_proposal.orders.get(underlying_instrument=cash).weighting == Decimal("0.5")
+
     def test_reset_orders_remove_invalid_orders(self, order_proposal, order_factory, instrument_price_factory):
         # create a invalid trade and its price
         invalid_trade = order_factory.create(order_proposal=order_proposal, weighting=Decimal(0))
@@ -423,13 +450,20 @@ class TestOrderProposal:
         order_proposal.portfolio.only_weighting = False
         order_proposal.portfolio.save()
         mock_fct.return_value = Decimal(1_000_000)  # 1 million cash
+        cash = cash_factory.create(currency=order_proposal.portfolio.currency)
         order_factory.create(  # equity trade
             order_proposal=order_proposal,
             value_date=order_proposal.trade_date,
             portfolio=order_proposal.portfolio,
             weighting=Decimal("0.7"),
         )
-
+        order_factory.create(  # cash trade
+            order_proposal=order_proposal,
+            value_date=order_proposal.trade_date,
+            portfolio=order_proposal.portfolio,
+            underlying_instrument=cash,
+            weighting=Decimal("0.2"),
+        )
         target_cash_position = order_proposal.get_estimated_target_cash()
         assert target_cash_position.weighting == Decimal("0.3")
         assert target_cash_position.initial_shares == Decimal(1_000_000) * Decimal("0.3")

wbportfolio/viewsets/orders/orders.py

@@ -198,22 +198,26 @@ class OrderOrderProposalModelViewSet(
             return Order.objects.none()
 
     def get_queryset(self):
-        return self.orders.annotate(
-            underlying_instrument_isin=F("underlying_instrument__isin"),
-            underlying_instrument_ticker=F("underlying_instrument__ticker"),
-            underlying_instrument_refinitiv_identifier_code=F("underlying_instrument__refinitiv_identifier_code"),
-            underlying_instrument_instrument_type=Case(
-                When(
-                    underlying_instrument__parent__is_security=True,
-                    then=F("underlying_instrument__parent__instrument_type__short_name"),
+        return (
+            self.orders.exclude(underlying_instrument__is_cash=True)
+            .annotate(
+                underlying_instrument_isin=F("underlying_instrument__isin"),
+                underlying_instrument_ticker=F("underlying_instrument__ticker"),
+                underlying_instrument_refinitiv_identifier_code=F("underlying_instrument__refinitiv_identifier_code"),
+                underlying_instrument_instrument_type=Case(
+                    When(
+                        underlying_instrument__parent__is_security=True,
+                        then=F("underlying_instrument__parent__instrument_type__short_name"),
+                    ),
+                    default=F("underlying_instrument__instrument_type__short_name"),
                 ),
-                default=F("underlying_instrument__instrument_type__short_name"),
-            ),
-            effective_total_value_fx_portfolio=F("effective_weight") * Value(self.portfolio_total_asset_value),
-            target_total_value_fx_portfolio=F("target_weight") * Value(self.portfolio_total_asset_value),
-            portfolio_currency=F("portfolio__currency__symbol"),
-            security=F("underlying_instrument__parent"),
-            company=F("underlying_instrument__parent__parent"),
-        ).select_related(
-            "underlying_instrument", "underlying_instrument__parent", "underlying_instrument__parent__parent"
+                effective_total_value_fx_portfolio=F("effective_weight") * Value(self.portfolio_total_asset_value),
+                target_total_value_fx_portfolio=F("target_weight") * Value(self.portfolio_total_asset_value),
+                portfolio_currency=F("portfolio__currency__symbol"),
+                security=F("underlying_instrument__parent"),
+                company=F("underlying_instrument__parent__parent"),
+            )
+            .select_related(
+                "underlying_instrument", "underlying_instrument__parent", "underlying_instrument__parent__parent"
+            )
         )

{wbportfolio-1.54.16.dist-info → wbportfolio-1.54.17.dist-info}/METADATA

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

{wbportfolio-1.54.16.dist-info → wbportfolio-1.54.17.dist-info}/RECORD

@@ -288,7 +288,7 @@ wbportfolio/models/mixins/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJW
 wbportfolio/models/mixins/instruments.py,sha256=SgBreTpa_X3uyCWo7t8B0VaTtl49IjmBMe4Pab6TjAM,6796
 wbportfolio/models/mixins/liquidity_stress_test.py,sha256=iQVzT3QM7VtHnqfj9gT6KUIe4wC4MJXery-AXJHUYns,58820
 wbportfolio/models/orders/__init__.py,sha256=EH9UacGR3npBMje5FGTeLOh1xqFBh9kc24WbGmBIA3g,69
-wbportfolio/models/orders/order_proposals.py,sha256=uo0hWOjY6lKpGwa9fnBvskQmI0S_Usg7LYbY9PCJ4xo,40333
+wbportfolio/models/orders/order_proposals.py,sha256=U1vLwI_yS2E1TUs04NJGperusqmNyiE42XGlAVoPatk,40196
 wbportfolio/models/orders/orders.py,sha256=hVVw7NAFmAFHosMMs39V9DjGmWyFC_msSxF8rpDDG60,9683
 wbportfolio/models/reconciliations/__init__.py,sha256=MXH5fZIPGDRBgJkO6wVu_NLRs8fkP1im7G6d-h36lQY,127
 wbportfolio/models/reconciliations/account_reconciliation_lines.py,sha256=QP6M7hMcyFbuXBa55Y-azui6Dl_WgbzMntEqWzQkbfM,7394
@@ -403,7 +403,7 @@ wbportfolio/tests/models/test_roles.py,sha256=4Cn7WyrA2ztJNeWLk5cy9kYo5XLWMbFSvo
 wbportfolio/tests/models/test_splits.py,sha256=ytKcHsI_90kj1L4s8It-KEcc24rkDcElxwQ8q0QxEvk,9689
 wbportfolio/tests/models/utils.py,sha256=ORNJq6NMo1Za22jGZXfTfKeNEnTRlfEt_8SJ6xLaQWg,325
 wbportfolio/tests/models/orders/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-wbportfolio/tests/models/orders/test_order_proposals.py,sha256=xVoKYHhzm_UihJuEY5P8G1kUURCYYSnyFVAR4aPFmUA,29353
+wbportfolio/tests/models/orders/test_order_proposals.py,sha256=7OvZn9IhTNQ66lm2nLXhrdc9OzGuklrnbRgp2WeYQms,30802
 wbportfolio/tests/models/transactions/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 wbportfolio/tests/models/transactions/test_claim.py,sha256=NG3BKB-FVcIDgHSJHCjImxgMM3ISVUMl24xUPmEcPec,5570
 wbportfolio/tests/models/transactions/test_fees.py,sha256=tAp18x2wCNQr11LUnLtHNbBDbbX0v1DZnmW7i-cEi5Q,2423
@@ -532,7 +532,7 @@ wbportfolio/viewsets/configs/titles/roles.py,sha256=9LoJa3jgenXJ5UWRlIErTzdbjpSW
 wbportfolio/viewsets/configs/titles/trades.py,sha256=29XCLxvY0Xe3a2tjCno3tN2rRXCr9RWpbWnzurJfnYI,1986
 wbportfolio/viewsets/orders/__init__.py,sha256=N8v9jdEXryOzrLlc7ML3iBCO2lmNXph9_TWoQ7PTvi4,195
 wbportfolio/viewsets/orders/order_proposals.py,sha256=mw385zzU52nCq8p6xgionh43xLmOn5aX-2BPlCHnqlE,6214
-wbportfolio/viewsets/orders/orders.py,sha256=WH96S97MjZvCudP4b2Hp7YZPmdKfLNGKi90HY3oVQYY,10708
+wbportfolio/viewsets/orders/orders.py,sha256=mekuUVkG9-tEtWGrIW2QfgTOuwOZWONiRnbztYUmcVM,10875
 wbportfolio/viewsets/orders/configs/__init__.py,sha256=5MU57JXiKi32_PicHtiNr7YHmMN020FrlF5NFJf_Wds,94
 wbportfolio/viewsets/orders/configs/buttons/__init__.py,sha256=EHzNmAfa0UQFITEF-wxj_s4wn3Y5DE3DCbEUmmvCTIs,106
 wbportfolio/viewsets/orders/configs/buttons/order_proposals.py,sha256=Q_7LrsuLzjSXCIoscFQXMMHl8cuCporDNM5k735W7d8,3584
@@ -550,7 +550,7 @@ wbportfolio/viewsets/transactions/claim.py,sha256=Pb1WftoO-w-ZSTbLRhmQubhy7hgd68
 wbportfolio/viewsets/transactions/fees.py,sha256=WT2bWWfgozz4_rpyTKX7dgBBTXD-gu0nlsd2Nk2Zh1Q,7028
 wbportfolio/viewsets/transactions/mixins.py,sha256=WipvJoi5hylkpD0y9VATe30WAcwIHUIroVkK10FYw7k,636
 wbportfolio/viewsets/transactions/trades.py,sha256=xBgOGaJ8aEg-2RxEJ4FDaBs4SGwuLasun3nhpis0WQY,12363
-wbportfolio-1.54.16.dist-info/METADATA,sha256=ikKYoJRibnFOVA5vPklrR-QM2077_-0GboeKE0Xfp1A,703
-wbportfolio-1.54.16.dist-info/WHEEL,sha256=tkmg4JIqwd9H8mL30xA7crRmoStyCtGp0VWshokd1Jc,105
-wbportfolio-1.54.16.dist-info/licenses/LICENSE,sha256=jvfVH0SY8_YMHlsJHKe_OajiscQDz4lpTlqT6x24sVw,172
-wbportfolio-1.54.16.dist-info/RECORD,,
+wbportfolio-1.54.17.dist-info/METADATA,sha256=iv55pLD6PfL1brFSH400eFZJ1u_BieHNt_tO2nrQ3Us,703
+wbportfolio-1.54.17.dist-info/WHEEL,sha256=tkmg4JIqwd9H8mL30xA7crRmoStyCtGp0VWshokd1Jc,105
+wbportfolio-1.54.17.dist-info/licenses/LICENSE,sha256=jvfVH0SY8_YMHlsJHKe_OajiscQDz4lpTlqT6x24sVw,172
+wbportfolio-1.54.17.dist-info/RECORD,,