Merged
Conversation
Refactor indentation for readability Adjusted indentation in `stake.py` to maintain consistent code formatting. This change addresses alignment issues in functions, argument lists, and block structures, enhancing the readability and maintainability of the code.
added 2 commits
September 3, 2024 16:35
Corrected indentation across multiple functions in stake.py to maintain code consistency and readability. These changes ensure proper alignment as per Python syntax, improving maintainability and reducing potential bugs.
Corrected inconsistent indentation in multiple sections of stake.py, ensuring proper alignment for readability and code functionality. This change addresses formatting glitches without altering the core logic.
thewhaleking
requested changes
Sep 4, 2024
added 3 commits
September 4, 2024 09:43
Add an `all_netuids` parameter to allow fetching children from all subnets. Also, enhance prompts and options for better user experience. Adjust command usage examples accordingly.
Corrected the indentation of the `stake_get_children` method in `bittensor_cli/cli.py` for better readability and code consistency. This change does not modify functionality but improves code structure.
Refactor `netuid` parameter to be an optional string, and add logic to treat "all" as a special case by setting `netuid` to `None`. This improves the flexibility and user input handling in the staking command.
opendansor
commented
Sep 4, 2024
| network: Optional[str] = Options.network, | ||
| chain: Optional[str] = Options.chain, | ||
| netuid: int = Options.netuid, | ||
| netuid: Optional[str] = typer.Option( |
Author
There was a problem hiding this comment.
This is not Options.netuid because if --all flag is entered we dont want to prompt the user for netuid. (set prompt to false)
Contributor
There was a problem hiding this comment.
netuid should be an Optional[int] not str
thewhaleking
requested changes
Sep 4, 2024
Contributor
thewhaleking
left a comment
thewhaleking
left a comment
There was a problem hiding this comment.
Also please run ruff.
| network: Optional[str] = Options.network, | ||
| chain: Optional[str] = Options.chain, | ||
| netuid: int = Options.netuid, | ||
| netuid: Optional[str] = typer.Option( |
Contributor
There was a problem hiding this comment.
netuid should be an Optional[int] not str
| netuid = None | ||
| elif not netuid: | ||
| netuid = Prompt.ask("Netuid") | ||
| if netuid and netuid.lower() == "all": |
Contributor
There was a problem hiding this comment.
This will not work, because netuid should be an int (consistent and also what get_children expects), so does not have the attribute lower.
| if all_netuids: | ||
| netuid = None | ||
| elif not netuid: | ||
| netuid = Prompt.ask("Netuid") |
Author
There was a problem hiding this comment.
For all of the netuid(int) questions here,
If you do not enter netuid in the args, it will prompt you for netuid.
If you enter 'all' it should behave as if you entered the --all flag to get the children from all subnets.
added 2 commits
September 4, 2024 12:06
Updated the input prompt for netuid to use IntPrompt, ensuring integer input. Additionally, added a check to prevent specifying both netuid and all_netuids options simultaneously, exiting with an error message if both are given.
Updated the input prompt for netuid to use IntPrompt, ensuring integer input. Additionally, added a check to prevent specifying both netuid and all_netuids options simultaneously, exiting with an error message if both are given.
thewhaleking
approved these changes
Sep 4, 2024
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Refactor indentation for readability
Adjusted indentation in
stake.pyto maintain consistent code formatting. This change addresses alignment issues in functions, argument lists, and block structures, enhancing the readability and maintainability of the code.