Open 0xSulpiride opened 2 weeks ago
I'm taking a look into this.
Thank you @nikhilkumar1612
Do give us a shout if you have any questions.
hey i have a few questions:
Hi @nikhilkumar1612 thanks for picking up this issue!
CallGasEstimationProxy
is a proxy that delegates all calls to the _implementation()
contract (if no function from it's body matches the selector). So we need 2 overrides in eth_call
. One that overrides EntryPoint's address to CallGasEstimationProxy
's code and another to override PROXY_IMPLEMENTATION_ADDRESS_MARKER
to EntryPoint
's code. This way when we call simulateHandleOps
on CallGasEstimationProxy
, proxy will delegatecall it to EntryPoint
Let me know if you have more questions!
thanks @0xSulpiride . I have completed the changes needed for the types, are these changes relavant to v6 as well ? or is this enhancement only relavant to v7 ?
@nikhilkumar1612 this is only relevant to v7
thanks @0xSulpiride . I have completed the changes needed for the types, are these changes relavant to v6 as well ? or is this enhancement only relavant to v7 ?
Thank you @nikhilkumar1612 Please create a PR when ready.
hi guys please find my PR and let me know the changes. @0xSulpiride @ch4r10t33r
Is your feature request related to a problem? Please describe. Skandha started to misestimate gas limits of userops that use EntryPoint v0.7.0. We need to switch to smart-contract based binary search algorithm for better estimation of gas limits.
Describe the solution you'd like We should implement the same approach that Rundler does. They use this contract: https://github.com/alchemyplatform/rundler/blob/307e8c2cbeab80518a56a2f309e994de3b1631f0/crates/types/contracts/src/v0_7/CallGasEstimationProxy.sol