Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(arbiter-bin) forking state, disk i/o #563

Merged
merged 31 commits into from
Oct 3, 2023

Conversation

Autoparallel
Copy link
Collaborator

Give an overview of the tasks completed
This PR provides a new command fork for the Arbiter binary that allows for a user to fork state of contracts/storage from a certain block from an Ethereum chain. Everything for the fork will be set up in a .toml file, and an example one test_config.toml is given.

Link to issue(s) that this PR closes
Closes #461

@Autoparallel Autoparallel added the 🚧 WIP 🚧 DO NOT MERGE THIS PR label Oct 2, 2023
@Autoparallel Autoparallel self-assigned this Oct 2, 2023
@Autoparallel Autoparallel force-pushed the feat/arbiter-bin/forking branch from 22fb5be to ab93627 Compare October 3, 2023 14:36
@Autoparallel Autoparallel force-pushed the feat/arbiter-bin/forking branch from ab93627 to a743069 Compare October 3, 2023 14:42
Copy link
Collaborator

@0xJepsen 0xJepsen left a comment

Choose a reason for hiding this comment

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

I noticed sometime things don't compile with this error:

error[E0599]: no method named `into_environment_parameters` found for struct `EnvironmentBuilder` in the current scope
   --> /Users/shufflebottomhogwood/.cargo/git/checkouts/arbiter-e286ca1df4b4c9d0/1f58eee/arbiter-core/src/environment/builder.rs:131:31
    |
52  | pub struct EnvironmentBuilder {
    | ----------------------------- method `into_environment_parameters` not found for this struct
...
131 |         let parameters = self.into_environment_parameters();
    |                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^ method not found in `EnvironmentBuilder`

For more information about this error, try `rustc --explain E0599`.
error: could not compile `arbiter-core` (lib) due to previous error
warning: build failed, waiting for other jobs to finish...

@Autoparallel Autoparallel force-pushed the feat/arbiter-bin/forking branch from cc9ac7f to ce3c0f9 Compare October 3, 2023 16:58
Copy link
Contributor

@Alexangelj Alexangelj left a comment

Choose a reason for hiding this comment

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

Holy **** *** ********* ***

******* amazing

Looks excellent to me

@0xJepsen 0xJepsen self-requested a review October 3, 2023 18:21
@Autoparallel Autoparallel removed the 🚧 WIP 🚧 DO NOT MERGE THIS PR label Oct 3, 2023
@Autoparallel Autoparallel marked this pull request as ready for review October 3, 2023 22:44
Copy link
Collaborator

@0xJepsen 0xJepsen left a comment

Choose a reason for hiding this comment

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

Hell yeah brother

@0xJepsen 0xJepsen merged commit 1c640e7 into arbiter-bin/v0.4.0 Oct 3, 2023
@0xJepsen 0xJepsen deleted the feat/arbiter-bin/forking branch October 10, 2023 18:44
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants