krasi-georgiev
Repos
63
Followers
168
Following
22

Events

Please add timestamp_ms to the bidtrace api

the as what flashbot exposes in https://boost-relay.flashbots.net/relay/v1/data/bidtraces/builder_blocks_received?block_number=16046700 "timestamp_ms"

Created at 12 hours ago
opened issue
use an interface for core.ApplyTransaction to allow external use with adapters and mocking

Rationale

Why should this feature exist?

  • allow external use with adapters and mocking

What are the use-cases?

  • to allow state simulation without bootstrapping a full blockchain component.

Implementation

Do you have ideas regarding the implementation of this feature?

  • yes

Are you willing to implement this feature?

  • no
Created at 2 days ago
issue comment
Terminal: Copy on selection + new highlight in 1.68 copies previous term on CMD+F

What is the current status? Abandoned or added to a milestone?

Created at 3 days ago
closed issue
Add an endpoint to allow getting the TrieNode by a given root state hash.

Rationale

Why should this feature exist?

  • to allow quering archive nodes for the state at a given root hash

What are the use-cases?

  • running local simulations on top a given state at a good performance and minimal db/rpc calls.

Implementation

Do you have ideas regarding the implementation of this feature?

  • not yet, but willing to look into the code. In geth this is possible via the debug_dbget rpc call.

Are you willing to implement this feature?

  • yes
Created at 1 week ago
issue comment
Add an endpoint to allow getting the TrieNode by a given root state hash.

moved to using a different approach

Created at 1 week ago
issue comment
Allow operating a builder with no fee and no payout TX

Why does it matter that there could be other transfers to the coinbase?

The best block is the one that pays the most to the validator so even if there are other payments to the coinbase these are ok to assume as a reward.

The builder code does exactly that, just checks the coinbase balance before and after regardless of how the difference is accumulated.

Created at 1 week ago
issue comment
fix the ParseGas maxFeeGwei param

ping @bonedaddy

Created at 1 week ago
pull request closed
add docker build and for rpi4 arm64

Signed-off-by: Krasi Georgiev 8903888+krasi-georgiev@users.noreply.github.com

Created at 1 week ago
closed issue
Allow operating a builder with no fee and no payout TX

Rationale

Why should this feature exist?

  • when the builder doesn't collect any fee it is very enificient to create a payout TX on every block. some validators use a contract which consumes 70-100k gas so without the payout TX the total block value will be higher.

What are the use-cases?

  • operating a builder that doesn't collect block fees.

Implementation

Do you have ideas regarding the implementation of this feature?

  • yes the simulation checks -
    • if the coinbase address is the same as the validator's address
    • the coinbase balance difference is the same as the bid value

Are you willing to implement this feature?

  • yes
Created at 1 week ago
issue comment
Allow operating a builder with no fee and no payout TX

moved to https://github.com/flashbots/mev-boost/issues/419

Created at 1 week ago
opened issue
Allow operating a builder with no fee and no payout TX

Rationale

Why should this feature exist?

  • when the builder doesn't collect any fee it is very enificient to create a payout TX on every block. some validators use a contract which consumes 70-100k gas so without the payout TX the total block value will be higher.

What are the use-cases?

  • operating a builder that doesn't collect block fees.

Implementation

Do you have ideas regarding the implementation of this feature?

  • yes the simulation checks -
    • if the coinbase address is the same as the validator's address
    • the coinbase balance difference is the same as the bid value

Are you willing to implement this feature?

  • yes

cc @Ruteri

Created at 1 week ago
issue comment
Allow operating a builder with no fee and no payout TX

cc @Ruteri

Created at 1 week ago
opened issue
Allow operating a builder with no fee and no payout TX

Rationale

Why should this feature exist?

  • when the builder doesn't collect any fee it is very enificient to create a payout TX on every block. some validators use a contract which consumes 70-100k gas so without the payout TX the total block value will be higher.

What are the use-cases?

  • operating a builder that doesn't collect block fees.

Implementation

Do you have ideas regarding the implementation of this feature?

  • yes the simulation checks -
    • if the coinbase address is the same as the validator's address
    • the coinbase balance difference is the same as the bid value

Are you willing to implement this feature?

  • yes
Created at 1 week ago

added ToBigFloatDiv

Signed-off-by: Krasi Georgiev 8903888+krasi-georgiev@users.noreply.github.com

Created at 1 week ago

handle nil input on ToFloat, ToFloatDiv

Signed-off-by: Krasi Georgiev 8903888+krasi-georgiev@users.noreply.github.com

Created at 1 week ago
pull request closed
accounts/abi/backends: various changes/fixes for SimulatedBackend

added NewSimulatedBackendWithDatabaseAndGenesis for bootstrapping simulated with custom genesis params fixes missing error handling in a few functions added interface to get and set the coinbase address - needed to test builder simulations use proper faker for POS chain expose FilterSystem to allow head subscriptions

Signed-off-by: Krasi Georgiev 8903888+krasi-georgiev@users.noreply.github.com

Created at 1 week ago
issue comment
Simulated backend doesn't allow creating a beacon chain

I am testing coinbase balance changes and when running the simulator for every produced block the coinbase is rewarded 2eth which is not in line with the current consensus.

Created at 2 weeks ago

fix log panic for nil big number

Signed-off-by: Krasi Georgiev 8903888+krasi-georgiev@users.noreply.github.com

Created at 2 weeks ago
issue comment
Various changes and fixed in the simulated backend

thanks, updated, please check and let me know what else needs changing

Created at 2 weeks ago
issue comment
add Antratek and g62 parser

Last thing is to add a test for Antratek and G62 functions.

Created at 2 weeks ago