Skip to content
This repository was archived by the owner on Mar 13, 2023. It is now read-only.

Gas to weight = ? #718

Merged
merged 7 commits into from
Jul 26, 2021
Merged

Gas to weight = ? #718

merged 7 commits into from
Jul 26, 2021

Conversation

boundless-forest
Copy link
Member

@boundless-forest boundless-forest commented Jul 22, 2021

Close #705

This pull request answers a question this time in DVM. 1 gas =? weight

From substrate.dev:
Substrate defines one unit of weight as one picosecond of execution time, that is 10^12 weight = 1 second, or 1,000 weight = 1 nanosecond, on fixed reference hardware (Intel Core i7-7700K CPU with 64GB of RAM and an NVMe SSD).

According to the benchmark result, we set the radio of gas and weight 16_000.

@boundless-forest boundless-forest force-pushed the gas-to-weight-benchmark branch from 31b2ab1 to dfb6dbb Compare July 23, 2021 08:22
@boundless-forest boundless-forest marked this pull request as ready for review July 23, 2021 08:44
@boundless-forest boundless-forest added P-High [Priority] High N-Pangolin [Network] Pangolin labels Jul 23, 2021
Copy link
Contributor

@hackfisher hackfisher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@boundless-forest boundless-forest added the C-EVM [Component] Something about EVM label Jul 26, 2021
@aurexav aurexav merged commit 88588e1 into master Jul 26, 2021
@aurexav aurexav deleted the gas-to-weight-benchmark branch July 26, 2021 09:08
@aurexav aurexav mentioned this pull request Aug 3, 2021
11 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
C-EVM [Component] Something about EVM N-Pangolin [Network] Pangolin P-High [Priority] High
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The ratio of the gas to weight
3 participants