Merged
Conversation
…126) * Ask for wallet name for `root my-delegates` only if `--all` is not specified. If `--all` is specified, ask for the path. * Add validation.
…ndle SSL errors. (#127)
# Conflicts: # bittensor_cli/cli.py # tests/e2e_tests/test_root.py
* Set archive network if network is not specified as well as if network specified as finney (through option or config) * Update tests to remove dual-use of network/chain, since they call the same thing.
* Randomise rpc_request id to avoid possible dict duplicate keys.
* root register message * set-weights help message valis on root set weights.. not just senators. * remove --wallet-name form list delegates examples I think this is covered in my-delegates, and the help text was not removed from llist-delegates * fix spelling on change
…, both for `btcli` itself, and all major commands (not subcommands). (#145) Co-authored-by: Benjamin Himes <benhimes@opentensor.dev> Co-authored-by: Benjamin Himes <37844818+thewhaleking@users.noreply.github.com>
* Added the `--help` message automatically when no command is specified, both for `btcli` itself, and all major commands (not subcommands). * Adding PyPI badge --------- Co-authored-by: Benjamin Himes <benhimes@opentensor.dev> Co-authored-by: Benjamin Himes <37844818+thewhaleking@users.noreply.github.com> Co-authored-by: Watchmaker <rajk@opentensor.dev>
* Improves set-id command * Ruff * Fixes prompt in-case of undelegation * Fixes string type * WIP * Updates tests for set-id * return False in-case of failure * Retry Prompt Logic (#151) * Added retry prompt for use in set-id to allow for checking the input as it's entered for rejection/acceptance later. * Updated helper text in set_id_prompts * Use retry_prompt in set_id_prompts * Makes pgp and twitter fields optional * Fixed indentation issue * Handles edge cases and removes prompt for is_validator in nominate + subnet create * pgp validation false * Reverts encoding in set_id --------- Co-authored-by: Benjamin Himes <37844818+thewhaleking@users.noreply.github.com> Co-authored-by: Benjamin Himes <benhimes@opentensor.dev>
thewhaleking
previously approved these changes
Oct 2, 2024
* Added the `--help` message automatically when no command is specified, both for `btcli` itself, and all major commands (not subcommands). * Adding PyPI badge * Switch Keypair import to bittensor_wallet and update tests Replace the Keypair import from substrateinterface to bittensor_wallet across multiple files. Adjust the requirements to use bittensor-wallet version 2.0.0 and add slight adjustments to test timing with sleep intervals. * add tests dir init * use relative imports * change runner params and fix grab mne * assert output of exception and stdout first * Json tuple handling + ruff format. * change empty json handling * json for regen hotkey * Asserting error in stderr * lower wait time for mod check * Fixed wallet path by ensuring that we expand the user dir, working wallet list * Always ask for WALLET.PATH because the new Wallet doesn't handle None path the same as the old one. * Updated the helper text in wallet ask to let users know they can set these values in the config. * Fixed new error (TypeError) in btwallet 2.0 causing .DS_Store * Extend wallet_path fix to regen coldkey, coldkey pub + improve text layout of hints * Add a 10 second sleep, which seems to be long enough to avoid flakiness with the test. * Testing alpha release * Bumping wallet version for testing * Wallet -> 2.0.0 * Removes comments --------- Co-authored-by: Watchmaker <rajk@opentensor.dev> Co-authored-by: opendansor <daniel@opentensor.dev> Co-authored-by: Cameron Fairchild <cameron@opentensor.ai> Co-authored-by: ibraheem-opentensor <ibraheem@opentensor.dev>
garrett-opentensor
previously approved these changes
Oct 4, 2024
opendansor
previously approved these changes
Oct 4, 2024
ibraheem-abe
dismissed stale reviews from opendansor and garrett-opentensor
via
c23d0fa
garrett-opentensor
approved these changes
Oct 4, 2024
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
5 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.
8.1.0 /2024-10-02
What's Changed
root my-delegatesask for path instead of name when using--allby @thewhaleking inroot my-delegatesask for path instead of name when using--all#126not_subtensortosubtensorby @thewhaleking in Renamenot_subtensortosubtensor#157New Contributors
Full Changelog: v8.0.0...v8.1.0