Skip to content

Commit

Permalink
Merge pull request #2140 from opentensor/fix/extra-no-prompts
Browse files Browse the repository at this point in the history
Removes extra no_prompts in commands
  • Loading branch information
ibraheem-opentensor authored Jul 16, 2024
2 parents 9f99bd4 + 1433092 commit 67daf09
Show file tree
Hide file tree
Showing 4 changed files with 20 additions and 41 deletions.
12 changes: 4 additions & 8 deletions bittensor/commands/metagraph.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,11 @@
# DEALINGS IN THE SOFTWARE.

import argparse
import bittensor

from rich.table import Table

import bittensor

from .utils import check_netuid_set

console = bittensor.__console__ # type: ignore
Expand Down Expand Up @@ -261,12 +264,5 @@ def add_args(parser: argparse.ArgumentParser):
help="""Set the netuid to get the metagraph of""",
default=False,
)
metagraph_parser.add_argument(
"--no_prompt",
dest="no_prompt",
action="store_true",
help="""Set true to avoid prompting the user.""",
default=False,
)

bittensor.subtensor.add_args(metagraph_parser)
14 changes: 0 additions & 14 deletions bittensor/commands/network.py
Original file line number Diff line number Diff line change
Expand Up @@ -534,13 +534,6 @@ def add_args(parser: argparse.ArgumentParser):
parser.add_argument(
"--netuid", dest="netuid", type=int, required=False, default=False
)
parser.add_argument(
"--no_prompt",
dest="no_prompt",
action="store_true",
help="""Set true to avoid prompting the user.""",
default=False,
)
bittensor.subtensor.add_args(parser)


Expand Down Expand Up @@ -639,13 +632,6 @@ def add_args(parser: argparse.ArgumentParser):
parser.add_argument(
"--netuid", dest="netuid", type=int, required=False, default=False
)
parser.add_argument(
"--no_prompt",
dest="no_prompt",
action="store_true",
help="""Set true to avoid prompting the user.""",
default=False,
)
bittensor.subtensor.add_args(parser)


Expand Down
14 changes: 8 additions & 6 deletions bittensor/commands/wallets.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,15 +16,18 @@
# DEALINGS IN THE SOFTWARE.

import argparse
import bittensor
import os
import sys
from rich.prompt import Prompt, Confirm
from rich.table import Table
from typing import Optional, List, Tuple
from . import defaults
from typing import List, Optional, Tuple

import requests
from rich.prompt import Confirm, Prompt
from rich.table import Table

import bittensor

from ..utils import RAOPERTAO
from . import defaults


class RegenColdkeyCommand:
Expand Down Expand Up @@ -637,7 +640,6 @@ class UpdateWalletCommand:
Optional arguments:
- ``--all`` (bool): When set, updates all legacy wallets.
- ``--no_prompt`` (bool): Disables user prompting during the update process.
Example usage::
Expand Down
21 changes: 8 additions & 13 deletions bittensor/wallet.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,13 +17,15 @@
# OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
# DEALINGS IN THE SOFTWARE.

import os
import copy
import argparse
import bittensor
from termcolor import colored
import copy
import os
from typing import Dict, Optional, Tuple, Union, overload

from substrateinterface import Keypair
from typing import Optional, Union, Tuple, Dict, overload
from termcolor import colored

import bittensor
from bittensor.utils import is_valid_bittensor_address_or_public_key


Expand Down Expand Up @@ -138,18 +140,11 @@ def add_args(cls, parser: argparse.ArgumentParser, prefix: str = None):
parser (argparse.ArgumentParser): Argument parser object.
prefix (str): Argument prefix.
"""
prefix_str = "" if prefix == None else prefix + "."
prefix_str = "" if prefix is None else prefix + "."
try:
default_name = os.getenv("BT_WALLET_NAME") or "default"
default_hotkey = os.getenv("BT_WALLET_NAME") or "default"
default_path = os.getenv("BT_WALLET_PATH") or "~/.bittensor/wallets/"
parser.add_argument(
"--no_prompt",
dest="no_prompt",
action="store_true",
help="""Set true to avoid prompting the user.""",
default=False,
)
parser.add_argument(
"--" + prefix_str + "wallet.name",
required=False,
Expand Down

0 comments on commit 67daf09

Please sign in to comment.