Skip to content

[Do not merge] Add Dec 1st 2025 TACo rewards distribution#18

Closed
manumonti wants to merge 2 commits intomainfrom
dec-1-dist
Closed

[Do not merge] Add Dec 1st 2025 TACo rewards distribution#18
manumonti wants to merge 2 commits intomainfrom
dec-1-dist

Conversation

@manumonti
Copy link

Adds the calculated rewards distribution and updates the latest distribution file to point to this new distribution.

Reviews needed: 3

@manumonti manumonti self-assigned this Dec 4, 2025
@manumonti
Copy link
Author

CI checks are failing because CI is being run in a incompatible version of node (I guess). But locally the tests pass:

▶ npm run check-distribution

> [email protected] check-distribution
> mocha scripts/test/check_distribution.test.js



  Checking last TACo rewards distribution
    Checking calculations of the last distribution
      ✔ latest and the last distribution files should be identical
      ✔ should total distribution amounts be correctly calculated
      ✔ should number of claims be equal or greater than in the previous distribution
      ✔ should all previous stakes be present in the latest distribution
      ✔ should accumulated amount of each stake be correctly calculated
      ✔ should the Merkle proof verification pass for all the stakes
    Checking known stakes
      ✔ should known stakes receive expected rewards
      ✔ should beta stakers not receive rewards
    Checking penalizations
      ✔ should infractors be penalized

  Checking 15M T cap for eligible stake
    ✔ should rewards for any stake not exceed the corresponding to 15M


  10 passing (19ms)

@derekpierre
Copy link
Member

CI checks are failing because CI is being run in a incompatible version of node (I guess). But locally the tests pass:

What version of node is needed / being used locally? Can we update the gh-action - https://github.com/nucypher/taco-rewards/blob/main/.github/workflows/check_distribution.yaml#L20?

@manumonti
Copy link
Author

CI checks are failing because CI is being run in a incompatible version of node (I guess). But locally the tests pass:

What version of node is needed / being used locally? Can we update the gh-action - https://github.com/nucypher/taco-rewards/blob/main/.github/workflows/check_distribution.yaml#L20?

@derekpierre

Done in: #20

I'll merge #20 and then I'll rebase this PR to see if that works 👍

This commit adds the calculated rewards distribution and updates the
latest distribution file to point to this new distribution.
Copy link
Member

@derekpierre derekpierre left a comment

Choose a reason for hiding this comment

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

🎸

@manumonti
Copy link
Author

CI checks are failing because CI is being run in a incompatible version of node (I guess). But locally the tests pass:

What version of node is needed / being used locally? Can we update the gh-action - https://github.com/nucypher/taco-rewards/blob/main/.github/workflows/check_distribution.yaml#L20?

@derekpierre that worked! thanks

@manumonti manumonti changed the title Add Dec 1st 2025 TACo rewards distribution [Do not merge] Add Dec 1st 2025 TACo rewards distribution Dec 4, 2025
@manumonti
Copy link
Author

manumonti commented Dec 4, 2025

It seems there are some possible miscalculations. Investigating about it...

@derekpierre
Copy link
Member

derekpierre commented Dec 10, 2025

@manumonti, seems like a CI check failed.

Seems that 0xa2e44666BAB3ED3be4F87b7c18eD2ffB911c0bEf failed 4 heartbeats and therefore receives 0 rewards.

@manumonti
Copy link
Author

Discarding because related PR #21 is being discarded and we're just going to use the existing calculation mechanism for November rewards.

@manumonti manumonti closed this Dec 11, 2025
@manumonti manumonti deleted the dec-1-dist branch December 11, 2025 13:44
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