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.

Sign up for GitHub

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

Jump to bottom

feat(chainwatch): Add more complete power and reward metrics #3367

Merged
merged 2 commits into from
Aug 28, 2020
Merged

feat(chainwatch): Add more complete power and reward metrics #3367

merged 2 commits into from
Aug 28, 2020

Conversation

placer14
Copy link
Contributor

@placer14 placer14 commented Aug 28, 2020 •
edited
Loading

Merge #3275, this then PR.

Add full Power state metrics

+       totalRawBytes                      big.Int
+       totalRawBytesCommitted             big.Int
+       totalQualityAdjustedBytes          big.Int
+       totalQualityAdjustedBytesCommitted big.Int
+       totalPledgeCollateral              big.Int
+       newRawBytes             big.Int
+       newQualityAdjustedBytes big.Int
+       newPledgeCollateral     big.Int
+       newQAPowerSmoothed      *smoothing.FilterEstimate
+       minerCount                  int64
+       minerCountAboveMinimumPower int64

add full Reward state metrics

+       cumSumBaselinePower big.Int
+       cumSumRealizedPower big.Int
+       effectiveNetworkTime   int64
+       effectiveBaselinePower big.Int
+       newBaselinePower     big.Int
+       newBaseReward        big.Int
+       newSmoothingEstimate *smoothing.FilterEstimate
+       totalMinedReward big.Int

@placer14 placer14 added the kind/enhancement Kind: Enhancement label Aug 28, 2020
@placer14 placer14 requested a review from frrist August 28, 2020 00:12
@placer14 placer14 self-assigned this Aug 28, 2020
@placer14 placer14 force-pushed the feat/chainwatch/add-power-reward-metrics branch 3 times, most recently from ef30f29 to 4807916 Compare August 28, 2020 02:05
frrist
frrist approved these changes Aug 28, 2020
View reviewed changes
Copy link
Member

@frrist frrist left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assuming manual validation passes LGTM

@placer14 placer14 force-pushed the feat/chainwatch/add-power-reward-metrics branch 2 times, most recently from 2b3f554 to 2317125 Compare August 28, 2020 02:58
@placer14 placer14 changed the base branch from master to fix/chainwatch/sync August 28, 2020 02:58
@placer14 placer14 force-pushed the feat/chainwatch/add-power-reward-metrics branch 2 times, most recently from 3661085 to 5caafcb Compare August 28, 2020 03:16
@placer14
fix(chainwatch): Upsert on chain_economics
ff5ac51 
This table may get multiple updates depending on how many blocks get
added into a tipset. Each new block affects the tipset state, but the
parent state root will remain the same. This allows the very latest
value to be applied.
@placer14 placer14 force-pushed the feat/chainwatch/add-power-reward-metrics branch from 5caafcb to ff5ac51 Compare August 28, 2020 03:21
@placer14 placer14 marked this pull request as ready for review August 28, 2020 03:27
@magik6k magik6k merged commit fa6f9f4 into fix/chainwatch/sync Aug 28, 2020
@magik6k magik6k deleted the feat/chainwatch/add-power-reward-metrics branch August 28, 2020 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@frrist frrist frrist approved these changes

@magik6k magik6k magik6k approved these changes

Assignees

@placer14 placer14

Labels
kind/enhancement Kind: Enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@placer14 @magik6k @frrist