Closes #131

CHANGELOG.md

@@ -5,6 +5,18 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.29.1] - Unpublished
+
+### Added
+
+### Fixed
+
+### Changed
+
+- [change](https://github.com/FrankC01/pysui/issues/131) - improve SuiTransaction constructor performance
+
+### Removed
+
 ## [0.29.0] - 2023-07-07
 
 ### Added

pysui/sui/sui_clients/transaction.py

@@ -16,6 +16,7 @@
 
 import base64
 import os
+import time
 
 # import asyncio
 from pathlib import Path
@@ -692,19 +693,18 @@ def _compile_source(
 class SuiTransactionAsync(_SuiTransactionBase):
     """."""
 
+    @versionchanged(
+        version="0.29.1", reason="Eliminated redundant gas price RPC call"
+    )
     def __init__(
         self,
         client: AsyncClient,
         merge_gas_budget: bool = False,
         initial_sender: Union[SuiAddress, SigningMultiSig] = False,
     ) -> None:
         """Transaction initializer."""
-        assert client.is_synchronous is False, "Requires asynch SuiClient"
         super().__init__(client, merge_gas_budget, initial_sender)
-        s_client = SyncClient(client.config)
-        self._current_gas_price = handle_result(
-            s_client.execute(GetReferenceGasPrice())
-        )
+        self._current_gas_price = client.current_gas_price
 
     @versionchanged(version="0.17.0", reason="Only used internally.")
     @versionchanged(
@@ -1875,6 +1875,9 @@ class SuiTransaction(_SuiTransactionBase):
     @versionchanged(
         version="0.21.1", reason="Takes a 'initial_sender' as option."
     )
+    @versionchanged(
+        version="0.29.1", reason="Eliminated redundant gas price RPC call"
+    )
     def __init__(
         self,
         client: SyncClient,
@@ -1883,22 +1886,7 @@ def __init__(
     ) -> None:
         """Transaction initializer."""
         super().__init__(client, merge_gas_budget, initial_sender)
-        self._current_gas_price = handle_result(
-            self.client.execute(GetReferenceGasPrice())
-        )
-
-    # def _reset_gas(self, for_owner: Union[SuiAddress, MultiSig]) -> list:
-    #     """_reset_gas Returns gas objects imbued with owners."""
-    #     coin_objects = handle_result(self.client.get_gas(for_owner)).data
-    #     out_list = []
-    #     for coin_object in coin_objects:
-    #         setattr(coin_object, "owner", for_owner)
-    #         out_list.append(coin_object)
-    #     return out_list
-
-    # def _gas_price(self) -> int:
-    #     """Retrieve the current gas price from the chain during initialization."""
-    #     return handle_result(self.client.execute(GetReferenceGasPrice()))
+        self._current_gas_price = client.current_gas_price
 
     @versionchanged(
         version="0.16.1",

pysui/version.py

@@ -12,4 +12,4 @@
 # -*- coding: utf-8 -*-
 
 """Pysui Version."""
-__version__ = "0.29.0"
+__version__ = "0.29.1"