Skip to content

feat: set discriminator weights on current KOTH kings - #433

Merged
kenobijon merged 6 commits into
testnetfrom
feat/koth-phase-a
Sep 4, 2026
Merged

feat: set discriminator weights on current KOTH kings#433
kenobijon merged 6 commits into
testnetfrom
feat/koth-phase-a

Conversation

@kenobijon

Copy link
Copy Markdown
Contributor

Summary

  • Replace escrow UID weight setting with /api/v1/validator/current-kings.
  • Image/video/audio shares (40/40/4) go to the reigning miner hotkey; 16% stays on generators. Missing kings burn that lane.
  • Last-known kings persist across restarts. Spec version bumped to 4.10.0 so stale validators cannot keep setting escrow weights.

Depends on BitMind-AI/bmcore KOTH current-kings API.

Test plan

  • pytest tests/test_koth_weights.py
  • Confirm set_weights logs king modalities and never resolves the old escrow SS58s.
  • Simulate API down and confirm last-known kings are reused.
  • After deploy, inspect on-chain weights: miner UIDs, not 5EUJF… / 5G6BJ… / 5F9Qo….

Made with Cursor

Comment thread gas/koth_weights.py
@dylanuys
dylanuys marked this pull request as draft September 2, 2026 18:21
@dylanuys

dylanuys commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

DO NOT MERGE until v23 completes. Keep the current staging system isolated until the v23 benchmark cycle is finished.

@kenobijon
kenobijon marked this pull request as ready for review September 2, 2026 19:02
Bittensor metagraph threads kept the process alive, and a 409 with
--skip-chain was treated as failure. Cap chain retries and _exit.

Co-authored-by: Cursor <cursoragent@cursor.com>

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

There are 2 total unresolved issues (including 1 from previous review).

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 60e38a8. Configure here.

Comment thread neurons/discriminator/push_model.py
dylanuys and others added 5 commits September 2, 2026 15:18
Keep the current king dominant while paying the previous two distinct
crowned hotkeys a residual so near-ties are not winner-take-lane.

Co-authored-by: Cursor <cursoragent@cursor.com>
Paying last-3 kings 85/10/5 is a breaking incentive change, so spec_version
should move with a major rather than sit on 4.10.0.

Co-authored-by: Cursor <cursoragent@cursor.com>
@kenobijon
kenobijon changed the base branch from main to testnet September 4, 2026 03:01
@kenobijon
kenobijon merged commit 1de92f9 into testnet Sep 4, 2026
3 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.

2 participants