chia-blockchain icon indicating copy to clipboard operation
chia-blockchain copied to clipboard

Coin Selection CLI / RPC Expansion

Open jack60612 opened this issue 1 year ago • 15 comments

Add New Cli Commands...

  • [x] chia wallet coins list: List all coins
  • [x] chia wallet coins combine: allow users to manage dust and aggregate smaller coins.
  • [x] chia wallet coins split: Allow users to split a larger coin into a large amount of smaller coins.

Also:

  • [x] expand existing RPC's with max coin amount
  • [x] add ability to exclude specific coin when using chia wallet send
  • [x] Add RPC that outputs all coins that pass a set of filters.
  • [x] Add wallet coin record rpc & various coin record functions that interact between datastores

Please test the commands if you can!

jack60612 avatar Sep 04 '22 04:09 jack60612

This pull request introduces 2 alerts when merging 3fbd34c387098dc1fe5679dd9988810d5e927b79 into 0d78058daa09715741c596e0cace129b59fb7083 - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Sep 04 '22 04:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 6e6ffff671aa6823027447c423e39ae8d75117bf into 0d78058daa09715741c596e0cace129b59fb7083 - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Sep 04 '22 16:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging e3a0f4f75d1c84a3f0b3e55e40f831e11a183bdb into 0d78058daa09715741c596e0cace129b59fb7083 - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Sep 04 '22 17:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 93c7ac364b51f50151697527404df5fcd0d9129e into 0d78058daa09715741c596e0cace129b59fb7083 - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Sep 05 '22 00:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 85358c203f8431d5439de647fb676496f27805c1 into 0d78058daa09715741c596e0cace129b59fb7083 - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Sep 05 '22 02:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 04f94f34b2826ce372565d90e76b2416ba58da0b into 6543778ce3b10f6483bad6cfcfb023d84f4e1bd2 - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Sep 07 '22 22:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 515ea9c117218a26da21281750c0766d0efe68f2 into dd6b0962d2c7e601f04cfe36e04089ae6c0cc8a5 - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Sep 08 '22 02:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging b2d0e7dd0b21a26c4d36fcde0de8cf4ba1b783bf into dd6b0962d2c7e601f04cfe36e04089ae6c0cc8a5 - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Sep 08 '22 03:09 lgtm-com[bot]

Tried running with: chia wallet coins split -a 0.1

Exception from 'wallet' 'NoneType' object has no attribute 'startswith':
Traceback (most recent call last):
  File "/Users/trepca/dev/chia-blockchain/chia/cmds/cmds_util.py", line 86, in get_any_service_client
    yield node_client, config, fingerprint
  File "/Users/trepca/dev/chia-blockchain/chia/cmds/cmds_util.py", line 167, in execute_with_wallet
    await function(extra_params, wallet_client, new_fp)
  File "/Users/trepca/dev/chia-blockchain/chia/cmds/coin_funcs.py", line 125, in async_split
    target_coin_id: bytes32 = bytes32.from_hexstr(args["target_coin_id"])
  File "/Users/trepca/dev/chia-blockchain/chia/util/byte_types.py", line 49, in from_hexstr
    if input_str.startswith("0x") or input_str.startswith("0X"):
AttributeError: 'NoneType' object has no attribute 'startswith'

trepca avatar Sep 08 '22 13:09 trepca

This pull request introduces 2 alerts when merging 8083280dda4cbae285a69ac6b9c30e39f5c7cf89 into 87a3e93cc40ab71150e5f9015937ef32763a575e - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Sep 08 '22 18:09 lgtm-com[bot]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Sep 13 '22 23:09 github-actions[bot]

Conflicts have been resolved. A maintainer will review the pull request shortly.

github-actions[bot] avatar Sep 13 '22 23:09 github-actions[bot]

This pull request introduces 2 alerts when merging 194b9fea4b0c7cd0f67cec9ec4deadac94141b8d into 2de55aff17303980fea5b60633eefeaa441aa83e - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Sep 13 '22 23:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 42c18db62823256aab69c33e86418e0ac65059c1 into 2de55aff17303980fea5b60633eefeaa441aa83e - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Sep 14 '22 00:09 lgtm-com[bot]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Sep 16 '22 01:09 github-actions[bot]

Conflicts have been resolved. A maintainer will review the pull request shortly.

github-actions[bot] avatar Sep 28 '22 19:09 github-actions[bot]

This pull request introduces 2 alerts when merging de6f50890a3930c03638db33656926a70f664b9f into 88cf629737c4c54a18197ffffa38fa8a68d4b694 - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Sep 28 '22 19:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 3b0b6ea2feac67c2cd2864daf0e87435916dea2f into 88cf629737c4c54a18197ffffa38fa8a68d4b694 - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Sep 28 '22 20:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 21b5eaa2f14a5a62a81d800d7988a091aa116200 into 88cf629737c4c54a18197ffffa38fa8a68d4b694 - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Sep 28 '22 22:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 41a200488e6cd37ce4bd71f6335234f31509a687 into 88cf629737c4c54a18197ffffa38fa8a68d4b694 - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Sep 29 '22 16:09 lgtm-com[bot]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Sep 30 '22 08:09 github-actions[bot]

Conflicts have been resolved. A maintainer will review the pull request shortly.

github-actions[bot] avatar Oct 02 '22 16:10 github-actions[bot]

This pull request introduces 2 alerts when merging fcec7ba6d65e413dacc31bd1415a8385c7390069 into a5ea7fb5728a5b8b9b020ef6cec09260307ec462 - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Oct 02 '22 16:10 lgtm-com[bot]

This pull request introduces 2 alerts when merging 6bfa7bb9047a5e44dab144fe531e354be1301080 into a5ea7fb5728a5b8b9b020ef6cec09260307ec462 - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Oct 02 '22 17:10 lgtm-com[bot]

This pull request introduces 2 alerts when merging e35a9884d1bc6e6fcfa3786cb3fd255de6906596 into 76a78ed180ca9cc4d07c155cf45d7679ad11c64a - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Oct 06 '22 21:10 lgtm-com[bot]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar Oct 09 '22 01:10 github-actions[bot]

Conflicts have been resolved. A maintainer will review the pull request shortly.

github-actions[bot] avatar Oct 10 '22 13:10 github-actions[bot]

This pull request introduces 2 alerts when merging f307b8fd328a0379c001ee8d70493c270be5852a into 71d33eb86b81dd29282ec6001bdeda020e0635ab - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Oct 10 '22 13:10 lgtm-com[bot]

This pull request introduces 2 alerts when merging c87855d72cdc1490d76cef2f1ccb4aada245c282 into 71d33eb86b81dd29282ec6001bdeda020e0635ab - view on LGTM.com

new alerts:

  • 2 for Module-level cyclic import

lgtm-com[bot] avatar Oct 10 '22 14:10 lgtm-com[bot]