Skip to content

Conversation

@xqft
Copy link
Contributor

@xqft xqft commented Dec 1, 2025

Motivation

To measure execution and proving time correctly. This means removing the client initialization from the equation. In the case of ZisK, we use it's own timing exposed via a JSON file generated after proving.

@github-actions github-actions bot added the L2 Rollup client label Dec 1, 2025
@github-actions
Copy link

github-actions bot commented Dec 1, 2025

Lines of code report

Total lines added: 105
Total lines removed: 0
Total lines changed: 105

Detailed view
+----------------------------------------------+-------+------+
| File                                         | Lines | Diff |
+----------------------------------------------+-------+------+
| ethrex/crates/l2/prover/src/backend/exec.rs  | 58    | +16  |
+----------------------------------------------+-------+------+
| ethrex/crates/l2/prover/src/backend/risc0.rs | 130   | +25  |
+----------------------------------------------+-------+------+
| ethrex/crates/l2/prover/src/backend/sp1.rs   | 153   | +25  |
+----------------------------------------------+-------+------+
| ethrex/crates/l2/prover/src/lib.rs           | 97    | +39  |
+----------------------------------------------+-------+------+

@ilitteri
Copy link
Contributor

ilitteri commented Dec 1, 2025

Superseded by #5478

@ilitteri ilitteri closed this Dec 1, 2025
@github-project-automation github-project-automation bot moved this to Done in ethrex_l2 Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

L2 Rollup client

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants