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

Change punishment transfer memo #120

Merged
merged 7 commits into from
Dec 27, 2019

Conversation

Alladin9393
Copy link
Collaborator

Change Description

  • missed blocks and pct missed blocks

Example

punishment transfer: missed blocks - 116 (4.221200%)

@Alladin9393 Alladin9393 force-pushed the feature/change_punishment_transfer_memo branch from c398de8 to dd8589d Compare December 23, 2019 17:14
@Alladin9393 Alladin9393 force-pushed the feature/change_punishment_transfer_memo branch from b17c5eb to 2a8edfe Compare December 26, 2019 18:37
}
if ( punishment > 0 ) {
double pct_missed_blocks = 100 - (prod.unpaid_blocks / double(expected_produced_blocks)) * 100;
string punishment_memo = "punishment transfer: missed blocks - " + std::to_string(expected_produced_blocks - prod.unpaid_blocks) + " (" + std::to_string(pct_missed_blocks) + "%)";

Choose a reason for hiding this comment

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

: missed XXX blocks out of YYY

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@roman-tik, done.

@Alladin9393 Alladin9393 force-pushed the feature/change_punishment_transfer_memo branch 2 times, most recently from 29de6da to 289960a Compare December 27, 2019 12:43
@Alladin9393 Alladin9393 force-pushed the feature/change_punishment_transfer_memo branch from 289960a to c9163cb Compare December 27, 2019 15:01
@roman-tik roman-tik merged commit 93622ca into develop Dec 27, 2019
@roman-tik roman-tik deleted the feature/change_punishment_transfer_memo branch December 27, 2019 21:27
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.

2 participants