Skip to content

Conversation

@pietrodimarco-dfinity
Copy link
Contributor

@pietrodimarco-dfinity pietrodimarco-dfinity commented Feb 24, 2025

Node Provider Rewards V1 Library
This library computes performance-based rewards (in XDR) for node providers over a given reward period. Rewards are adjusted based on each node's performance using trustworthy node metrics. The implementation follows the accepted proposal 135054.

This PR implement the first step of the reward computation process:

  • Add PerformanceMultipliers computation for each node based on its failure rate, derived from the number of blocks it failed to propose vs. successfully proposed.
  • Includes tests for both the PerformanceMultipliers computation and library inputs.
  • Adds detailed logging to track each step of the computation process.

Next Steps (TODO)
The next phase will apply the computed PerformanceMultipliers to the monthly XDR incentives each node receives for participating in the IC network.

@pietrodimarco-dfinity pietrodimarco-dfinity requested a review from a team as a code owner February 24, 2025 12:20
@pietrodimarco-dfinity pietrodimarco-dfinity marked this pull request as draft February 24, 2025 12:20
@pietrodimarco-dfinity pietrodimarco-dfinity changed the title Add NPR V1 Add NPR V1 lib Feb 24, 2025
@pietrodimarco-dfinity pietrodimarco-dfinity changed the title Add NPR V1 lib feat: IC-1680: Add NPR V1 lib Feb 24, 2025
@pietrodimarco-dfinity pietrodimarco-dfinity changed the title feat: IC-1680: Add NPR V1 lib feat: IC-1680: Add NPR V1 lib - Performance Multiplier Calculator Feb 24, 2025
@pietrodimarco-dfinity pietrodimarco-dfinity marked this pull request as ready for review February 25, 2025 18:35
@pietrodimarco-dfinity pietrodimarco-dfinity enabled auto-merge (squash) March 4, 2025 17:13
@pietrodimarco-dfinity pietrodimarco-dfinity merged commit b5eecf3 into main Mar 4, 2025
5 checks passed
@pietrodimarco-dfinity pietrodimarco-dfinity deleted the pmarco/add-rewards-multiplier-lib branch March 4, 2025 17:15
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