Skip to content

fix: skip unregistered escrow UIDs when setting weights - #439

Merged
dylanuys merged 2 commits into
mainfrom
fix/skip-unregistered-escrow-weight-uids
Sep 5, 2026
Merged

fix: skip unregistered escrow UIDs when setting weights#439
dylanuys merged 2 commits into
mainfrom
fix/skip-unregistered-escrow-weight-uids

Conversation

@dylanuys

@dylanuys dylanuys commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Unregistered image/video/audio escrow hotkeys resolve to None. Indexing the weight vector with None is a numpy broadcast that zeros every UID, so process_weights_for_netuid falls back to all-ones.
  • Skip those None UIDs when building special_uids and when assigning escrow weights. Burn is unchanged and still assigned directly.
  • This restores the intended 84% burn / 16% generator split instead of uniform weights.

Test plan

  • Confirmed on SN34 that escrow lookups return None and burn still resolves to UID 135
  • After deploy, block 9001800 and 9002160 logged Total weight sum: 1.0000, Actual burn rate: 0.8400 and set_weights on chain successfully!
  • Revealed on-chain weights for UID 135: 140 non-zero dests, 84% on burn
  • Confirm next tempo still reveals 84/16 (not all-ones)

Made with Cursor

dylanuys and others added 2 commits September 5, 2026 17:12
Unregistered escrow hotkeys resolve to None, and numpy treats arr[None] = x as a broadcast that zeros the entire weight vector, so set_weights fell back to all-ones.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@dylanuys
dylanuys merged commit a024c54 into main Sep 5, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant