Skip to content

Remove hyperparameter max_weight_limit#2141

Merged
sam0x17 merged 3 commits into
devnet-readyfrom
disable-max-weight-limit
Oct 17, 2025
Merged

Remove hyperparameter max_weight_limit#2141
sam0x17 merged 3 commits into
devnet-readyfrom
disable-max-weight-limit

Conversation

@sam0x17

@sam0x17 sam0x17 commented Oct 16, 2025

Copy link
Copy Markdown
Contributor

replaces max_weight_limit hyperparameter with a constant limit of u16::MAX

@sam0x17 sam0x17 self-assigned this Oct 16, 2025
@sam0x17 sam0x17 added the skip-cargo-audit This PR fails cargo audit but needs to be merged anyway label Oct 16, 2025
@sam0x17 sam0x17 merged commit 249fa62 into devnet-ready Oct 17, 2025
55 of 56 checks passed
kkowalski-reef added a commit to bittensor-church/agcli that referenced this pull request Jun 26, 2026
There is no longer any extrinsic to set a subnet's max weight limit:
opentensor/subtensor#2141 ("disable max weight limit", Oct 2025) deleted the
AdminUtils.sudo_set_max_weight_limit dispatchable and made get_max_weight_limit
a constant u16::MAX. This predates the current finney runtime — agcli's call
has been dead for a long time.

In scaffold it was silently a no-op: submit_sudo_raw_call_checked bails with
"Call ... not found in runtime metadata" and try_admin! swallows anything
containing "not found", so the configured max_weight_limit was never applied
and the run just printed a warning.

Remove the scaffold surface for it — the SubnetConfig.max_weight_limit field,
its default, the set call, the example config line, and the test fixtures.
The general admin::set_max_weight_limit helper and its CLI command are left
in place (separate machinery, out of scope here).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-cargo-audit This PR fails cargo audit but needs to be merged anyway

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants