chia-blockchain
chia-blockchain copied to clipboard
Ak.fee estimation integration rebase main sept 15
Combined fee estimator branch.
Algorithm is from: https://github.com/bitcoin/bitcoin/tree/master/src/policy
Contains FullNode / Wallet gossip of fee rate
command line:
chia show -f
This pull request introduces 1 alert when merging 737910292ed49bfd828443038840d70bd2bbd835 into 564782a76d733ea3a1e8687df85aa3638aa88896 - view on LGTM.com
new alerts:
- 1 for Unnecessary pass
Can you undo/restore the chia-blockchain-gui
submodule pin since it's not relevant to this PR? Also, it looks like it's pointing at baa47b29db10b0408c051cbf7da8f6e1f31a7b66
which is not up to date.
I asked about whether we're looking for a final review or what but haven't heard quite yet. In the mean time I'll submit this partial review with some suggestions and questions.
The plan is to have the fee estimator in the 1.6.1 release, so we'll need this backend PR to land soon so that the GUI PR can follow. This should be the final review.
This pull request has conflicts, please resolve those before we can evaluate the pull request.
Conflicts have been resolved. A maintainer will review the pull request shortly.
This pull request introduces 1 alert when merging 491ee31ee4406ec185d7d5877cfdb48f32d56658 into 5001c9030d4d6add949d89b2494bfa0b0ffa0f91 - view on LGTM.com
new alerts:
- 1 for Unnecessary pass
This pull request introduces 2 alerts when merging c141e70df6f302f1d7e83cb6ae8d5a7e3a658879 into 88cf629737c4c54a18197ffffa38fa8a68d4b694 - view on LGTM.com
new alerts:
- 1 for Unnecessary pass
- 1 for Unused import
This pull request introduces 2 alerts when merging 258e77de8c2362e0e68abc73301a705e22e72e40 into 88cf629737c4c54a18197ffffa38fa8a68d4b694 - view on LGTM.com
new alerts:
- 1 for Unnecessary pass
- 1 for Unused import
This pull request introduces 2 alerts when merging de842f0819bb62afe2d0172e6c511c48fb05baea into 88cf629737c4c54a18197ffffa38fa8a68d4b694 - view on LGTM.com
new alerts:
- 1 for Unnecessary pass
- 1 for Unused import
This pull request introduces 1 alert when merging cd197d030f040b5463948c087eace2da331c594c into 88cf629737c4c54a18197ffffa38fa8a68d4b694 - view on LGTM.com
new alerts:
- 1 for Unnecessary pass
This pull request introduces 1 alert when merging 8561a9e0ccfaa9f704171f06a758d4113e71db4c into 88cf629737c4c54a18197ffffa38fa8a68d4b694 - view on LGTM.com
new alerts:
- 1 for Unnecessary pass
This pull request introduces 1 alert when merging 9d1f3f7f464c8345ed0191d5287ff9ee94dd6589 into 88cf629737c4c54a18197ffffa38fa8a68d4b694 - view on LGTM.com
new alerts:
- 1 for Unnecessary pass
This pull request introduces 1 alert when merging df99272a719cc3a36743bb3a44bc8b5d50b3bb2e into 88cf629737c4c54a18197ffffa38fa8a68d4b694 - view on LGTM.com
new alerts:
- 1 for Unnecessary pass
This pull request has conflicts, please resolve those before we can evaluate the pull request.
Conflicts have been resolved. A maintainer will review the pull request shortly.
This pull request introduces 1 alert when merging 03a86bfc75ddb51f833e06c3726e4e5556b28baa into a5ea7fb5728a5b8b9b020ef6cec09260307ec462 - view on LGTM.com
new alerts:
- 1 for Unnecessary pass
This pull request introduces 1 alert when merging eb31c2738a50fa5617c9e6f12d59dcfa363f05e3 into a5ea7fb5728a5b8b9b020ef6cec09260307ec462 - view on LGTM.com
new alerts:
- 1 for Unnecessary pass
This pull request introduces 1 alert when merging 6dbd698758f881068ec1c22634f1bbfa8b4b153a into a5ea7fb5728a5b8b9b020ef6cec09260307ec462 - view on LGTM.com
new alerts:
- 1 for Unnecessary pass
This pull request introduces 1 alert when merging 51392046bd82b3cb6f01e6cf13e314480542e46a into a5ea7fb5728a5b8b9b020ef6cec09260307ec462 - view on LGTM.com
new alerts:
- 1 for Unnecessary pass
This pull request has conflicts, please resolve those before we can evaluate the pull request.
Conflicts have been resolved. A maintainer will review the pull request shortly.
Moved to https://github.com/Chia-Network/chia-blockchain/pull/13645