Concordium / concordium-node

The main concordium node implementation.
GNU Affero General Public License v3.0
45 stars 22 forks source link

GetBakersRewardPeriod gives incorrect data #1176

Closed td202 closed 1 month ago

td202 commented 4 months ago

Bug Description

The GetBakersRewardPeriod gives incorrect data concerning the pool commission rates, baker equity capital and delegated capital. The values presented are based on the active staking, whereas they should be based on the staking that applies to the current epoch (i.e. the snapshot of the active values that was taken one epoch before the last payday).

Steps to Reproduce

Compare the results from GetBakersRewardPeriod (aka GetValidatorsRewardPeriod in the client) and GetPoolInfo.

Expected Result

Actual Result

Versions

td202 commented 4 months ago

This will be addressed as part of the API changes for cooldown (#1146).