Skip to content

[N3] Flat rewards (21 first nodes) #4312

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

Open
shargon wants to merge 3 commits into
base: master-n3
Choose a base branch
from
+31 −8
Open

[N3] Flat rewards (21 first nodes) #4312

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
0b775b1
Flat rewards (21)
shargon Nov 14, 2025
ec89f82
Update src/Neo/SmartContract/Native/NeoToken.cs
shargon Dec 12, 2025
508c11f
Optimize previous change
shargon Dec 12, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
39 changes: 31 additions & 8 deletions src/Neo/SmartContract/Native/NeoToken.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -164,17 +164,17 @@ private BigInteger CalculateBonus(DataCache snapshot, NeoAccountState state, uin
// Compute Neo holder reward

// In the unit of datoshi, 1 GAS = 10^8 datoshi
BigInteger sumNeoHold = 0;
BigInteger sumGasPerBlock = 0;
foreach (var (index, gasPerBlock) in GetSortedGasRecords(snapshot, end - 1))
{
if (index > start)
{
sumNeoHold += gasPerBlock * (end - index);
sumGasPerBlock += gasPerBlock * (end - index);
end = index;
}
else
{
sumNeoHold += gasPerBlock * (end - start);
sumGasPerBlock += gasPerBlock * (end - start);
break;
}
}
Expand All @@ -190,7 +190,7 @@ private BigInteger CalculateBonus(DataCache snapshot, NeoAccountState state, uin
voteReward = state.Balance * (latestGasPerVote - state.LastGasPerVote) / VoteFactor;
}

return (state.Balance * sumNeoHold * NeoHolderRewardRatio / 100 / TotalAmount, voteReward);
return (state.Balance * sumGasPerBlock * NeoHolderRewardRatio / 100 / TotalAmount, voteReward);
}

private void CheckCandidate(DataCache snapshot, ECPoint pubkey, CandidateState candidate)
Expand Down Expand Up @@ -272,18 +272,41 @@ internal override async ContractTask PostPersistAsync(ApplicationEngine engine)

if (ShouldRefreshCommittee(engine.PersistingBlock.Index, m))
{
BigInteger voterRewardOfEachCommittee = gasPerBlock * VoterRewardRatio * VoteFactor * m / (m + n) / 100; // Zoom in VoteFactor times, and the final calculation should be divided VoteFactor
var withFlatRewards = engine.IsHardforkEnabled(Hardfork.HF_Faun);
BigInteger voterRewardOfEachCommittee =
withFlatRewards ?
gasPerBlock * VoterRewardRatio * VoteFactor / 100 : // Total reward for voters when it's flat rewards
gasPerBlock * VoterRewardRatio * VoteFactor * m / (m + n) / 100; // Zoom in VoteFactor times, and the final calculation should be divided VoteFactor
var votersCount = (BigInteger)engine.SnapshotCache[_votersCount];

if (withFlatRewards && votersCount.IsZero)
{
// Skip reward distribution when there are no voters
return;
}

for (index = 0; index < committee.Count; index++)
{
var (publicKey, votes) = committee[index];
var factor = index < n ? 2 : 1; // The `voter` rewards of validator will double than other committee's
if (votes > 0)

if (withFlatRewards)
{
BigInteger voterSumRewardPerNEO = factor * voterRewardOfEachCommittee / votes;
BigInteger voterSumRewardPerNEO = voterRewardOfEachCommittee / votersCount;
Copy link
Member

@vncoelho vncoelho Nov 14, 2025

Choose a reason for hiding this comment

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

This is another change apart of flat rewards, this also set a defined level among the committee itself

Copy link
Member

@vncoelho vncoelho Nov 14, 2025

Choose a reason for hiding this comment

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

I see now, @shargon, one main point to change here is not just this factor but it is the votersCount.

We should not consider votersCount for each committee reward, that is what removes the incentive for free voting.

StorageKey voterRewardKey = CreateStorageKey(Prefix_VoterRewardPerCommittee, publicKey);
StorageItem lastRewardPerNeo = engine.SnapshotCache.GetAndChange(voterRewardKey, () => new StorageItem(BigInteger.Zero));
lastRewardPerNeo.Add(voterSumRewardPerNEO);
}
else
{
if (votes > 0)
{
var factor = index < n ? 2 : 1; // The `voter` rewards of validator will double than other committee's
BigInteger voterSumRewardPerNEO = factor * voterRewardOfEachCommittee / votes;
StorageKey voterRewardKey = CreateStorageKey(Prefix_VoterRewardPerCommittee, publicKey);
StorageItem lastRewardPerNeo = engine.SnapshotCache.GetAndChange(voterRewardKey, () => new StorageItem(BigInteger.Zero));
lastRewardPerNeo.Add(voterSumRewardPerNEO);
}
}
}
}
}
Expand Down
Loading