Merge pull request #3189 from opentensor/fix/roman/Transaction-has-an-ancient-birth-block

[WIP][v10]Transaction-has-an-ancient-birth-block

Author: basfroman

bittensor/core/async_subtensor.py

@@ -7937,7 +7937,7 @@ async def set_weights(
         version_key: int = version_as_int,
         *,
         mev_protection: bool = DEFAULT_MEV_PROTECTION,
-        period: Optional[int] = 8,
+        period: Optional[int] = DEFAULT_PERIOD,
         raise_error: bool = False,
         wait_for_inclusion: bool = True,
         wait_for_finalization: bool = True,

bittensor/core/subtensor.py

@@ -6638,7 +6638,7 @@ def set_weights(
         version_key: int = version_as_int,
         *,
         mev_protection: bool = DEFAULT_MEV_PROTECTION,
-        period: Optional[int] = 8,
+        period: Optional[int] = DEFAULT_PERIOD,
         raise_error: bool = False,
         wait_for_inclusion: bool = True,
         wait_for_finalization: bool = True,

tests/unit_tests/test_subtensor.py

@@ -3011,7 +3011,7 @@ def test_set_weights_with_commit_reveal_enabled(subtensor, fake_wallet, mocker):
         wait_for_inclusion=fake_wait_for_inclusion,
         wait_for_finalization=fake_wait_for_finalization,
         block_time=12.0,
-        period=8,
+        period=DEFAULT_PERIOD,
         raise_error=False,
         mechid=0,
         wait_for_revealed_execution=True,