Skip to content

[DNM] With Trace Dump #110

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

Closed
wants to merge 258 commits into from
Closed

[DNM] With Trace Dump #110

wants to merge 258 commits into from

Conversation

JulianGCalderon
Copy link
Contributor

@JulianGCalderon JulianGCalderon commented Dec 30, 2024

Adds support for Trace Dump with both Sierra Emu and Cairo Native. See https://lambdaclass.github.io/cairo_native/cairo_native/docs/section06/index.html#comparing-with-sierra-emulator

Requires to use:

You may need to update Cairo Native's branch with the latest changes.

Remember to delete compiled programs (rm -rf compiled_programs).

Setup

Clone repositories

git clone [email protected]:lambdaclass/starknet-replay.git
git clone [email protected]:lambdaclass/sequencer.git
git clone [email protected]:lambdaclass/cairo_native.git
git clone [email protected]:lambdaclass/sierra-emu.git

Adapt Cairo native

cd cairo_native
git checkout add-trace-dump-contracts
make runtime-with-trace-dump

Adapt Sequencer

cd sequencer
git checkout replay-w-trace-dump

Adapt Replay

cd starknet-replay
git checkout w-trace-dump

Usage

To run with Native:

cargo run --features with-trace-dump \
  tx 0x04c397201320923d0b4a25b08ada7a5323fcb2c94bc7d865ac4484634674eef2 mainnet 901178

To run with Sierra Emu:

cargo run --features with-sierra-emu \
  tx 0x04c397201320923d0b4a25b08ada7a5323fcb2c94bc7d865ac4484634674eef2 mainnet 901178

Copy link

github-actions bot commented Dec 30, 2024

✅ Code is now correctly formatted.

@JulianGCalderon JulianGCalderon changed the base branch from add-sierra-emu to main January 17, 2025 21:17
@JulianGCalderon JulianGCalderon force-pushed the w-trace-dump branch 2 times, most recently from 0e19ffe to ce062a1 Compare January 17, 2025 21:21
@JulianGCalderon JulianGCalderon changed the base branch from main to bump-cairo-2.10.0 February 20, 2025 19:31
@FrancoGiachetta FrancoGiachetta changed the base branch from bump-cairo-2.10.0 to main April 14, 2025 15:47
@JulianGCalderon
Copy link
Contributor Author

This has been merged into main already. #156

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