Skip to content

hyperparameter refactor#2048

Merged
lorismoulin merged 10 commits into
devnet-readyfrom
sam-change-hyperparameter-rate-limit
Sep 16, 2025
Merged

hyperparameter refactor#2048
lorismoulin merged 10 commits into
devnet-readyfrom
sam-change-hyperparameter-rate-limit

Conversation

@sam0x17

@sam0x17 sam0x17 commented Sep 11, 2025

Copy link
Copy Markdown
Contributor
  • set rate limit to be based on tempo
  • rate limit default => 2 tempos
  • make rate limit apply individually for each hyperparameter instead of one global rate limit
  • all hyperparameters still use the same rate limit

@sam0x17 sam0x17 self-assigned this Sep 11, 2025
@sam0x17 sam0x17 added the skip-cargo-audit This PR fails cargo audit but needs to be merged anyway label Sep 11, 2025
@sam0x17 sam0x17 changed the title lower hyperparameter rate limit to 2 tempos hyperparameter refactor Sep 11, 2025
- return `rate_limit` naming
@ales-otf ales-otf self-assigned this Sep 12, 2025
Comment thread pallets/subtensor/src/migrations/migrate_owner_hparam_rl_to_epochs.rs Outdated
lorismoulin
lorismoulin previously approved these changes Sep 15, 2025
@lorismoulin
lorismoulin merged commit a313461 into devnet-ready Sep 16, 2025
54 of 56 checks passed
@IntiTechnologies
IntiTechnologies deleted the sam-change-hyperparameter-rate-limit branch July 9, 2026 18:07
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.

4 participants