Skip to content
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

prover: implement token methods, gas price, fix copy tracer on deploy #195

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

x-mass
Copy link
Member

@x-mass x-mass commented Feb 5, 2025

To enhance the tracing capabilities for a broader set of contracts, several new methods need to be implemented: AddToken, SubToken, SetTokenTransfer, AddRefund, SubRefund, and GetGasPrice. This pull request enables us to gather traces from every block produced by the nil_load_generator.
Additionally, a bug in the copy tracer has been fixed; previously, there was a discrepancy between the execution EVM and the tracer EVM during the deployment transaction.

@knazarov
Copy link
Collaborator

knazarov commented Feb 7, 2025

If this PR is not a draft, it needs a description that explains "why" this change is made and goes into details of key changes.

…ate db

To enhance the tracing capabilities for a broader set of contracts, several new methods need to be implemented: `AddToken`, `SubToken`, `SetTokenTransfer`, `AddRefund`, `SubRefund`, and `GetGasPrice`. This commit enables us to gather traces from every block produced by the `nil_load_generator`.
@x-mass x-mass force-pushed the prover-token-methods branch from 1a1befa to 7c6d7d9 Compare February 11, 2025 08:35
Copy link
Contributor

@akokoshn akokoshn left a comment

Choose a reason for hiding this comment

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

We also need implementation for GetConfigAccessor, panic fails on unit tests TestTracerNildTestSuite after enable feature which should be in main soon (#249)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants