Skip to content

Show gas per second in the UI#1279

Merged
arcz merged 1 commit intomasterfrom
gasPerSecond
Jun 26, 2024
Merged

Show gas per second in the UI#1279
arcz merged 1 commit intomasterfrom
gasPerSecond

Conversation

@samalws-tob
Copy link
Copy Markdown
Collaborator

@samalws-tob samalws-tob commented Jun 26, 2024

This will help for determining performance. We keep track of gas consumption after each transaction call, rather than after each instruction: hevm handles the bookkeeping at the instruction level. Should have a very minimal effect on performance, since hevm already keeps track of gas consumption strictly: burned :: !(Gas t) (hevm/src/EVM/Types.hs:620).

I'm unsure whether we should be keeping track of totalGas as an Int or as hevm's native Gas type. One way or another we need to cast to a different type to display it onscreen. In any case this choice is only a minor issue

@arcz arcz merged commit 882c699 into master Jun 26, 2024
@arcz arcz deleted the gasPerSecond branch June 26, 2024 20:50
datradito pushed a commit to datradito/echidna-mcp that referenced this pull request Dec 29, 2025
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 this pull request may close these issues.

2 participants