Skip to content

feat(l1): notebook for high-level profiling #3633

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 23, 2025
Merged

feat(l1): notebook for high-level profiling #3633

merged 1 commit into from
Jul 23, 2025

Conversation

Oppen
Copy link
Contributor

@Oppen Oppen commented Jul 14, 2025

Introduce a new notebook to analyze contribution of eaxh part of the
block production process to its overall time, producing graphs for
visual clarity.
Instructions included in the README.

Based on #3274
Coauthored-by: @Arkenan

Part of: #3331

@Oppen Oppen requested a review from a team as a code owner July 14, 2025 19:37
@review-notebook-app Review Notebook App
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@github-actions github-actions bot added the L1 Ethereum client label Jul 14, 2025
@Oppen Oppen force-pushed the log_analysis_p2 branch from f9a9e6c to 80e3dd6 Compare July 14, 2025 19:39
@Oppen Oppen force-pushed the log_merklization_time branch from 45c01a2 to 0d2cb3e Compare July 14, 2025 19:59
Base automatically changed from log_merklization_time to main July 18, 2025 17:00
@Oppen
feat(l1): notebook for high-level profiling
3eedb15 
Introduce a new notebook to analyze contribution of eaxh part of the
block production process to its overall time, producing graphs for
visual clarity.

Based on #3274
Coauthored-by: @Arkenan
@Oppen Oppen force-pushed the log_analysis_p2 branch from 80e3dd6 to 3eedb15 Compare July 22, 2025 18:58
@github-actions GitHub Actions
Copy link

Lines of code report

Total lines added: 0
Total lines removed: 1
Total lines changed: 1

Detailed view 
+----------------------------------+-------+------+
| File                             | Lines | Diff |
+----------------------------------+-------+------+
| ethrex/cmd/ef_tests/state/lib.rs | 6     | -1   |
+----------------------------------+-------+------+

@github-project-automation github-project-automation bot moved this to In Review in ethrex_l1 Jul 23, 2025
@Arkenan Arkenan added this pull request to the merge queue Jul 23, 2025
Merged via the queue into main with commit 1802f66 Jul 23, 2025
31 checks passed
@Arkenan Arkenan deleted the log_analysis_p2 branch July 23, 2025 17:26
@github-project-automation github-project-automation bot moved this from In Review to Done in ethrex_l1 Jul 23, 2025
pedrobergamini pushed a commit to pedrobergamini/ethrex that referenced this pull request Aug 24, 2025
@Oppen
feat(l1): notebook for high-level profiling (lambdaclass#3633)
379a084 
Introduce a new notebook to analyze contribution of eaxh part of the
block production process to its overall time, producing graphs for
visual clarity.
Instructions included in the README.

Based on lambdaclass#3274
Coauthored-by: @Arkenan

Part of: lambdaclass#3331
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rodrigo-o rodrigo-o rodrigo-o approved these changes

@Arkenan Arkenan Arkenan approved these changes

Assignees

@Oppen Oppen

Labels
L1 Ethereum client
Projects
ethrex_l1
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Oppen @rodrigo-o @Arkenan