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

Estimate transaction gas limit properly #2921

Closed
palango opened this Issue Oct 29, 2018 · 2 comments

Comments

Projects
None yet
2 participants
@palango
Collaborator

palango commented Oct 29, 2018

Currently we estimate the gas limit with the following code:

    def gaslimit(self, location='latest') -> int:
        gas_limit = self.web3.eth.getBlock(location)['gasLimit']
        return gas_limit * 8 // 10

In order to make the gas reserve work as intended and to lower the minimal amount of Ether a raiden user has to possess we should properly estimate the gas cost of transactions.

@palango palango added the enhancement label Oct 29, 2018

@palango

This comment has been minimized.

Collaborator

palango commented Oct 29, 2018

Work on this in done in #2623

@palango

This comment has been minimized.

Collaborator

palango commented Oct 30, 2018

@andrevmatos this was the issue we talked about.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment