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

Import/Export simulation fails #83

Closed
noandrea opened this issue Apr 4, 2022 · 1 comment
Closed

Import/Export simulation fails #83

noandrea opened this issue Apr 4, 2022 · 1 comment
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@noandrea
Copy link
Contributor

noandrea commented Apr 4, 2022

We have an error in the simulation for the import/export of the state, here are some details:

the issue seems somewhat related to these two (fairly old) issues in the cosmos-SDK:

when we run the TestAppSimulationAfterImport the result is panic:

panic: calculated final stake for delegator elesto1qytdwn45zw6rjrstm6zzjvl2x2ck9jfq0yygtv greater than current stake
final stake:	469303483982.000000000000000000
current stake:	424203504734.191994867366789693 [recovered]

the code is here: https://github.com/elesto-dao/elesto/blob/main/app/simulation_test.go

a note is when we run a single simulation setting the seed to 100_000_000_000 the simulation ends successfully

@noandrea noandrea added the bug Something isn't working label Apr 4, 2022
@noandrea noandrea changed the title Inport/Export simulation fails Import/Export simulation fails Apr 11, 2022
@minkyraccoon minkyraccoon added the help wanted Extra attention is needed label Apr 28, 2022
@noandrea noandrea assigned PaddyMc, gsora and noandrea and unassigned noandrea and PaddyMc May 31, 2022
@gsora
Copy link
Contributor

gsora commented Jun 3, 2022

#123 should address the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants