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

Gas value for delegate messages too low #177

Closed
webmaster128 opened this issue Oct 24, 2023 · 0 comments · Fixed by #178
Closed

Gas value for delegate messages too low #177

webmaster128 opened this issue Oct 24, 2023 · 0 comments · Fixed by #178

Comments

@webmaster128
Copy link
Member

It seems like the current delegation gas value is too low in practive. Here are a few gas values on the Cosmos Hub I found:

gas used delegations autoclaims transaction
257,208 1 0 https://www.mintscan.io/cosmos/tx/7D1012DC944B9F3D312365C01C07712299075CA67B10891689D48FBD87C231F6?height=17555823
610,413 1 13 https://www.mintscan.io/cosmos/tx/1E7035ED9A0D16B5CB11A9907B8C334C1834E30802A54BE61B2078157A461F76?height=17555866
405,802 1 1 https://www.mintscan.io/cosmos/tx/93B5208F4686399F0B8A59602A37F8489770E15246DB4547213EFB1A82C13F1E?height=17555855
2,237,609 4 52 https://www.mintscan.io/cosmos/tx/C8C319559AC24F665E5850784E2B21A2BAB650DE8568DEFF0BFFFD2942B2EEBF?height=17555871
623,468 1 13 https://www.mintscan.io/cosmos/tx/8357F6EDC1CF31140A10D2A66B4345ECCC0E3B2C5EB913717AF5A99B91258D5C?height=17555900
408,628 1 1 https://www.mintscan.io/cosmos/tx/AB5B92A877C6E6C7C10E4BE9E884EBA85810024A7F507943F31E5A895865C324?height=17555900

As you can see, the auto-claiming that is happening by default is increasing the gas amounts by a lot. But even the cases with 1 delegation and 1 auto-claim are higher than what the tool calculates by default.

Long term I think we should work towards gas simulation. But for now we can just bump the delegate gas amount. Will create a PR in a bit.

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 a pull request may close this issue.

1 participant