Skip to content

Remove commit_reveal_weights_interval and add commit_reveal_weights_p…#6

Closed
opendansor wants to merge 2 commits intomainfrom
feat/opendansor/cr_hyperparam
Closed

Remove commit_reveal_weights_interval and add commit_reveal_weights_p…#6
opendansor wants to merge 2 commits intomainfrom
feat/opendansor/cr_hyperparam

Conversation

@opendansor
Copy link

…eriods

This change updates variable names in multiple files for consistency and clarity. The renamed variables reflect the intended meaning more accurately across the codebase and tests.

…eriods

This change updates variable names in multiple files for consistency and clarity. The renamed variables reflect the intended meaning more accurately across the codebase and tests.
@opendansor opendansor self-assigned this Oct 18, 2024
Updated bt_decode.pyi, src/lib.rs, and tests/test_types.json to reflect the new naming convention for consistency. This change improves code readability and aligns terminology across the codebase.
@camfairchild
Copy link
Contributor

Depends on opentensor/subtensor#864

Copy link
Collaborator

@basfroman basfroman left a comment

Choose a reason for hiding this comment

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

LGTM

@opendansor opendansor closed this Oct 31, 2024
@opendansor
Copy link
Author

Hyperparam changed back. No longer need this

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.

3 participants