Fix alplha accounting - #3074
Open
UnArbosFour wants to merge 3 commits into
Open
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
🛡️ AI Review — Skeptic (security review)VERDICT: VULNERABLE VERY HIGH scrutiny: 40-day-old account with one public repository; repository write access, no Gittensor association found, and no author/committer mismatch. Findings
Other findings
Prior-comment reconciliation
ConclusionConsensus-critical runtime logic and migrations still change without incrementing 📜 Previous run (superseded)
# 🔍 AI Review — Auditor (domain review) has not yet run on this PR. |
Contributor
|
🔄 AI review updated — Skeptic: VULNERABLE |
Closed
Contributor
|
🔄 AI review updated — Skeptic: VULNERABLE |
UnArbosFour
force-pushed
the
fix/alplha-accounting
branch
from
August 11, 2026 20:24
22ee995 to
3e38998
Compare
Contributor
|
🔄 AI review updated — Skeptic: VULNERABLE |
UnArbosFour
force-pushed
the
fix/alplha-accounting
branch
from
August 11, 2026 20:28
3e38998 to
b747d7a
Compare
Contributor
|
🔄 AI review updated — Skeptic: VULNERABLE |
Contributor
|
🔄 AI review updated — Skeptic: VULNERABLE |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes two historical
SubnetAlphaOutaccounting deviations and updates the subnet ownership-transfer quorum to exclude alpha that cannot support a challenger.The migration only corrects accounting counters. It does not mint alpha, modify stake, or replay historical chain state.
Historical accounting reconstruction
Every block from the RAO activation at block
4,920,351through block8,780,303was scanned across the applicable runtime versions.Duplicated RAO dividends
Between blocks
4,920,351and4,962,968, local dividends were duplicated: participant stake received both the full local dividend and the root portion, whileSubnetAlphaOutrecorded only the intended issuance.At hotfix block
4,962,968, the per-subnet difference betweenTotalHotkeyAlphaandSubnetAlphaOutwas used to calculate the missing accounting amount. The migration increasesSubnetAlphaOutby that stabilized difference.Subnets 65 and 66 were not missed:
4,950,813and accumulated a correction of4,932.684752021alpha.4,958,013and accumulated a correction of2,003.518827049alpha.Their corrections are smaller because they were exposed to the duplication only from their registration blocks until the hotfix.
Root accounting
Legacy root dividends were credited directly to root stake without increasing
SubnetAlphaOut(0).The root discrepancy was measured immediately before Root Reborn at block
8,765,683and verified again at block8,822,961. The difference remained unchanged at728,652.620877147alpha, confirming that the deviation had stopped before applying the correction.Excluded subnet generations
Eleven netuids were excluded:
15, 16, 26, 31, 36, 38, 40, 47, 49, 57, 58Their affected historical subnet generations were dissolved and the netuids were subsequently reused. Applying the old correction would modify an unrelated current subnet asset.
Migration behavior
SubnetAlphaOutby a total of1,618,308.219994798alpha.Ownership-transfer threshold
Changes the challenger requirement from:
to:
Subtraction is saturating, and ownership cannot transfer when the resulting eligible alpha is zero.
AlphaBurnedis exposed throughAlphaAssetsInterfaceto avoid coupling Subtensor directly to alpha-assets storage.Testing