Skip to content

Conversation

moliholy
Copy link

This PR changes the current per-session reward system into a counter-checkpoint system, bringing the following features:

  • No limit on the number of sessions rewards are kept.
  • Order of magnitude more efficiency when computing the rewards.
  • Automatic system-managed compound of rewards, provided stakers have enabled the autocompounding feature.
  • Refactor of the autocompound percentage: it has been replaced by a boolean field, so that stakers compound either 0% or 100% of the funds.

@moliholy moliholy requested a review from metricaez March 19, 2025 15:08
@moliholy moliholy self-assigned this Mar 19, 2025
@moliholy moliholy changed the title Feat/optimized rewrads Optimize rewards Mar 19, 2025
@moliholy moliholy changed the title Optimize rewards Optimize rewards and enable actual autocompounding Mar 19, 2025
@moliholy moliholy force-pushed the feat/optimized-rewrads branch from 980aa74 to a9d2736 Compare March 19, 2025 16:37
@moliholy moliholy force-pushed the feat/optimized-rewrads branch from 75998e2 to 3fbfeb1 Compare March 20, 2025 17:23
@moliholy moliholy force-pushed the feat/optimized-rewrads branch from 6fa04cc to b205dee Compare March 20, 2025 17:26
@moliholy moliholy force-pushed the feat/optimized-rewrads branch from 9428300 to 9fde453 Compare March 20, 2025 17:32
@moliholy moliholy force-pushed the feat/optimized-rewrads branch from b44f16b to f9df0ac Compare March 31, 2025 21:05
@moliholy moliholy force-pushed the feat/optimized-rewrads branch from f9df0ac to 5d6fc17 Compare March 31, 2025 21:11
José Molina and others added 6 commits March 31, 2025 23:42
* Refactor autocompund test

* Added Autocompounding test

* Fix setting autocompound to false

* Added invalid origin tests

* Add more tests

---------

Co-authored-by: José Molina <[email protected]>
@moliholy moliholy marked this pull request as ready for review April 3, 2025 12:16
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