Skip to content

split process_weights_for_netuid#2777

Merged
thewhaleking merged 1 commit intoopentensor:stagingfrom
backend-developers-ltd:process_weights_for_netuid
Apr 9, 2025
Merged

split process_weights_for_netuid#2777
thewhaleking merged 1 commit intoopentensor:stagingfrom
backend-developers-ltd:process_weights_for_netuid

Conversation

@andreea-popescu-reef
Copy link
Contributor

Description of the Change

Refactor process_weights_for_netuid so that the subtensor calls (for getting min_allowed_weights and max_weight_limit params and optionally the metagraph) remain in the original function body while the rest of the logic moves to a new process_weights function. This allows users to use directly process_weights without extra subtensor calls. No functional changes.

Alternate Designs

N/A

Possible Drawbacks

N/A

Verification Process

checked locally

Release Notes

Introduced process_weights for direct usage of process_weights_for_netuid without subtensor calls.

Copy link
Contributor

@thewhaleking thewhaleking left a comment

Choose a reason for hiding this comment

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

Approved but should be merged through #2798

@thewhaleking thewhaleking merged commit a555e5e into opentensor:staging Apr 9, 2025
111 of 118 checks passed
@ibraheem-abe ibraheem-abe mentioned this pull request Apr 9, 2025
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