* Making source code cleaner
* Cleaned up `install_plotter.py`
* Now `$CHIA_ROOT/plotters/bladebit` is searched first then bundled bladebit
* Fixed a lint error
* WIP: Implement bladebit update/checkout feature
* Enable to specify git branch/tag/commit on bladebit installation
* Modify arguments display order
* Added `chia plotters version` command
* Fixed lint errors
* Fixed lint error 2
* Implemented `chia plotters bladebit2` and handling for its options
* Fixed typo
* Fixed a lint error
* Added comment
* Added missing plotter arguments
* Fixed LGTM warning
* Fixed wrong typing
* Fixed a lint error
* Fixed an issue where `None` was evaluated as an empty string
* Support bladebit2 plotting from web request
* Updated response of `get_plotters` API for upcoming `bladebit2`
* Fixed bladebit2 argument handling
* Modified progress dict
* Added --no-cpu-affinity option to bladebit1
* Fixed a minor error
* Fixed an error where `chia plotters version` failed
* Fixed an issue where building bladebit2 failed
* Fixed an issue where bladebit2 stopped working if `--cache` option was set
* Updated branch for bladebit2
* Fixed an issue where plotting with madmax/bladebit failed for Python < 3.8 on Windows
* Now `chia plotters install madmax|bladebit2` should work on RedHat like OSes
* Fixed a wrong yum package name
* Added yum packages required for Bladebit2
* Changed the way to update bladebit
* Implemented option handling for madmax as well as bladebit
* Fixed a lint error
* Fixed a minor issue
* Fixed LGTM warning
* Applied changes for bladebit 2.0.0-alpha-2
* Fixed a lint error
* Trying to fix windows pre-commit error
* Fixed isort error
* Make IP section in connections table 1 character wider
Note how the table spacing is one character short of handling a full `(8 * 4) + 7` characters for an IPv6 address.
```
$ chia wallet show
Wallet height: 0
Sync status: Not synced
Connections:
Type IP Ports NodeID Last Connect MiB Up|Dwn
FULL_NODE 2803:xxxx:xxxx:xxxx:xxxx:xxxx:xxxx:573d 8444/8444 353e724c... Aug 25 23:00:09 0.1|4.6
-Height: No Info -Hash: No Info
FULL_NODE 66.xxx.xx.161 8444/8444 0582d93f... Aug 25 23:00:12 0.0|0.0
-Height: No Info -Hash: No Info
FULL_NODE 127.0.0.1 8444/8444 ea10c38b... Aug 25 23:00:12 0.0|0.0
-Height: No Info -Hash: No Info -Trusted: True
```
* more 1 characters
* provide a skeleton hint protocol for rpc apis
* more
* more self.custom_get_connections = None
* fix RpcApiProtocol.service hinting
* catch up with RpcApiProtocol._state_changed()
* FullNode.server is now non-optional. 'ish
* drop another todo
* custom_get_connections -> get_connections
* just make get_connections() a mandatory method with a helper provided
* lots
* try again
* ._start_with_fingerprint()
* fill out the Service generic parameter in start scripts
* ... instead of pass
* add docstrings to the protocols
* remove no longer needed hasattr/not None checks
* remove outdated comment
* refactor Harvester._is_shutdown to ._shut_down
* add final coverage handling
* switch to development coverage
* correct url
* only run one directory for now
* only 3.10
* delete windows data for now
* drop the 35% failure point
* recover full test matrix
* Use regular coverage
* Update test.yml
* put coverage reporting into an artifact
* oops
* oopsy
* pytest-cov
* maybe
* maybe help out flakes
* farm the spend bundle...
* some more
* more funds for the did test?
* more...
* report chia/ and tests/ separately
* job_timeout = 45 for core
* tidy and disable diff-cover
* remove job_timeout = 45
* --rcfile=.coveragerc
* Update .github/workflows/test.yml
* revert time and blocks
* assert spend is in mempool, lower times and block count
* remove sleep
* lint
* up timeout
* cat test - farm blocks until spend is included
* add spend check for nft endpoints
* fix nft endpoint test
* remove unused code
* 4
Co-authored-by: almog <almogdepaz@gmail.com>
* make a WalletProtocol, defining the shared API of all wallets and fix type checking
* add protocol check for DataLayerWallet. remove outdated comment
* Extend `is_running` command to support checking multiple/all services
* Strip whitespace around service names.
Added tests for is_running()
* mypy
* Added `running_services` command to list all running services
Refactored `is_running` in terms of a common `is_service_running` fn
Added tests for the new/updated functions
* mypy
* Update chia/daemon/server.py
Co-authored-by: Kyle Altendorf <sda@fstab.net>
* Update chia/daemon/server.py
Co-authored-by: Kyle Altendorf <sda@fstab.net>
* Removed option to specify specific services to query.
Addressed PR feedback.
Co-authored-by: Kyle Altendorf <sda@fstab.net>
When claiming rewards for a self-pooling plotnft, the end user will see multiple "ALREADY_INCLUDING_TRANSACTION" errors on the console (if using the CLI) and printed in the log (both CLI and GUI)
This is because the action for claiming creates multiple individual transactions, and adding one transaction can cause the other transaction already in the pending list to be resent.
Adjust the pending tx handler to only get called in cases where there is a tx.spend_bundle present