Skip to content
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.

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

[VS Incentives]: distribution test for changing volume between epochs #6598

Merged
merged 1 commit into from
Sep 29, 2023

Conversation

p0mvn
Copy link
Member

@p0mvn p0mvn commented Sep 29, 2023

Progress towards: #6559

What is the purpose of the change

Adds test for group epoch distribution. This test focuses on volume being updated across epochs.

Made some refactors to other tests to address a rounding off-by-one issue in estimating test amounts that came up.

@p0mvn p0mvn added V:state/compatible/no_backport State machine compatible PR, depends on prior breaks A:no-changelog labels Sep 29, 2023
@p0mvn p0mvn marked this pull request as ready for review September 29, 2023 16:12
Copy link
Member

@czarcas7ic czarcas7ic left a comment

Choose a reason for hiding this comment

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

Nice 👍

@p0mvn p0mvn merged commit 6f2d9a5 into main Sep 29, 2023
@p0mvn p0mvn deleted the roman/epoch-distr-test branch September 29, 2023 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A:no-changelog C:x/incentives V:state/compatible/no_backport State machine compatible PR, depends on prior breaks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants