Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add dust metrics to getblockstats #3393

Open
wants to merge 1 commit into
base: 1.15.0-dev
Choose a base branch
from

Conversation

patricklodder
Copy link
Member

Addition on top of #3297, to get more relevant stats.

adds:

  • maxoutamount - highest value output
  • minoutamount - lowest value output
  • dustouts - number of outputs under our soft dust limit

Each of these ignores OP_RETURN and coinbase outputs

This makes some assumptions about getblockstats fixes. Draft until #3297 gets merged in any form.

@patricklodder patricklodder added enhancement RPC RPC related changes and questions labels Dec 30, 2023
@patricklodder patricklodder added this to the 1.14.7 milestone Dec 30, 2023
src/rpc/blockchain.cpp Outdated Show resolved Hide resolved
src/validation.cpp Outdated Show resolved Hide resolved
@patricklodder

This comment was marked as outdated.

@patricklodder patricklodder removed this from the 1.14.7 milestone Feb 8, 2024
@patricklodder patricklodder changed the base branch from 1.14.7-dev to 1.15.0-dev February 28, 2024 19:29
adds:
- maxoutamount - highest value output
- minoutamount - lowest value output
- dustouts - number of outputs under our soft dust limit

Each of these ignores OP_RETURN and coinbase outputs
@patricklodder patricklodder marked this pull request as ready for review March 12, 2024 21:14
@patricklodder
Copy link
Member Author

Rebased on top of 1.15.0-dev as of ba6480d

@patricklodder patricklodder added this to 🚀 needs review in Review & merge board Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement RPC RPC related changes and questions
Projects
Review & merge board
🚀 needs review
Development

Successfully merging this pull request may close these issues.

None yet

2 participants