don't clear alpha maps

Author: camfairchild

pallets/subtensor/src/migrations/migrate_reset_unactive_sn.rs

@@ -134,44 +134,9 @@ pub fn migrate_reset_unactive_sn<T: Config>() -> Weight {
 
         // Reset pool alpha
         SubnetAlphaIn::<T>::insert(*netuid, pool_initial_alpha);
-        SubnetAlphaOut::<T>::insert(*netuid, AlphaCurrency::ZERO);
         // Reset volume
         SubnetVolume::<T>::insert(*netuid, 0u128);
         weight = weight.saturating_add(T::DbWeight::get().writes(4));
-
-        // Reset Alpha stake entries for this subnet
-        let to_reset: Vec<T::AccountId> = match to_remove_alpha_hotkeys.get(netuid) {
-            Some(hotkeys) => hotkeys.clone(),
-            None => Vec::new(),
-        };
-
-        for hotkey in to_reset {
-            TotalHotkeyAlpha::<T>::remove(&hotkey, *netuid);
-            TotalHotkeyShares::<T>::remove(&hotkey, *netuid);
-            TotalHotkeyAlphaLastEpoch::<T>::remove(&hotkey, *netuid);
-            weight = weight.saturating_add(T::DbWeight::get().writes(3));
-
-            // Reset root claimable and claimed
-            RootClaimable::<T>::mutate(&hotkey, |claimable| {
-                claimable.remove(netuid);
-            });
-            weight = weight.saturating_add(T::DbWeight::get().reads_writes(1, 1));
-            let removal_result = RootClaimed::<T>::clear_prefix((*netuid, &hotkey), u32::MAX, None);
-            weight = weight.saturating_add(
-                T::DbWeight::get()
-                    .reads_writes(removal_result.loops as u64, removal_result.backend as u64),
-            );
-
-            let to_reset_alpha: Vec<(T::AccountId, T::AccountId)> =
-                match to_remove_alpha_coldkeys.get(netuid) {
-                    Some(coldkeys) => coldkeys.clone(),
-                    None => Vec::new(),
-                };
-            for (hotkey, coldkey) in to_reset_alpha {
-                Alpha::<T>::remove((hotkey, coldkey, netuid));
-                weight = weight.saturating_add(T::DbWeight::get().writes(1));
-            }
-        }
     }
 
     // Run total issuance migration

pallets/subtensor/src/tests/migration.rs

@@ -2581,7 +2581,7 @@ fn test_migrate_reset_unactive_sn() {
             assert_eq!(PendingOwnerCut::<Test>::get(netuid), AlphaCurrency::ZERO);
             assert_eq!(SubnetTAO::<Test>::get(netuid), initial_tao);
             assert_eq!(SubnetAlphaIn::<Test>::get(netuid), initial_alpha);
-            assert_eq!(SubnetAlphaOut::<Test>::get(netuid), AlphaCurrency::ZERO);
+            assert_ne!(SubnetAlphaOut::<Test>::get(netuid), AlphaCurrency::ZERO);
             assert_eq!(SubnetTaoInEmission::<Test>::get(netuid), TaoCurrency::ZERO);
             assert_eq!(
                 SubnetAlphaInEmission::<Test>::get(netuid),
@@ -2594,23 +2594,23 @@ fn test_migrate_reset_unactive_sn() {
             assert_eq!(SubnetVolume::<Test>::get(netuid), 0u128);
             for hotkey in 0..10 {
                 let hk = U256::from(hotkey);
-                assert_eq!(
+                assert_ne!(
                     TotalHotkeyAlpha::<Test>::get(hk, netuid),
                     AlphaCurrency::ZERO
                 );
-                assert_eq!(
+                assert_ne!(
                     TotalHotkeyShares::<Test>::get(hk, netuid),
                     U64F64::from_num(0.0)
                 );
-                assert_eq!(
+                assert_ne!(
                     TotalHotkeyAlphaLastEpoch::<Test>::get(hk, netuid),
                     AlphaCurrency::ZERO
                 );
-                assert_eq!(RootClaimable::<Test>::get(hk).get(netuid), None);
+                assert_ne!(RootClaimable::<Test>::get(hk).get(netuid), None);
                 for coldkey in 0..10 {
                     let ck = U256::from(coldkey);
-                    assert_eq!(Alpha::<Test>::get((hk, ck, netuid)), U64F64::from_num(0.0));
-                    assert_eq!(RootClaimed::<Test>::get((netuid, hk, ck)), 0u128);
+                    assert_ne!(Alpha::<Test>::get((hk, ck, netuid)), U64F64::from_num(0.0));
+                    assert_ne!(RootClaimed::<Test>::get((netuid, hk, ck)), 0u128);
                 }
             }
 

runtime/src/lib.rs

@@ -220,7 +220,7 @@ pub const VERSION: RuntimeVersion = RuntimeVersion {
     //   `spec_version`, and `authoring_version` are the same between Wasm and native.
     // This value is set to 100 to notify Polkadot-JS App (https://polkadot.js.org/apps) to use
     //   the compatible custom types.
-    spec_version: 344,
+    spec_version: 345,
     impl_version: 1,
     apis: RUNTIME_API_VERSIONS,
     transaction_version: 1,