Merge pull request #694 from opentensor/devnet

Testnet Deployment 260724

Author: sam0x17

Diff for: Cargo.lock

@@ -47,4 +47,8 @@ lint:
   @echo "Running cargo clippy with automatic fixes on potentially dirty code..."
   just clippy-fix
   @echo "Running cargo clippy..."
-  just clippy
+  just clippy
+
+production:
+  @echo "Running cargo build with metadata-hash generation..."
+  cargo +{{RUSTV}} build --profile production --features="runtime-benchmarks metadata-hash"

Diff for: Cargo.toml

@@ -54,6 +54,7 @@ sp-io = { workspace = true }
 sp-timestamp = { workspace = true }
 sp-inherents = { workspace = true }
 sp-keyring = { workspace = true }
+frame-metadata-hash-extension = { workspace = true }
 frame-system = { workspace = true }
 pallet-transaction-payment = { workspace = true }
 pallet-commitments = { path = "../pallets/commitments" }

Diff for: justfile

@@ -136,6 +136,7 @@ pub fn create_benchmark_extrinsic(
         pallet_transaction_payment::ChargeTransactionPayment::<runtime::Runtime>::from(0),
         pallet_subtensor::SubtensorSignedExtension::<runtime::Runtime>::new(),
         pallet_commitments::CommitmentsSignedExtension::<runtime::Runtime>::new(),
+        frame_metadata_hash_extension::CheckMetadataHash::<runtime::Runtime>::new(true),
     );
 
     let raw_payload = runtime::SignedPayload::from_raw(
@@ -152,6 +153,7 @@ pub fn create_benchmark_extrinsic(
             (),
             (),
             (),
+            None,
         ),
     );
     let signature = raw_payload.using_encoded(|e| sender.sign(e));

Diff for: node/Cargo.toml

@@ -240,5 +240,21 @@ mod benchmarks {
 		_(RawOrigin::Root, 1u16/*netuid*/, true/*enabled*/)/*set_commit_reveal_weights_enabled*/;
     }
 
+    #[benchmark]
+    fn sudo_set_hotkey_emission_tempo() {
+        T::Subtensor::init_new_network(1u16 /*netuid*/, 1u16 /*sudo_tempo*/);
+
+        #[extrinsic_call]
+        _(RawOrigin::Root, 1u64/*emission_tempo*/)/*set_hotkey_emission_tempo*/;
+    }
+
+    #[benchmark]
+    fn sudo_set_network_max_stake() {
+        T::Subtensor::init_new_network(1u16 /*netuid*/, 1u16 /*tempo*/);
+
+        #[extrinsic_call]
+        _(RawOrigin::Root, 1u16/*netuid*/, 1_000_000_000_000_000u64/*max_stake*/)/*sudo_set_network_max_stake*/;
+    }
+
     //impl_benchmark_test_suite!(AdminUtils, crate::mock::new_test_ext(), crate::mock::Test);
 }

Diff for: node/src/benchmarking.rs

@@ -1035,6 +1035,87 @@ pub mod pallet {
             T::Subtensor::ensure_subnet_owner_or_root(origin.clone(), netuid)?;
             T::Subtensor::do_set_alpha_values(origin, netuid, alpha_low, alpha_high)
         }
+
+        /// Sets the hotkey emission tempo.
+        ///
+        /// This extrinsic allows the root account to set the hotkey emission tempo, which determines
+        /// the number of blocks before a hotkey drains accumulated emissions through to nominator staking accounts.
+        ///
+        /// # Arguments
+        /// * `origin` - The origin of the call, which must be the root account.
+        /// * `emission_tempo` - The new emission tempo value to set.
+        ///
+        /// # Emits
+        /// * `Event::HotkeyEmissionTempoSet` - When the hotkey emission tempo is successfully set.
+        ///
+        /// # Errors
+        /// * `DispatchError::BadOrigin` - If the origin is not the root account.
+        // #[pallet::weight(T::WeightInfo::sudo_set_hotkey_emission_tempo())]
+        #[pallet::call_index(52)]
+        #[pallet::weight((0, DispatchClass::Operational, Pays::No))]
+        pub fn sudo_set_hotkey_emission_tempo(
+            origin: OriginFor<T>,
+            emission_tempo: u64,
+        ) -> DispatchResult {
+            ensure_root(origin)?;
+            T::Subtensor::set_hotkey_emission_tempo(emission_tempo);
+            log::info!(
+                "HotkeyEmissionTempoSet( emission_tempo: {:?} )",
+                emission_tempo
+            );
+            Ok(())
+        }
+
+        /// Sets the maximum stake allowed for a specific network.
+        ///
+        /// This function allows the root account to set the maximum stake for a given network.
+        /// It updates the network's maximum stake value and logs the change.
+        ///
+        /// # Arguments
+        ///
+        /// * `origin` - The origin of the call, which must be the root account.
+        /// * `netuid` - The unique identifier of the network.
+        /// * `max_stake` - The new maximum stake value to set.
+        ///
+        /// # Returns
+        ///
+        /// Returns `Ok(())` if the operation is successful, or an error if it fails.
+        ///
+        /// # Example
+        ///
+        ///
+        /// # Notes
+        ///
+        /// - This function can only be called by the root account.
+        /// - The `netuid` should correspond to an existing network.
+        ///
+        /// # TODO
+        ///
+        // - Consider adding a check to ensure the `netuid` corresponds to an existing network.
+        // - Implement a mechanism to gradually adjust the max stake to prevent sudden changes.
+        // #[pallet::weight(T::WeightInfo::sudo_set_network_max_stake())]
+        #[pallet::call_index(53)]
+        #[pallet::weight((0, DispatchClass::Operational, Pays::No))]
+        pub fn sudo_set_network_max_stake(
+            origin: OriginFor<T>,
+            netuid: u16,
+            max_stake: u64,
+        ) -> DispatchResult {
+            // Ensure the call is made by the root account
+            ensure_root(origin)?;
+
+            // Set the new maximum stake for the specified network
+            T::Subtensor::set_network_max_stake(netuid, max_stake);
+
+            // Log the change
+            log::trace!(
+                "NetworkMaxStakeSet( netuid: {:?}, max_stake: {:?} )",
+                netuid,
+                max_stake
+            );
+
+            Ok(())
+        }
     }
 }
 
@@ -1137,4 +1218,6 @@ pub trait SubtensorInterface<AccountId, Balance, RuntimeOrigin> {
         alpha_low: u16,
         alpha_high: u16,
     ) -> Result<(), DispatchError>;
+    fn set_hotkey_emission_tempo(emission_tempo: u64);
+    fn set_network_max_stake(netuid: u16, max_stake: u64);
 }

Diff for: pallets/admin-utils/src/benchmarking.rs

@@ -62,7 +62,6 @@ pub trait WeightInfo {
 	fn sudo_set_tempo() -> Weight;
 	fn sudo_set_commit_reveal_weights_interval() -> Weight;
 	fn sudo_set_commit_reveal_weights_enabled() -> Weight;
-	
 }
 
 /// Weights for `pallet_admin_utils` using the Substrate node and recommended hardware.

Diff for: pallets/admin-utils/src/lib.rs

@@ -114,7 +114,8 @@ parameter_types! {
     pub const InitialAlphaHigh: u16 = 58982; // Represents 0.9 as per the production default
     pub const InitialAlphaLow: u16 = 45875; // Represents 0.7 as per the production default
     pub const InitialLiquidAlphaOn: bool = false; // Default value for LiquidAlphaOn
-    pub const InitialBaseDifficulty: u64 = 10_000; // Base difficulty
+    pub const InitialHotkeyEmissionTempo: u64 = 1;
+    pub const InitialNetworkMaxStake: u64 = 500_000_000_000_000; // 500_000 TAO
 }
 
 impl pallet_subtensor::Config for Test {
@@ -170,7 +171,8 @@ impl pallet_subtensor::Config for Test {
     type AlphaHigh = InitialAlphaHigh;
     type AlphaLow = InitialAlphaLow;
     type LiquidAlphaOn = InitialLiquidAlphaOn;
-    type InitialBaseDifficulty = InitialBaseDifficulty;
+    type InitialHotkeyEmissionTempo = InitialHotkeyEmissionTempo;
+    type InitialNetworkMaxStake = InitialNetworkMaxStake;
 }
 
 #[derive_impl(frame_system::config_preludes::TestDefaultConfig)]
@@ -477,6 +479,13 @@ impl pallet_admin_utils::SubtensorInterface<AccountId, Balance, RuntimeOrigin> f
     fn set_liquid_alpha_enabled(netuid: u16, enabled: bool) {
         SubtensorModule::set_liquid_alpha_enabled(netuid, enabled);
     }
+    fn set_hotkey_emission_tempo(emission_tempo: u64) {
+        SubtensorModule::set_hotkey_emission_tempo(emission_tempo)
+    }
+    fn set_network_max_stake(netuid: u16, max_stake: u64) {
+        SubtensorModule::set_network_max_stake(netuid, max_stake)
+    }
+
     fn do_set_alpha_values(
         origin: RuntimeOrigin,
         netuid: u16,

Diff for: pallets/admin-utils/src/weights.rs

@@ -6,7 +6,7 @@ use frame_support::{
 use frame_system::Config;
 use pallet_admin_utils::Error;
 use pallet_subtensor::Error as SubtensorError;
-use pallet_subtensor::{migration, Event};
+use pallet_subtensor::{migrations, Event};
 use sp_core::U256;
 
 mod mock;
@@ -1232,7 +1232,7 @@ fn test_sudo_get_set_alpha() {
 
         // Enable Liquid Alpha and setup
         SubtensorModule::set_liquid_alpha_enabled(netuid, true);
-        migration::migrate_create_root_network::<Test>();
+        migrations::migrate_create_root_network::migrate_create_root_network::<Test>();
         SubtensorModule::add_balance_to_coldkey_account(&coldkey, 1_000_000_000_000_000);
         assert_ok!(SubtensorModule::root_register(signer.clone(), hotkey,));
         assert_ok!(SubtensorModule::add_stake(signer.clone(), hotkey, 1000));

Diff for: pallets/admin-utils/tests/mock.rs

@@ -951,9 +951,9 @@ impl<T: Config<I>, I: 'static> Pallet<T, I> {
     ///
     /// If not `approved`:
     /// - one event deposited.
-    /// Two removals, one mutation.
-    /// Computation and i/o `O(P)` where:
-    /// - `P` is number of active proposals
+    /// - two removals, one mutation.
+    /// - computation and i/o `O(P)` where:
+    ///   - `P` is number of active proposals
     fn do_approve_proposal(
         seats: MemberCount,
         yes_votes: MemberCount,