Skip to content

Comments

Change spec to use criminal's incomes as additional rewards#1562

Merged
mergify[bot] merged 2 commits intoCodeChain-io:masterfrom
sgkim126:fee-spec
May 31, 2019
Merged

Change spec to use criminal's incomes as additional rewards#1562
mergify[bot] merged 2 commits intoCodeChain-io:masterfrom
sgkim126:fee-spec

Conversation

@sgkim126
Copy link
Contributor

Hi,
Currently, the informant receives deposit and rewards of the criminal as a prize.
However, it incentives for informants to wait for reporting to maximize their profits.

So I propose giving the deposit to the informant and using the incomes as additional rewards for diligent validators.

@sgkim126
Copy link
Contributor Author

sgkim126 commented May 28, 2019

FYI, intermediate_rewards is cleared at the beginning of the term, not the end of the term, because it needs the signature of the last block in the term needs to calculate the penalty.

majecty
majecty previously approved these changes May 28, 2019
Copy link

@majecty majecty left a comment

Choose a reason for hiding this comment

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

LGTM

@kseo
Copy link
Member

kseo commented May 28, 2019

LGTM. Let's wait for @ScarletBlue's review.

foriequal0
foriequal0 previously approved these changes May 28, 2019
kseo
kseo previously approved these changes May 28, 2019
@kseo kseo requested review from foriequal0, kseo and majecty May 28, 2019 05:31
@sgkim126 sgkim126 dismissed stale reviews from kseo, foriequal0, and majecty via 0b640d2 May 30, 2019 06:15
@mergify mergify bot merged commit ab9fdcb into CodeChain-io:master May 31, 2019

The informant receives all deposit and rewards(TBD) as prize money immediately.
The informant receives the deposit of the criminal as prize money immediately.
The express fee that the criminal would earn is used as additional rewards for diligent validators.
Copy link
Contributor

Choose a reason for hiding this comment

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

The express fee that the criminal would have earned is used as additional rewards for diligent validators.

@sgkim126 sgkim126 deleted the fee-spec branch June 5, 2019 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants