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.

Sign up for GitHub

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

Jump to bottom

Transactions running out of gas #225

Closed
danfinlay opened this issue May 26, 2016 · 4 comments
Closed

Transactions running out of gas #225

danfinlay opened this issue May 26, 2016 · 4 comments
Labels
type-bug
Milestone
MVP

Comments

@danfinlay
Copy link
Contributor

I'm just playing with a sample Truffle app. This is stopping me from successfully submitting an txs to my local testrpc!
@danfinlay danfinlay added this to the Developer Beta milestone May 26, 2016
@danfinlay danfinlay changed the title Transactions running out of gas again Transactions running out of gas May 26, 2016
@danfinlay
Copy link
Contributor Author

Oh sorry, @tcoulter this is the issue I meant to point you to. That other issue is about how we render out of gas errors in Metamask.

We're getting weird out of gas errors and I was hoping you could try to reproduce.

@niran
Copy link

niran commented May 28, 2016

This looks like #60 that was revived because the workaround was removed in provider-engine. I'm sticking with web3-provider-engine@7.6.2 for now, but it looks like ethereumjs-vm has broken gas estimation.

@danfinlay
Copy link
Contributor Author

It may be captured in this issue at ethereumjs-vm.

kumavis referenced this issue in MetaMask/web3-provider-engine May 30, 2016
@kumavis
hot fix - re-add estimateGas 1.5x multiplier
84dee23
@kumavis kumavis added the ready label Jun 3, 2016
@kumavis
Copy link
Member

kumavis commented Jun 3, 2016

fixed here trufflesuite/ganache-cli-archive#99

@kumavis kumavis closed this as completed Jun 3, 2016
@kumavis kumavis removed the ready label Jun 3, 2016
SeungjunWe pushed a commit to DcentWallet/metamask-extension that referenced this issue Jul 13, 2020
@vbaranov
Merge pull request MetaMask#225 from dennis00010011b/e2e-outputs
99c79e0 
e2e: test different types of outputs for feature 'Ecxecute method'
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type-bug
Projects
None yet
Milestone
MVP
Development

No branches or pull requests
3 participants
@niran @danfinlay @kumavis