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

WIP: use block power throughout distribution (II) #2552

Merged

Conversation

jaekwon
Copy link
Contributor

@jaekwon jaekwon commented Oct 22, 2018

Replaces #2541.

@jaekwon jaekwon force-pushed the jae/check-supply-in-simulation branch from 1164c9e to 1cc7432 Compare October 22, 2018 08:47
@alexanderbez
Copy link
Contributor

Can we close #2541 then?

GetValidator() ValAddress // validator operator address
GetShares() Dec // amount of validator's shares held in this delegation
GetDelegatorAddr() AccAddress // delegator AccAddress for the bond
GetValidatorAddr() ValAddress // validator operator address
Copy link
Contributor

Choose a reason for hiding this comment

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

We should keep renaming as a separate PR - muddies being able to view the core changes

Copy link
Contributor

Choose a reason for hiding this comment

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

even though ++ on the change here - leave for now

@codecov
Copy link

codecov bot commented Oct 22, 2018

Codecov Report

Merging #2552 into cwgoes/check-supply-in-simulation will increase coverage by 0.04%.
The diff coverage is 73.4%.

@@                          Coverage Diff                          @@
##           cwgoes/check-supply-in-simulation    #2552      +/-   ##
=====================================================================
+ Coverage                              59.84%   59.89%   +0.04%     
=====================================================================
  Files                                    150      150              
  Lines                                   8632     8692      +60     
=====================================================================
+ Hits                                    5166     5206      +40     
- Misses                                  3112     3130      +18     
- Partials                                 354      356       +2

@rigelrozanski rigelrozanski merged commit e088c8b into cwgoes/check-supply-in-simulation Oct 22, 2018
@rigelrozanski rigelrozanski deleted the jae/check-supply-in-simulation branch October 22, 2018 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants