Restore wallet registration overview - #3079
Draft
UnArbosFive wants to merge 1 commit into
Draft
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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.
Problem
wallet overviewnow presents a stake portfolio. That leaves miners without a single command to list all hotkeys owned by local coldkeys, their subnet registrations, UIDs, and current incentive data. Composing the existing per-hotkey and per-subnet queries also misses the old overview workflow, especially for registered hotkeys with no alpha stake.Root cause
The current overview derives its rows from stake positions. Ownership and subnet membership are separate chain state, so an owned registered miner can be absent from that view when the coldkey has no stake on it.
Fix
btcli wallet registrations, with--alland optional--netuidfiltering.wallet overviewbehavior unchanged.Validation
python -m pytest tests/unit/test_cli.py -q— 40 passed.python -m pytest tests/unit -q— 1,094 passed, 1 skipped.ruff format --checkandruff checkpass for all changed files.