-
Notifications
You must be signed in to change notification settings - Fork 358
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
[frontier] Add support for PoV gas refunding #3036
Conversation
test/suites/dev/moonbase/test-storage-growth/test-precompile-storage-growth.ts
Outdated
Show resolved
Hide resolved
WASM runtime size check:Compared to target branchMoonbase runtime: 2248 KB (no changes) ✅ Moonbeam runtime: 2224 KB (no changes) ✅ Moonriver runtime: 2224 KB (no changes) ✅ Compared to latest release (runtime-3300)Moonbase runtime: 2248 KB (+220 KB compared to latest release) Moonbeam runtime: 2224 KB (+228 KB compared to latest release) Moonriver runtime: 2224 KB (+232 KB compared to latest release) |
Coverage Report@@ Coverage Diff @@
## master agustin-frontier-pov-reclaim +/- ##
================================================================
+ Coverage 74.75% 78.98% +4.23%
- Files 369 303 -66
- Lines 94246 88231 -6015
================================================================
- Hits 70453 69685 -768
- Misses 23793 18546 -5247
|
@Agusrodri I can see the ref_time of weight (v2) jumps from 93M to 7M. Is that normal ? |
Not sure if this is normal to be honest. From my understanding the refund should only take place in the pov size right? Will need to investigate. |
After more consideration, it is expected to also be lower. For Ethereum transaction, the weight (ref_time) is computed from the gas cost, using a gas to weight ratio. The gas being lower (due to PoV) the ref_time is lower being lower. So all good |
What does it do?
Enables PoV gas refunding, which means that the difference between the benchmarked worst-case and the real proof-size consumption is now reclaimed back to the user.
Related frontier (fork) PRs